Location: crates/mds-python/tests/test_errors.py:246
What: E11 uses try/except/else + pytest.fail rather than pytest.raises, which is less idiomatic in pytest.
Why deferred: Test style improvement, no functional change. Better addressed as part of broader test idiom standardization.
Source review: .devflow/docs/reviews/fix-esc-injection-176/2026-07-25_1625 (python-8)
Current pattern:
```python
try:
# code that should raise
except SomeException:
# handle expected case
else:
pytest.fail("Expected exception not raised")
```
Idiomatic pytest pattern:
```python
with pytest.raises(SomeException):
# code that should raise
```
Acceptance Criteria:
Note: This is a low-priority style cleanup; the test functions correctly as-is.
Location: crates/mds-python/tests/test_errors.py:246
What: E11 uses try/except/else + pytest.fail rather than pytest.raises, which is less idiomatic in pytest.
Why deferred: Test style improvement, no functional change. Better addressed as part of broader test idiom standardization.
Source review: .devflow/docs/reviews/fix-esc-injection-176/2026-07-25_1625 (python-8)
Current pattern:
```python
try:
# code that should raise
except SomeException:
# handle expected case
else:
pytest.fail("Expected exception not raised")
```
Idiomatic pytest pattern:
```python
with pytest.raises(SomeException):
# code that should raise
```
Acceptance Criteria:
Note: This is a low-priority style cleanup; the test functions correctly as-is.