Add setQuerySettings tests

[alinaliBQ]

This change adds tests for the setQuerySettings command operator. In order to define the SettingsTestCase class, I renamed query-planning to query_planning. Reason is that hyphens aren't allowed in identifiers, so directories that need to be imported must use underscores instead.

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

• Add compatibility test for setQuerySettings #409

Edit: replica_set is needed because query settings are not supported on standalone instances

[documentdb-triage-tool]

🤖 Auto-triaged by documentdb-triage-tool.

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

Reasoning

component from path globs (test-coverage, test-framework); effort from diff stats (3863+1 LOC, 31 files); LLM: Adds new test coverage for the setQuerySettings command operator, covering behavior, output, syntax, and error cases — a functional test expansion across multiple test scenarios.

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.

[alinaliBQ]

In order to define the SettingsTestCase class, I renamed query-planning to query_planning. Reason is that hyphens aren't allowed in identifiers, so directories that need to be imported must use underscores instead.

[karthikvarma22]

why no_parallel in this CR ?

[alinaliBQ]

no_parallel is only used in the smoke test. Perhaps you meant replica_set? In that case, replica set is required to enable query settings.

[karthikvarma22]

Ack

[karthikvarma22]

Looks like this skeleton is present in most of files. Do we need a shared runner ?

[alinaliBQ]

Could you clarify on what you mean by shared runner? Currently CI is set up to use a shared mongodb instance.
According to the style guide, it says the tests should be as standalone as possible and that helper functions that hide the test logic need to be avoided, so we needed to have separate def test_* functions.

[karthikvarma22]

Got it.