Add $commitTransaction command tests and enable replica set in CI

[alinaliBQ]

This change adds tests for the $commitTransaction command operator.

Add command operator tests for $commitTransaction. Tests database $commitTransaction behavior, output collection, syntax, and expected errors.

• Add compatibility test for commitTransaction #411

This PR adds workflow to run replica_set tests against an instance with replica set enabled.

[documentdb-triage-tool]

🤖 Auto-triaged by documentdb-triage-tool.

Applied: compatibility test, enhancement
Project fields suggested: Component test-coverage · Priority P2 · Effort L · Status In Progress
Confidence: 0.72 (mixed)

Reasoning

component from path globs (test-coverage, test-framework, ci); effort from diff stats (1591+13 LOC, 15 files); LLM: Adds new $commitTransaction command test coverage and enables replica set in CI, touching both test files and CI workflow configuration.

If a label is wrong, remove it manually and ping @patty-chow so the rules can be tuned. The bot will not re-label items that already have component labels.

[Rishabh998]

I think this value is wrong. MongoDB's NoSuchTransaction is error code 251, not 125.

https://www.mongodb.com/docs/manual/reference/error-codes/

[alinaliBQ]

Yes you are right and I have fixed the naming. The 125 value should actually be named COMMAND_FAILED_ERROR, not NO_SUCH_TRANSACTION_ERROR.

[Rishabh998]

When a test supplies its own commit_command, we commit via the raw admin command but pymongo's session doesn't know we did, so on with block exit it and auto-abort the "still open" transaction.

So for commit_response_ok: insert {_id: 1} → {commitTransaction: 1} returns ok:1 → block exits → pymongo aborts the already-committed txn. The test only checks ok:1, so it'd stay green even if the data got rolled back. Since all the writeConcern / comment / field_types success cases use this path, can you confirm the data actually lands? Reading the doc back after commit (e.g. find_one({"_id": 1})) and asserting it exists would prove it.

[alinaliBQ]

added check len(docs) > 0 to verify doc returned persisted

[alinaliBQ]

@Rishabh998 Addressed the comments, please have a look

[alinaliBQ]

Yes you are right and I have fixed the naming. The 125 value should actually be named COMMAND_FAILED_ERROR, not NO_SUCH_TRANSACTION_ERROR.

[alinaliBQ]

added check len(docs) > 0 to verify doc returned persisted