diff options
| author | TheFox0x7 <thefox0x7@gmail.com> | 2024-07-30 21:41:10 +0200 |
|---|---|---|
| committer | Earl Warren <earl-warren@noreply.codeberg.org> | 2024-07-30 21:41:10 +0200 |
| commit | 4de909747bdf322bbb37d500b9705d7a3a050b78 (patch) | |
| tree | 62dad5d457c227628877d235f0105db9ae4bc200 /services/issue/assignee_test.go | |
| parent | Merge pull request 'Implement external release assets' (#1445) from maltejur/... (diff) | |
| download | forgejo-4de909747bdf322bbb37d500b9705d7a3a050b78.tar.xz forgejo-4de909747bdf322bbb37d500b9705d7a3a050b78.zip | |
Add testifylint to lint checks (#4535)
go-require lint is ignored for now
Reviewed-on: https://codeberg.org/forgejo/forgejo/pulls/4535
Reviewed-by: Gusted <gusted@noreply.codeberg.org>
Co-authored-by: TheFox0x7 <thefox0x7@gmail.com>
Co-committed-by: TheFox0x7 <thefox0x7@gmail.com>
Diffstat (limited to 'services/issue/assignee_test.go')
| -rw-r--r-- | services/issue/assignee_test.go | 15 |
1 files changed, 8 insertions, 7 deletions
diff --git a/services/issue/assignee_test.go b/services/issue/assignee_test.go index 38d56f9d9d..2b70b8c8ce 100644 --- a/services/issue/assignee_test.go +++ b/services/issue/assignee_test.go @@ -12,36 +12,37 @@ import ( user_model "code.gitea.io/gitea/models/user" "github.com/stretchr/testify/assert" + "github.com/stretchr/testify/require" ) func TestDeleteNotPassedAssignee(t *testing.T) { - assert.NoError(t, unittest.PrepareTestDatabase()) + require.NoError(t, unittest.PrepareTestDatabase()) // Fake issue with assignees issue, err := issues_model.GetIssueByID(db.DefaultContext, 1) - assert.NoError(t, err) + require.NoError(t, err) err = issue.LoadAttributes(db.DefaultContext) - assert.NoError(t, err) + require.NoError(t, err) assert.Len(t, issue.Assignees, 1) user1, err := user_model.GetUserByID(db.DefaultContext, 1) // This user is already assigned (see the definition in fixtures), so running UpdateAssignee should unassign him - assert.NoError(t, err) + require.NoError(t, err) // Check if he got removed isAssigned, err := issues_model.IsUserAssignedToIssue(db.DefaultContext, issue, user1) - assert.NoError(t, err) + require.NoError(t, err) assert.True(t, isAssigned) // Clean everyone err = DeleteNotPassedAssignee(db.DefaultContext, issue, user1, []*user_model.User{}) - assert.NoError(t, err) + require.NoError(t, err) assert.Empty(t, issue.Assignees) // Reload to check they're gone issue.ResetAttributesLoaded() - assert.NoError(t, issue.LoadAssignees(db.DefaultContext)) + require.NoError(t, issue.LoadAssignees(db.DefaultContext)) assert.Empty(t, issue.Assignees) assert.Empty(t, issue.Assignee) } |