Updates TEST_COVERAGE.md with rules from $push#262
Draft
alinaliBQ wants to merge 7 commits into
Draft
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>
|
🤖 Auto-triaged by documentdb-triage-tool. Applied: Reasoningcomponent from path globs (docs); effort from diff stats (13+0 LOC, 1 files); LLM: Updates TEST_COVERAGE.md with new rules and guidance discovered during $push accumulator test work — purely a documentation improvement with no code changes. If a label is wrong, remove it manually and ping |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Updates TEST_COVERAGE.md with additional coverage rules discovered while writing$push accumulator tests (#257).
Section 1 – Data Type Coverage:
• Added array traversal via field paths (e.g. $a.b on arrays-of-objects) to
the nested structure coverage guidance
Section 2 – Arithmetic Operator Coverage:
• Added rule for testing special float values (NaN, Infinity, -Infinity, -0.0)
mixed with non-numeric types (string, boolean, null, object, array) in
operators that collect or pass through values
Section 15 – Pipeline Stage Coverage:
• Added rule for testing multiple $group stages in a single pipeline, where
each stage performs actual grouping (not just _id: null) and the second re-
aggregates the output of the first
Section 18 – Accumulator Coverage:
• Added Actual Grouping Requirement: accumulator tests must include at least
one multi-group test per test category, not rely exclusively on _id: null
• Added Multiple Accumulators in Single $group: test multiple instances of the
same accumulator in one $group with actual grouping to verify independent
field collection without cross-contamination
• Added Large-Scale Result Verification: prefer verifying actual content over
count-only checks at scale; use loops for expected results and fall back to
server-side content checks ($sum/$min/$max) when full element-by-element
comparison is impractical