Add explain diagnostic tests#631
Conversation
…and duplicates, edited docStrings Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
|
🤖 Auto-triaged by documentdb-triage-tool. Applied: Reasoningcomponent from path globs (test-coverage); effort from diff stats (1061+0 LOC, 8 files); LLM: Adds new test coverage for the explain diagnostic command using CommandTestCase, touching the compatibility tests component within a single area. If a label is wrong, remove it manually and ping |
…case return Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
eerxuan
left a comment
There was a problem hiding this comment.
Please also verify queryPlanner.planCacheShapeHash and queryShapeHash exist.
We need a hash equivalence test for both planCacheShapeHash and queryShapeHash.
#438 (comment)
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Added tests for both types of ShapeHash, covering cases in issue #438 and some further gaps. Please review the new file and let me know if I have missed anything. |
This PR adds the tests for the explain diagnostic command. Explain is an interesting case, as it has both more internal complexity and crosses more features than many of the other diagnostic commands. It also requires a collection as part of its own function call. Because of this, explain tests primarily rely on CommandTestCase rather than DiagnosticTestCase in order to take advantage of runtime collection resolution.
Ref: Issue #43, #438