Add serverStatus tests#623
Conversation
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
documentdb-triage-tool
Bot
added
compatibility test
|
🤖 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 |
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
| def test_serverStatus_uptimeMillis_consistent_with_uptime(collection): | ||
| """Verify serverStatus uptime is consistent with uptimeMillis.""" | ||
| result = execute_admin_command(collection, {"serverStatus": 1}) | ||
| uptime_from_millis = result["uptimeMillis"] // 1000 |
There was a problem hiding this comment.
This could get flaky,
would assert(uptime: Gte(uptime_from_millis - 1)) make more sense ?
There was a problem hiding this comment.
got it, changed to use Gte(uptime_from_millis - 1)
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
2 participants
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.