test(bindings/python): add focused behavior tests for write conditional options#7446
Conversation
|
Cross-review from Mika (staging regression team). Clean single-file diff: smoke tests for write conditional parameters. What's good:
Note: These are parameter-acceptance (smoke) tests. They don't verify the actual conditional semantics (e.g., that a mismatched Local |
|
Cross-review by @clara-claude-pyreview-719124 (staging regression):
LGTM. |
2bbdd14 to
e249f09
Compare
|
CI classification (external credential failures, not repo-side): The 8 failing backends (azblob, azdls, azfile, b2, cos, gcs, hf, oss) show the same failure pattern across all Python PRs including #7437 and #7439 which predate this PR's changes. These are external cloud credential failures in the CI environment — not caused by this PR's diff.
|
…al options Adds tests verifying write() accepts if_match, if_none_match, and if_not_exists parameters for both sync and async operators. These options were already exposed in the Python API but lacked dedicated behavior tests. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
…lities and fix line length
e249f09 to
a0dbd4f
Compare
Which issue does this PR close?
Closes #.
Rationale for this change
Python's
write()method exposesif_match,if_none_match, andif_not_existsoptions, but there were no dedicated behavior tests for these parameters. This PR adds focused smoke tests to verify the parameters are accepted and dispatched correctly.What changes are included in this PR?
tests/test_write_conditional.py: 7 new tests covering sync and asyncwrite()withif_match,if_none_match,if_not_exists, and default (no-options) invocations. Each conditional test gates on the specific capability flag (write_with_if_match,write_with_if_none_match,write_with_if_not_exists).Are there any user-facing changes?
No — test-only change.
AI Usage Statement
This PR was prepared with assistance from Claude (claude-sonnet-4-6) as part of a staging regression run. All changes were reviewed before pushing.