Add distinct aggregation command tests#215
Conversation
Signed-off-by: Daniel Frankcom <[email protected]>
danielfrankcom
force-pushed
the
pr/distinct
branch
from
6e2208e to
c77d155
Compare
documentdb-triage-tool
Bot
added
compatibility test
|
🤖 Auto-triaged by documentdb-triage-tool. Applied: Reasoningcomponent from path globs (test-coverage, test-framework); effort from diff stats (4333+37 LOC, 16 files); LLM: Adds new test coverage for the If a label is wrong, remove it manually and ping |
Signed-off-by: Daniel Frankcom <[email protected]>
Signed-off-by: Daniel Frankcom <[email protected]>
|
Added a few extra tests for views which fell out of working on the dedicated view collection tests. |
eerxuan
left a comment
eerxuan
left a comment
There was a problem hiding this comment.
Other than view and collation trim as discussed before, LGTM.
Remove collation semantic/subfield tests (covered in collation/), namespace naming rules and UUID resolution (common spec), and view tests (merged into collection type tests). Add collection type acceptance tests. Signed-off-by: Daniel Frankcom <[email protected]>
Signed-off-by: Daniel Frankcom <[email protected]>
2 participants
This change adds tests for the
distinctaggregation command.There is some overlap here with #191 as some of the framework changes are needed by both. I've included them here so they should merge cleanly.