gfx-rs/wgpu

Enable `clippy::tests_outside_test_module` lint

Open

#6.106 aberto em 12 de ago. de 2024

Ver no GitHub
 (2 comments) (0 reactions) (0 assignees)Rust (703 forks)batch import
area: infrastructuregood first issuetype: enhancement

Métricas do repositório

Stars
 (9.290 stars)
Métricas de merge de PR
 (Mesclagem média 10d 7h) (61 fundiu PRs em 30d)

Description

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

Guia do colaborador

Direção de pesquisa
Habilite o lint `clippy::tests outside test module` na configuração do clippy do workspace (provavelmente em `Cargo.toml` ou `.cargo/config.toml`). Em seguida, execute `cargo clippy all targets` para identificar todas as funções de teste atualmente fora de um módulo `#[cfg(test)]`. Mova cada teste identificado para um módulo de teste apropriado dentro de seu arquivo. Certifique se de que os testes ainda passem com `cargo test`. Nenhum PR vinculado ou comentário do mantenedor indica bloqueadores. A issue está aberta sem responsáveis atribuídos, portanto, um novato pode reivindicá la.
Pilha de tecnologia
rust
Domain
testingdeveloper experience
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-2 dias
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Rust basicscargo clippy familiarity
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
50