rust-lang/rust-clippy

lint: assertion-less #[test] function

Open

#626 ouverte le 6 févr. 2016

Voir sur GitHub
 (2 commentaires) (0 réactions) (1 assigné)Rust (1 391 forks)batch import
A-lintL-testsT-middlegood first issue

Métriques du dépôt

Stars
 (10 406 stars)
Métriques de merge PR
 (Merge moyen 19j 22h) (113 PRs mergées en 30 j)

Description

allow, because assert!()s may be in the tested code

Guide contributeur

Direction de recherche
Cette issue demande un nouveau lint pour détecter les fonctions de test (annotées avec #[test]) qui ne contiennent aucune assertion. La description actuelle est brève et mentionne 'allow', ce qui pourrait indiquer que le lint devrait être autorisé par défaut. Tout d'abord, vérifiez si l'attribution à royrustdev est toujours active en consultant les commentaires récents. Si l'issue n'est pas bloquée, étudiez les lints clippy existants pour les fonctions de test (par exemple dans clippy lints/src/tests.rs) pour comprendre le modèle. Ensuite, proposez une conception et discutez en avec les mainteneurs.
Stack technique
rust
Domaine
tooling
Type d'issue
Fonctionnalité
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
RustClippy
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50