Remove unnecessary panic from concurrency test

datacharmer · datacharmer · commit 6a2cc84caf81 · 2022-08-29T20:30:40.000+02:00
diff --git a/concurrent/concurrent_test.go b/concurrent/concurrent_test.go
@@ -97,7 +97,7 @@ func TestConcurrency(t *testing.T) {
 	for _, item := range testData {
 		wantedIndex := item.expectedIndex
 		if len(results) < wantedIndex {
-			panic(fmt.Errorf("result does not have %d items", wantedIndex))
+			t.Fatalf("result does not have %d items", wantedIndex)
 		}
 		if results[wantedIndex] == item.ID {
 			t.Logf("ok - Item %-10s is at position %d in the results\n", item.ID, wantedIndex)

Original file line number	Diff line number	Diff line change
`@@ -97,7 +97,7 @@ func TestConcurrency(t *testing.T) {`
`97`	`97`	`for _, item := range testData {`
`98`	`98`	`wantedIndex := item.expectedIndex`
`99`	`99`	`if len(results) < wantedIndex {`
`100`		`- panic(fmt.Errorf("result does not have %d items", wantedIndex))`
	`100`	`+ t.Fatalf("result does not have %d items", wantedIndex)`
`101`	`101`	`}`
`102`	`102`	`if results[wantedIndex] == item.ID {`
`103`	`103`	`t.Logf("ok - Item %-10s is at position %d in the results\n", item.ID, wantedIndex)`