refactor(config): move inline tests to module#3345
Conversation
Signed-off-by: cyq <15000851237@163.com>
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
There was a problem hiding this comment.
Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.
|
Thanks @cyq1017 — this config test extraction was useful for #3307. The PR branch was stale/conflicting against the current v0.8.64 integration branch, so I reapplied the same extraction to the current test body rather than cherry-picking outdated content. I preserved you as the commit author with the canonical noreply identity from I am leaving this PR open until the integration branch is pushed/landed so the visible GitHub state stays honest. |
Closes #3307
Problem
crates/config/src/lib.rscarries a large inline test module, which increases the production file size and future conflict surface.Change
Move the config tests into
crates/config/src/tests.rsand keep the production module wired through#[cfg(test)] mod tests;. No assertions or production logic were changed.Verification
cargo fmt --checkcargo test -p codewhale-config