Fix Issue #202: Add $bitAnd operator expression compatibility tests by krishnasai453 · Pull Request #662 · documentdb/functional-tests · GitHub
Skip to content

Fix Issue #202: Add $bitAnd operator expression compatibility tests - #662

Open
krishnasai453 wants to merge 51 commits into
documentdb:mainfrom
krishnasai453:fix-issue-202-add_bitAnd_compatibility_test
Open

krishnasai453 wants to merge 51 commits into
documentdb:mainfrom
krishnasai453:fix-issue-202-add_bitAnd_compatibility_test

Conversation

@krishnasai453

Copy link
Copy Markdown

Description

This PR resolves failing compatibility tests for the $bitAnd operator in test_expression_bitAnd_additional.py.

Why this is needed

Under MongoDB and DocumentDB specifications, the $bitAnd aggregation operator only supports int and long operands. Passing an array expression (such as [["$a", "$b"], 3]) or an object expression (such as [{"x": "$a", "y": "$b"}, 0]) evaluates to array/object structures, which are non-numeric types. These operations should fail with a type mismatch rather than succeed.

What changed

  • Updated ARRAY_EXPRESSION_TESTS and OBJECT_EXPRESSION_TESTS in test_expression_bitAnd_additional.py to expect TYPE_MISMATCH_ERROR (code 14) instead of assuming successful evaluation.
  • Forwarded the error_code parameter to assert_expression_result in test_bitAnd_expression_additional.
  • Removed the untracked template file test_expression_bitAnd_testing_strategy.txt.

How to test

Run the bitwise operator tests locally:

pytest documentdb_tests/compatibility/tests/core/operator/expressions/bitwise/bitAnd/test_expression_bitAnd.py 
pytest documentdb_tests/compatibility/tests/core/operator/expressions/bitwise/bitAnd/test_expression_bitAnd_additional.py

@krishnasai453
krishnasai453 requested a review from a team as a code owner July 1, 2026 03:46
@krishnasai453

Copy link
Copy Markdown
Author

@documentdb-triage-tool documentdb-triage-tool Bot added bug Something isn't working compatibility test Compatibility test related labels Jul 1, 2026
@documentdb-triage-tool

Copy link
Copy Markdown

krishnasai453 and others added 25 commits July 8, 2026 01:24
Signed-off-by: krishnasai453 <krishnasai453@gmail.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.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: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: krishnasai453 <krishnasai453@gmail.com>
Signed-off-by: krishnasai453 <krishnasai453@gmail.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Victor Tsang <vitsangp@amazon.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Ian Forster <forstaia@amazon.com>
Signed-off-by: Victor Tsang <vitsangp@amazon.com>
Co-authored-by: Victor Tsang <vitsangp@amazon.com>
Signed-off-by: Victor Tsang <vitsangp@amazon.com>
Signed-off-by: Ian Forster <forstaia@amazon.com>
Signed-off-by: Victor Tsang <vitsangp@amazon.com>
Co-authored-by: Victor Tsang <vitsangp@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
vic-tsang and others added 25 commits July 8, 2026 01:24
Signed-off-by: Victor [C] Tsang <vitsangp@amazon.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Victor [C] Tsang <vitsangp@amazon.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Victor [C] Tsang <vitsangp@amazon.com>
\Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: Victor [C] Tsang <vitsangp@amazon.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Victor [C] Tsang <vitsangp@amazon.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Alina (Xi) Li <Alina.Li@improving.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: PatersonProjects <keldonhoff@gmail.com>
@krishnasai453
krishnasai453 force-pushed the fix-issue-202-add_bitAnd_compatibility_test branch from 795d3c0 to 740bc4e Compare July 8, 2026 05:25
Signed-off-by: Daniel Frankcom <frankcom@amazon.com>
Signed-off-by: krishnasai453 <krishnasai453@gmail.com>
@krishnasai453
krishnasai453 force-pushed the fix-issue-202-add_bitAnd_compatibility_test branch from 740bc4e to e48274a Compare August 5, 2026 23:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working compatibility test Compatibility test related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants