diff --git a/executor_test.go b/executor_test.go index 4ac1d20ea..87b72689c 100644 --- a/executor_test.go +++ b/executor_test.go @@ -3035,8 +3035,7 @@ func TestExecutor_Execute_GroupBy(t *testing.T) { t.Run("test wrapping with previous", func(t *testing.T) { totalResults := make([]pilosa.GroupCount, 0) - var results []pilosa.GroupCount - results = c.Query(t, "i", `GroupBy(Rows(field=ppa), Rows(field=ppb), Rows(field=ppc), limit=3)`).Results[0].([]pilosa.GroupCount) + results := c.Query(t, "i", `GroupBy(Rows(field=ppa), Rows(field=ppb), Rows(field=ppc), limit=3)`).Results[0].([]pilosa.GroupCount) totalResults = append(totalResults, results...) for len(totalResults) < 64 { lastGroup := results[len(results)-1].Group diff --git a/fragment_internal_test.go b/fragment_internal_test.go index eaf28acfe..4e3007957 100644 --- a/fragment_internal_test.go +++ b/fragment_internal_test.go @@ -2080,7 +2080,7 @@ func TestFragmentRowIterator(t *testing.T) { if id != i { t.Fatalf("expected row %d but got %d", i, id) } - if wrapped != false { + if wrapped { t.Fatalf("shouldn't have wrapped") } if !reflect.DeepEqual(row.Columns(), []uint64{0}) { @@ -2094,7 +2094,7 @@ func TestFragmentRowIterator(t *testing.T) { if id != 0 { t.Fatalf("id should be 0 after iterator is exhausted, got %d", id) } - if wrapped != true { + if !wrapped { t.Fatalf("wrapped should be true after iterator is exhausted") } f.Close() @@ -2113,7 +2113,7 @@ func TestFragmentRowIterator(t *testing.T) { if id != i { t.Fatalf("expected row %d but got %d", i, id) } - if wrapped != false { + if wrapped { t.Fatalf("shouldn't have wrapped") } if !reflect.DeepEqual(row.Columns(), []uint64{0}) { @@ -2127,7 +2127,7 @@ func TestFragmentRowIterator(t *testing.T) { if id != 0 { t.Fatalf("id should be 0 after iterator is exhausted, got %d", id) } - if wrapped != true { + if !wrapped { t.Fatalf("wrapped should be true after iterator is exhausted") } f.Close()