refactor: move tests to proper files, mark replace as temporary

- Move TestPrettyNoOp to collection_test.go (cursor method behavior)
- Move TestInt32StringArg to bson_helpers_test.go (BSON helper behavior)
- Delete compat_test.go
- Add TODO comment on replace directive (remove after parser published)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: h3n4l

collection_test.go

@@ -2262,3 +2262,37 @@ func TestCursorMinMethod(t *testing.T) {
 		require.Equal(t, 2, len(result.Value))
 	})
 }
+
+func TestPrettyNoOp(t *testing.T) {
+	testutil.RunOnAllDBs(t, func(t *testing.T, db testutil.TestDB) {
+		dbName := fmt.Sprintf("testdb_pretty_%s", db.Name)
+		defer testutil.CleanupDatabase(t, db.Client, dbName)
+
+		ctx := context.Background()
+		collection := db.Client.Database(dbName).Collection("users")
+		_, err := collection.InsertMany(ctx, []any{
+			bson.M{"name": "alice", "age": 30},
+			bson.M{"name": "bob", "age": 25},
+		})
+		require.NoError(t, err)
+
+		gc := gomongo.NewClient(db.Client)
+
+		// pretty() should be a no-op cursor method
+		result, err := gc.Execute(ctx, dbName, `db.users.find().pretty()`)
+		require.NoError(t, err)
+		require.Equal(t, 2, len(result.Value))
+
+		// pretty() chained after sort
+		result, err = gc.Execute(ctx, dbName, `db.users.find().sort({name: 1}).pretty()`)
+		require.NoError(t, err)
+		require.Equal(t, 2, len(result.Value))
+		rows := valuesToStrings(result.Value)
+		require.Contains(t, rows[0], `"alice"`)
+
+		// aggregate().pretty()
+		result, err = gc.Execute(ctx, dbName, `db.users.aggregate([{$match: {name: "alice"}}]).pretty()`)
+		require.NoError(t, err)
+		require.Equal(t, 1, len(result.Value))
+	})
+}

compat_test.go

@@ -77,4 +77,5 @@ require (
 
 replace github.com/antlr4-go/antlr/v4 => github.com/bytebase/antlr/v4 v4.0.0-20240827034948-8c385f108920
 
+// TODO: Remove after parser is published with Int32 string arg support.
 replace github.com/bytebase/parser => ../parser

go.mod

@@ -204,3 +204,33 @@ func TestDecimal128Helper(t *testing.T) {
 		require.NotNil(t, price)
 	})
 }
+
+func TestInt32StringArg(t *testing.T) {
+	testutil.RunOnAllDBs(t, func(t *testing.T, db testutil.TestDB) {
+		dbName := "testdb_int32str_" + db.Name
+		defer testutil.CleanupDatabase(t, db.Client, dbName)
+
+		gc := gomongo.NewClient(db.Client)
+		ctx := context.Background()
+
+		// Int32("123") should parse the string as int32
+		_, err := gc.Execute(ctx, dbName, `db.test.insertOne({val: Int32("123")})`)
+		require.NoError(t, err)
+
+		result, err := gc.Execute(ctx, dbName, `db.test.findOne({})`)
+		require.NoError(t, err)
+		require.Equal(t, 1, len(result.Value))
+		doc, ok := result.Value[0].(bson.D)
+		require.True(t, ok)
+		val := getDocField(doc, "val")
+		require.Equal(t, int32(123), val)
+
+		// NumberInt("456") should also work
+		_, err = gc.Execute(ctx, dbName, `db.test.insertOne({val2: NumberInt("456")})`)
+		require.NoError(t, err)
+
+		// NumberLong("1774250313") — from real user report
+		_, err = gc.Execute(ctx, dbName, `db.test.insertOne({val3: NumberLong("1774250313")})`)
+		require.NoError(t, err)
+	})
+}