Deprecate should_not raise_error in MSpec and replaces usages in ruby/spec
Personne n'a encore pris cette issue.
Évaluation
- Difficulté
- 5/5
- Temps estimé
- Plus d'une semaine
- Accessibilité débutants
- 35/100
- Type d'issue
- Refactorisation
- Clarté
- Plutôt claire
- Activité
- À l'abandon
- Stack technique
- ruby
- Domaine
- testing-qa
Piste de recherche
Examinez la discussion de la pull request ruby/spec référencée et recherchez dans le dépôt les quelque 176 utilisations de should_not raise_error. Séparez les vérifications légitimes d’exceptions des cas qui vérifient uniquement qu’aucune exception n’est levée, puis remplacez ou supprimez ces derniers au profit d’attentes utiles. La tâche est terminée lorsque les utilisations inutiles ont été traitées et que le comportement de MSpec est déprécié de manière cohérente.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
See https://github.com/ruby/spec/pull/753/files#r376792963
should_not raise_error is the same as testing nothing.
Each example already implicitly test no exception is raised.
There might be a few legit cases, but I would expect the vast majority could be removed and replaced with useful expectations actually testing it works as expected.
Currently there are about 176 should_not raise_error in ruby/spec.
- Langage dominant
- Ruby
- Étoiles
- 623
- Forks
- 400
- Merge moyen
- 11 h 11 min
- PR mergées (30 j)
- 3
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de ruby/spec
-
Difficulté 3/5 1-2 jours Accessibilité débutants 52/100
-
Improving Marshal specsOuverte
Difficulté 4/5 3-5 jours Accessibilité débutants 38/100
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
-
Replace most matchers by `obj.should.foo`Peut-être à nouveau libre @eregon l’a pris il y a 156 jours, et aucune pull request n’est ouverte. Ouverte
ruby/spec#1350 · 3 commentaires · 3 réactions · 1 personne assignée ·
-
Difficulté 3/5 1-2 jours Accessibilité débutants 38/100
Toutes les issues de ruby/spec
Issues similaires
-
ds-drift
Difficulté 1/5 Moins d'une heure Accessibilité débutants 76/100
we-promise/sure#4120 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 72/100
Les mainteneurs répondent en général sous 4 jours
-
Difficulté 2/5 1-3 heures Accessibilité débutants 84/100
scanaislop/aislop#476 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 Moins d'une heure Accessibilité débutants 82/100
avo-hq/avo#4878 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
Dependencies view: `getParent` loops forever on untitled documents, extension host runs out of memoryPeut-être pris Une pull request liée à cette issue est ouverte ou déjà fusionnée. Ouvertebug
Difficulté 2/5 1-3 heures Accessibilité débutants 70/100
Les mainteneurs répondent en général sous 1 jour