Add admin command tests for logRotate#642
Conversation
Signed-off-by: Victor [C] Tsang <vitsangp@amazon.com>
|
🤖 Auto-triaged by documentdb-triage-tool. Applied: Reasoningcomponent from path globs (test-coverage, test-framework); effort from diff stats (263+10 LOC, 10 files); LLM failed: Invalid response body while trying to fetch https://api.anthropic.com/v1/messages: Premature close If a label is wrong, remove it manually and ping |
| {"ok": 1.0}, | ||
| msg=f"comment should accept {bson_type.value}", | ||
| ) | ||
|
|
There was a problem hiding this comment.
Other admin command tests (killOp, getParameter) assert "comment": NotExists() in the response. Here only {"ok": 1.0} is checked. Add NotExists() to verify the comment is consumed and not echoed back
| result, UNRECOGNIZED_COMMAND_FIELD_ERROR, msg="Should reject unrecognized fields" | ||
| ) | ||
|
|
||
|
|
There was a problem hiding this comment.
Missing: does an unrecognized field still fail when a valid comment is also present?
This PR contains:
Background:
current timestamp with one-second resolution, e.g. my_server_log.2026-06-24T17-18-27. The rename target is fully determined by the wall-clock second.
Discussion Points:
Ref: