Replace t.Fatalf with require in tests

AI disclosure: Claude Code assisted with development. Every line of code was either written by or carefully reviewed by me :)
Signed-off-by: Brett Wines <bwines@slack-corp.com>

Author: bgwines

go/sync2/consolidator_test.go

@@ -21,6 +21,8 @@ import (
 	"sync"
 	"testing"
 
+	"github.com/stretchr/testify/require"
+
 	"vitess.io/vitess/go/sqltypes"
 )
 
@@ -49,30 +51,20 @@ func TestAddWaiterCount(t *testing.T) {
 	wgAdd.Wait()
 	wgSub.Wait()
 
-	if con.TotalWaiterCount() != 0 {
-		t.Fatalf("Expect 0 totalWaiterCount but got: %d", con.TotalWaiterCount())
-	}
+	require.Zero(t, con.TotalWaiterCount(), "expected 0 totalWaiterCount")
 }
 
 func TestHasWaiters(t *testing.T) {
 	con := NewConsolidator()
 	sql := "select * from SomeTable"
 
 	orig, created := con.Create(sql)
-	if !created {
-		t.Fatalf("expected consolidator to register a new entry")
-	}
-	if orig.HasWaiters() {
-		t.Fatalf("expected no waiters for a fresh entry")
-	}
+	require.True(t, created, "expected consolidator to register a new entry")
+	require.False(t, orig.HasWaiters(), "expected no waiters for a fresh entry")
 
 	_, created = con.Create(sql)
-	if created {
-		t.Fatalf("did not expect consolidator to register a new entry")
-	}
-	if !orig.HasWaiters() {
-		t.Fatalf("expected waiters after a duplicate Create")
-	}
+	require.False(t, created, "did not expect consolidator to register a new entry")
+	require.True(t, orig.HasWaiters(), "expected waiters after a duplicate Create")
 
 	orig.SetResult(&sqltypes.Result{})
 	orig.Broadcast()