| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
|
Pushed a test-only follow-up for the coverage gate. The HTTP mismatch path was already covered through the live server test, but that path runs out of process, so I added an in-process regression test for the 400 response branch. Verified locally:
|
Sorry, something went wrong.
|
Rebased this onto current upstream/main and resolved the StreamableHTTP in-process transport test refactor conflict. The mismatch coverage is now in the current async in-process style, so the old socket/requests test shape was not brought back. Validated locally on Windows:
|
Sorry, something went wrong.
|
Follow-up after the Ubuntu matrix failure: the tests themselves passed, but strict-no-cover caught an old pragma: no cover on the request-header validation branch that is now covered by the updated in-process tests. I removed that stale pragma and revalidated locally on Windows:
|
Sorry, something went wrong.
|
Rebased onto current main. Two things changed underneath this PR, so the resolution is non-trivial:
tests/shared/test_streamable_http.py passes in full (71 passed), including both mismatch cases. |
Sorry, something went wrong.
|
Closing this one. The initialize / protocol-version handling has since been reworked upstream into the dual-era model (#2923 ignoring pre-2026 version pins, #3040 hardening the era-lock and its rejection path), which subsumes the narrow conflict rejection this PR added. Thanks for the direction on it, and happy to revisit if a gap turns up in the new model. |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
Summary
Fixes #2618
To verify