diff --git a/server/.golangci.yml b/server/.golangci.yml index 9d7bf8f79cb..e150ec2418d 100644 --- a/server/.golangci.yml +++ b/server/.golangci.yml @@ -92,7 +92,6 @@ issues: channels/app/helper_test.go|\ channels/app/import_functions.go|\ channels/app/integration_action.go|\ - channels/app/job_test.go|\ channels/app/migrations.go|\ channels/app/permissions.go|\ channels/app/permissions_test.go|\ diff --git a/server/channels/app/job_test.go b/server/channels/app/job_test.go index 6ca5ee2d6ae..695ffc39688 100644 --- a/server/channels/app/job_test.go +++ b/server/channels/app/job_test.go @@ -25,8 +25,10 @@ func TestGetJob(t *testing.T) { _, err := th.App.Srv().Store().Job().Save(status) require.NoError(t, err) - - defer th.App.Srv().Store().Job().Delete(status.Id) + defer func() { + _, err = th.App.Srv().Store().Job().Delete(status.Id) + require.NoError(t, err) + }() received, appErr := th.App.GetJob(th.Context, status.Id) require.Nil(t, appErr) @@ -241,7 +243,10 @@ func TestGetJobByType(t *testing.T) { for _, status := range statuses { _, err := th.App.Srv().Store().Job().Save(status) require.NoError(t, err) - defer th.App.Srv().Store().Job().Delete(status.Id) + defer func() { + _, err = th.App.Srv().Store().Job().Delete(status.Id) + require.NoError(t, err) + }() } received, err := th.App.GetJobsByType(th.Context, jobType, 0, 2) @@ -285,7 +290,10 @@ func TestGetJobsByTypes(t *testing.T) { for _, status := range statuses { _, err := th.App.Srv().Store().Job().Save(status) require.NoError(t, err) - defer th.App.Srv().Store().Job().Delete(status.Id) + defer func() { + _, err = th.App.Srv().Store().Job().Delete(status.Id) + require.NoError(t, err) + }() } jobTypes := []string{jobType, jobType1, jobType2}