linter: fix linting issues (microsoft#1457)

anmaxvl · web-flow · commit 873b8e122faa · 2022-07-14T12:12:00.000-07:00
Additionally fix the queue tests logic.

Signed-off-by: Maksim An &lt;maksiman@microsoft.com&gt;
diff --git a/internal/queue/queue_test.go b/internal/queue/queue_test.go
@@ -50,6 +50,7 @@ func TestEnqueueDequeueClose(t *testing.T) {
 				// go around make sure we get ErrClosed()
 				q.Close()
 			}
+			continue
 		} else if err != ErrQueueClosed {
 			t.Fatalf("expected to receive ErrQueueClosed, instead got: %s", err)
 		}
@@ -177,5 +178,9 @@ func TestDequeueBlock(t *testing.T) {
 		if err != nil {
 			t.Fatal(err)
 		}
+	case <-time.After(10 * time.Second):
+		// Closing the queue will finish the Dequeue go routine.
+		q.Close()
+		t.Fatal("timeout waiting for Dequeue go routine to complete")
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -50,6 +50,7 @@ func TestEnqueueDequeueClose(t *testing.T) {`
`50`	`50`	`// go around make sure we get ErrClosed()`
`51`	`51`	`q.Close()`
`52`	`52`	`}`
	`53`	`+ continue`
`53`	`54`	`} else if err != ErrQueueClosed {`
`54`	`55`	`t.Fatalf("expected to receive ErrQueueClosed, instead got: %s", err)`
`55`	`56`	`}`
`@@ -177,5 +178,9 @@ func TestDequeueBlock(t *testing.T) {`
`177`	`178`	`if err != nil {`
`178`	`179`	`t.Fatal(err)`
`179`	`180`	`}`
	`181`	`+ case <-time.After(10 * time.Second):`
	`182`	`+ // Closing the queue will finish the Dequeue go routine.`
	`183`	`+ q.Close()`
	`184`	`+ t.Fatal("timeout waiting for Dequeue go routine to complete")`
`180`	`185`	`}`
`181`	`186`	`}`