Fix compilation errors in routes tests#98
Conversation
Reviewer's GuideFixes compilation errors in route tests by importing LengthPrefixedProcessor and removing the unused default_processor helper, eliminating related warnings. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing Touches🧪 Generate Unit Tests
🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
2054e8e
into
codex/extend-run_app_with_frame-for-multiple-frames
* Add multi-frame support for test helper * Added run_app_with_frames utilities to drive streaming tests.* Updated existing routes test and added new sequential frames test.* Removed Logger middleware from echo example for clarity. * Import LengthPrefixedProcessor in tests (#98)
Summary
LengthPrefixedProcessorin route testsTesting
make fmtmake lintmake testhttps://chatgpt.com/codex/tasks/task_e_6855e1dc01588322a2fc064c78702e1c
Summary by Sourcery
Fix compilation errors in route tests by importing the missing LengthPrefixedProcessor and removing the unused default_processor helper.
Bug Fixes: