Fix test_websocket_endpoint_added for newer FastAPI

[tschm]

Summary

CI test test_websocket_endpoint_added failed with AttributeError: '_IncludedRouter' object has no attribute 'path'.

Newer FastAPI versions append an _IncludedRouter marker object to app.routes when include_router is called. That object has no path attribute, so the route lookup in the test crashed.

Fix

Guard the attribute access with getattr(route, "path", None), skipping marker objects while still finding the /ws/heartbeat route. This is a test-only change; no production code affected.

🤖 Generated with Claude Code

[Copilot]

Pull request overview

Updates a lifecycle test to be compatible with newer FastAPI behavior where include_router appends an _IncludedRouter marker object (without a path attribute) into app.routes, preventing an AttributeError during route inspection.

Changes:

• Avoids direct route.path access in test_websocket_endpoint_added by using getattr(route, "path", None) when collecting route paths.
• Adds an explanatory comment documenting why the guard is needed for newer FastAPI versions.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.