Added core indexes property tests for sparse#185
Conversation
|
🤖 Auto-triaged by documentdb-triage-tool. Applied: Reasoningcomponent from path globs (test-coverage); effort from diff stats (991+0 LOC, 8 files); LLM: Adds new property-based test coverage for sparse indexes, expanding the compatibility test suite as part of a tracked issue. If a label is wrong, remove it manually and ping |
ba895ed to
205de84
Compare
Signed-off-by: Victor Tsang <vitsangp@amazon.com>
f21a33f to
5dff8f6
Compare
|
fixed conflict and rebase |
| assertSuccess(result, test.expected, raw_res=True) | ||
|
|
||
|
|
||
| def test_sparse_text_skips_missing_text(collection): |
There was a problem hiding this comment.
🟠 [Required] test_sparse_text_skips_missing_text and test_sparse_exists_false_not_affected both belong outside properties/sparse/
def test_sparse_text_skips_missing_text(collection):
"""Test text index is always sparse — skips docs without text field."""
...indexes=[{"key": {"content": "text"}, "name": "idx_text"}],
This test creates a text index without sparse: True and asserts text-index-implicit-sparse behavior. The "text index is always sparse" property belongs in a text-index test folder (or
properties/sparse/test_implicit_sparse.py), not in this folder which is about explicit sparse option.
Same for test_sparse_exists_false_not_affected — the property tested is "$exists:false works correctly" which is a $exists query operator concern, not sparse-specific.
These two would live better in:
- text_search/ for the text-implicit-sparse test
- query/misc/exists/ for the $exists:false test
This PR contains:
Ref: