Wave B-1 PR-B: missing-dep test coverage

[AndresL230]

Summary

Closes #9 (the remaining two missing-dep tests not covered by PR-A #33).

• test_local_transport_no_op_without_websockets — verifies _LocalTransport constructs, sends, and disposes gracefully when the optional websockets dependency is absent.
• test_flask_extension_raises_clear_error_without_flask — verifies the stub RecostExtension raises ImportError with the pip install recost[flask] hint.

Pattern

Both tests use monkeypatch.setitem(sys.modules, "<name>", None) plus importlib.reload(...) to fake the missing import, then monkeypatch.undo() + reload-back in finally to restore the real module so subsequent tests in the run aren't poisoned. Isolated as PR-B because the test shape differs meaningfully from everything in PR-A (#33).

Test plan

• CI green on Python 3.9, 3.10, 3.11, 3.12.
• pytest tests/test_transport.py -q clean (verifies no poisoning of TestLocalTransportSync / TestLocalTransportAsync / TestDisposeDuringConnect).
• pytest tests/test_flask.py -q clean (verifies no poisoning of TestRecostExtension).

🤖 Generated with Claude Code

Summary by CodeRabbit

• Tests
  • Added test coverage ensuring clear error messages with installation instructions when optional Flask dependency is unavailable.
  • Added test coverage validating graceful degradation when optional websockets dependency is missing.

[coderabbitai]

Caution

Review failed

Pull request was closed or merged during review

📝 Walkthrough

Walkthrough

Two tests address missing optional dependencies: Flask extension raises an ImportError with an install hint when Flask is unavailable, and local transport disables websockets gracefully with no-op send() and dispose() calls when the websockets package is missing.

Changes

Optional dependency graceful degradation tests

Layer / File(s)	Summary
Graceful degradation tests for missing optional dependencies tests/test_flask.py, tests/test_transport.py	test_flask_extension_raises_clear_error_without_flask verifies RecostExtension raises ImportError with pip install recost[flask] hint when Flask is unavailable. test_local_transport_no_op_without_websockets simulates missing websockets, asserts _LocalTransport sets _has_websockets=False, and confirms send() and dispose() complete without blocking.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 Two tests, one pattern clear,
When deps depart, we shed no tear—
Flask hints kindly, websockets rest,
Graceful fallbacks pass the test! 🌱

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 75.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'Wave B-1 PR-B: missing-dep test coverage' clearly indicates the PR adds test coverage for missing optional dependencies, which matches the changes shown in the summary (Flask and websockets dependency tests).
Linked Issues check	✅ Passed	The PR successfully implements two of the missing test gaps from issue #9: _LocalTransport graceful no-op when websockets missing and Flask graceful degradation when flask missing.
Out of Scope Changes check	✅ Passed	All changes are limited to test code in tests/test_flask.py and tests/test_transport.py, directly addressing the specific missing-dependency test coverage objectives from issue #9.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch wave-b1/pr-b-missing-dep-tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}