gfx-rs/wgpu

Enable `clippy::tests_outside_test_module` lint

Open

#6,106 建立於 2024年8月12日

在 GitHub 查看
 (2 留言) (0 反應) (0 負責人)Rust (9,290 star) (703 fork)batch import
area: infrastructuregood first issuetype: enhancement

描述

There's a whole bunch of tests that aren't in test modules. This should be solved once and for all using this clippy lint https://rust-lang.github.io/rust-clippy/master/index.html#/tests_outside_test_module

Not doing so means that the tests will be unnecessarily compiled as part of non test builds

貢獻者指南

技術棧
rust
領域
testingdeveloper experience
議題類型
refactor
難度面向新貢獻者的預計實作難度，1 表示很小改動，5 表示專家級工作。
2
預計時間有經驗貢獻者完成調查、實作、測試並準備 pull request 的粗略時間範圍。
1-2 days
活動狀態議題目前的可參與程度：新鮮、活躍、陳舊、阻塞或等待維護者輸入。
fresh
清晰度議題是否清楚說明預期改動、驗收標準和下一步。
clear
前置要求
Rust basicscargo clippy familiarity
新手友善度1-100 的估計分數，表示該議題對首次貢獻者的友善程度。
50
研究方向
Enable the `clippy::tests outside test module` lint in the workspace's clippy configuration (likely in `Cargo.toml` or `.cargo/config.toml`). Then run `cargo clippy all targets` to identify all test functions currently outside a `#[cfg(test)]` module. Move each identified test into an appropriate test module within its file. Ensure tests still pass with `cargo test`. No linked PRs or maintainer comments indicate blockers. The issue is open with no assignees, so a newcomer can claim it.