Feature: query-and-write writeConcern parameter#620
Open
imforster wants to merge 4 commits into
Open
Conversation
imforster
force-pushed
the
forstaia/writeConcern/query_and_write
branch
from
2cf971e to
8a923cc
Compare
Test cases: 344 (+ 6 replica_set skipped on standalone) Docs: https://www.mongodb.com/docs/v8.2/reference/write-concern/ Adds compatibility test coverage for the writeConcern cross-cutting parameter across write commands (update, delete, findAndModify) — from document type acceptance and sub-field validation (w, j, wtimeout) to full BSON type rejection matrix, value boundary cases, w:0 unacknowledged behavior, j:true override semantics, Decimal128 exponent-form coercions, case sensitivity, unknown field rejection, provenance sub-field acceptance, wtimeout edge cases, w:0 writeError suppression, fractional double coercion, and findAndModify return value independence from writeConcern. 350 test cases across 4 files following project test guidelines. insert already covered at commands/insert/test_insert_write_concern.py. bulkWrite excluded: admin-database command with different infrastructure. Signed-off-by: Ian Forster <forstaia@amazon.com>
imforster
force-pushed
the
forstaia/writeConcern/query_and_write
branch
from
8a923cc to
971c71d
Compare
- Move PROVENANCE_ACCEPTANCE_TESTS from field_validation to acceptance file - Remove unused assertNotError import from validation file - Clarify bulkWrite exclusion rationale in utils comment Signed-off-by: Ian Forster <forstaia@amazon.com>
imforster
added
compatibility test
…adConcern local availability
writeConcern field validation:
- Add INT64_MAX and INT64_MIN to w value rejection tests
- Add tagged object validation: reject empty {}, string tag values,
and nested object tag values
- Add wtimeout extended overflow rejection for double, Decimal128,
and +Infinity exceeding INT32_MAX
writeConcern acceptance:
- Add wtimeout boundary acceptance tests: INT32_MIN, -Infinity (double),
and -Infinity (Decimal128) are accepted (upper-bound only constraint)
readConcern:
- Add test_read_concern_local_availability.py covering 'local' level
availability without sessions, on aggregate/count/distinct, on empty
collections, and equivalence to omitting readConcern
Signed-off-by: Ian Forster <forstaia@amazon.com>
test_read_concern_local_is_default_equivalent had two assertNotError calls in one function, violating the one-assertion-per-test rule. Replaced with a single test_read_concern_local_does_not_error that asserts only the readConcern 'local' case. Signed-off-by: Ian Forster <forstaia@amazon.com>
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
1 participant
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.
#39
Test cases: 344 (+ 6 replica_set skipped on standalone)
Docs: https://www.mongodb.com/docs/v8.2/reference/write-concern/
Adds compatibility test coverage for the writeConcern cross-cutting parameter, validating its full surface area across write commands (update, delete, findAndModify) - from document type acceptance and sub-field validation (w, j, wtimeout) to full BSON type rejection matrix, value boundary cases, w:0 unacknowledged behavior, j:true override semantics, Decimal128 exponent-form coercions, case sensitivity, unknown field rejection, and findAndModify return value independence from writeConcern.
350 test cases across 5 files following project test guidelines.
Note: insert writeConcern already covered at
commands/insert/test_insert_write_concern.py.