From dfed899c08d6b2798f965f730867f00a82ba9950 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sybren=20A=2E=20St=C3=BCvel?= Date: Mon, 11 Nov 2024 19:20:41 +0100 Subject: [PATCH] Refactor: ensure test function params are in (expected, actual) order No functional changes --- internal/manager/persistence/jobs_blocklist_test.go | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/internal/manager/persistence/jobs_blocklist_test.go b/internal/manager/persistence/jobs_blocklist_test.go index 8894e681..0163ff28 100644 --- a/internal/manager/persistence/jobs_blocklist_test.go +++ b/internal/manager/persistence/jobs_blocklist_test.go @@ -56,9 +56,9 @@ func TestFetchJobBlocklist(t *testing.T) { if assert.Len(t, list, 1) { entry := list[0] - assert.Equal(t, entry.JobID, job.ID) - assert.Equal(t, entry.WorkerID, worker.ID) - assert.Equal(t, entry.TaskType, "blender") + assert.Equal(t, job.ID, entry.JobID) + assert.Equal(t, worker.ID, entry.WorkerID) + assert.Equal(t, "blender", entry.TaskType) assert.Nil(t, entry.Job, "should NOT fetch the entire job") assert.NotNil(t, entry.Worker, "SHOULD fetch the entire worker") @@ -107,9 +107,9 @@ func TestRemoveFromJobBlocklist(t *testing.T) { if assert.Len(t, list, 1) { entry := list[0] - assert.Equal(t, entry.JobID, job.ID) - assert.Equal(t, entry.WorkerID, worker.ID) - assert.Equal(t, entry.TaskType, "blender") + assert.Equal(t, job.ID, entry.JobID) + assert.Equal(t, worker.ID, entry.WorkerID) + assert.Equal(t, "blender", entry.TaskType) } }