fix: address PR review comments

- Pin DocumentDB image to version 1.0.0
- Fix port spec consistency ("10260/tcp")
- Add slices.SortFunc for stable DB ordering
- Add ping verification to setupMongoDB
- Require all 3 DBs in TestMultiContainer
- Fix distinct test to assert both values explicitly
- Remove fuzz tests

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: h3n4l

collection_test.go

@@ -1913,7 +1913,8 @@ func TestDistinct(t *testing.T) {
 		for _, row := range result.Rows {
 			values[row] = true
 		}
-		require.True(t, values[`"active"`] || values[`"inactive"`])
+		require.True(t, values[`"active"`])
+		require.True(t, values[`"inactive"`])
 	})
 }
 

fuzz_test.go

@@ -4,6 +4,7 @@ import (
 	"context"
 	"crypto/tls"
 	"fmt"
+	"slices"
 	"sync"
 	"testing"
 	"time"
@@ -115,6 +116,17 @@ func setupContainers(ctx context.Context) ([]TestDB, error) {
 		return nil, fmt.Errorf("container setup failed: %v", errs)
 	}
 
+	// Sort by name for stable ordering
+	slices.SortFunc(dbs, func(a, b TestDB) int {
+		if a.Name < b.Name {
+			return -1
+		}
+		if a.Name > b.Name {
+			return 1
+		}
+		return 0
+	})
+
 	return dbs, nil
 }
 
@@ -134,6 +146,13 @@ func setupMongoDB(ctx context.Context, name, image string) (TestDB, error) {
 		return TestDB{}, err
 	}
 
+	// Verify connection
+	pingCtx, cancel := context.WithTimeout(ctx, 10*time.Second)
+	defer cancel()
+	if err := client.Ping(pingCtx, nil); err != nil {
+		return TestDB{}, fmt.Errorf("ping failed: %w", err)
+	}
+
 	return TestDB{Name: name, Client: client}, nil
 }
 
@@ -158,7 +177,7 @@ func setupDocumentDB(ctx context.Context) (TestDB, error) {
 		return TestDB{}, err
 	}
 
-	port, err := container.MappedPort(ctx, "10260")
+	port, err := container.MappedPort(ctx, "10260/tcp")
 	if err != nil {
 		return TestDB{}, err
 	}

internal/testutil/container.go

@@ -11,7 +11,7 @@ import (
 
 func TestMultiContainer(t *testing.T) {
 	dbs := GetAllClients(t)
-	require.GreaterOrEqual(t, len(dbs), 2) // At least mongo4 and mongo8
+	require.Equal(t, 3, len(dbs)) // All three databases must be available: documentdb, mongo4, mongo8
 	for _, db := range dbs {
 		require.NotEmpty(t, db.Name)
 		require.NotNil(t, db.Client)