Deprecate should_not raise_error in MSpec and replaces usages in ruby/spec
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Refactorización
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- ruby
- Área
- testing-qa
Línea de trabajo
Revisa la discusión del pull request de ruby/spec referenciado y busca en el repositorio los aproximadamente 176 usos de should_not raise_error. Separa las comprobaciones legítimas de excepciones de los casos que solo verifican que no se produzca ninguna excepción, y reemplaza o elimina estos últimos por expectativas útiles. Se considera terminado cuando se hayan abordado los usos innecesarios y el comportamiento de MSpec se haya marcado como obsoleto de forma coherente.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
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.
- Lenguaje dominante
- Ruby
- Estrellas
- 622
- Forks
- 402
- Merge medio
- 14 h 51 min
- PR fusionados (30 d)
- 6
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de ruby/spec
-
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
-
Improving Marshal specs Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 38/100
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 35/100
-
ruby/spec#1350 · 3 comentarios · 3 reacciones · 1 asignado ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
palladius/rails8-app-on-gcp#145 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100
rubocop/rubocop-rspec#2236 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
riscv/riscv-unified-db#2624 · 1 reacción ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100