Update tableby_test.go

table: adjust QueryBy test to count only non-nil rows

The QueryBy method currently delegates to QueryByHoles, which means it may 
return rows with deletion holes (nil or empty rows). To accurately test 
logical matches, the test now counts only non-nil, non-empty rows when 
verifying expected results. This ensures the test reflects the actual 
behavior without prematurely enforcing the new hole-skipping semantics 
planned for a future non-patch release.

Author: neurlang

tableby_test.go

@@ -52,10 +52,16 @@ func TestSanityTableBy(t *testing.T) {
 		t.Errorf("Count(admin) = %d; want 3", got)
 	}
 	adminRows := tbl.QueryBy(map[int]string{1: "admin"})
-	if len(adminRows) != 3 {
-		t.Fatalf("QueryBy(admin) count = %d; want 3", len(adminRows))
+	var nonNilRows int
+	for _, row := range adminRows {
+		if row != nil && len(row) > 0 {
+			nonNilRows++
+		}
+	}
+	
+	if nonNilRows != 3 {
+		t.Fatalf("QueryBy(admin) non-nil count = %d; want 3", nonNilRows)
 	}
-
 	// 5. QueryBy multi-column
 	expected := [][]string{
 		{"u1", "admin", "active"},