Add serverStatus tests#623
Conversation
|
🤖 Auto-triaged by documentdb-triage-tool. Applied: Reasoningcomponent from path globs (test-coverage); effort from diff stats (1029+0 LOC, 6 files); LLM: Adds new test coverage for the If a label is wrong, remove it manually and ping |
eerxuan
left a comment
There was a problem hiding this comment.
[Major] wiredTiger assertions violate the "no engine-specific implementation details" rule. TEST_COVERAGE.md:65 is explicit: "Don't test internal storage details (wiredTiger, engine-specific fields, etc.). These belong in engine-specific test suites, not compatibility tests." Three cases assert the wiredTiger section directly:
- wiredTiger_is_object (response_structure) — asserts wiredTiger exists as an object
- exclude_wiredTiger (field_toggle) — asserts it can be excluded
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
|
@eerxuan |
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com> Signed-off-by: RyanGarfinkel <113050972+RyanGarfinkel@users.noreply.github.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com> Signed-off-by: Sarthak Dalmia <sadalmia@microsoft.com>
Add command operator tests for serverStatus. Tests database serverStatus behavior, output collection, and syntax. There is no specific error code from serverStatus, since serverStatus is permissive to its inputs.