gfx-rs/wgpu

Enable `clippy::tests_outside_test_module` lint

Open

#6,106 opened on 2024年8月12日

GitHub で見る
 (2 comments) (0 reactions) (0 assignees)Rust (703 forks)batch import
area: infrastructuregood first issuetype: enhancement

Repository metrics

Stars
 (9,290 stars)
PR merge metrics
 (平均マージ 10d 7h) (30d で 61 merged PRs)

説明

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

コントリビューターガイド

調査方針
ワークスペースの clippy 設定（おそらく `Cargo.toml` または `.cargo/config.toml` 内）で `clippy::tests outside test module` lint を有効にします。次に `cargo clippy all targets` を実行して、現在 `#[cfg(test)]` モジュール外にあるすべてのテスト関数を特定します。特定された各テストを、そのファイル内の適切なテストモジュールに移動します。`cargo test` でテストが引き続きパスすることを確認します。関連する PR やメンテナのコメントにブロッカーは示されていません。この issue はオープンで担当者もいないため、新規参加者が引き受けることができます。
技術スタック
rust
領域
testingdeveloper experience
Issue 種別
リファクタリング
難度新しい貢献者にとっての実装難度です。1 は非常に小さな変更、5 は専門的な作業です。
2
推定時間経験ある貢献者が調査、実装、テスト、pull request 準備にかけるおおよその時間範囲です。
1-2日
活動状況Issue が今どれだけ取り組みやすいかを示します。新しい、活発、古い、ブロック中、またはメンテナー入力待ちなどです。
新着
明確さ期待される変更、受け入れ条件、次の手順がどれだけ明確かを示します。
明確
前提条件
Rust basicscargo clippy familiarity
初心者向け度初回貢献者にどれだけ取り組みやすいかを 1-100 で推定したスコアです。
50