Replace test channel dead code allow (#61)#205
Conversation
Use a documented `dead_code` expectation for the integration-test scaffold so the compiler reports when the expectation is obsolete.
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
Reviewer's guide (collapsed on small PRs)Reviewer's GuideReplaces the file-level allow(dead_code) attribute in the test channel support module with a documented expect(dead_code) so the compiler will surface when the dead-code allowance is no longer needed, keeping the test scaffold honest over time. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
Summary
This branch replaces the file-level
allow(dead_code)in tests/support/test_channel.rs with a documentedexpect(dead_code)so the compiler can report when the test scaffold expectation is no longer needed.Closes #61.
Review walkthrough
risk: low.Validation
cargo check --all --benches --tests --examples: passed with no warnings.coderabbit review --agent: passed withfindings: 0before commit.make all: passed, including formatting, clippy, nextest, and GitHub tool tests.coderabbit review --agent: passed withfindings: 0after commit.Summary by Sourcery
Enhancements: