Update unit test to avoid deprecated `ExpectedException.none()` by using `Assert.assertThrows()`
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
Línea de trabajo
Busca en el repositorio ExpectedException.none(), expectIllegalStateIsNotShowingComponent y expectIllegalStateIsDisabledComponent para identificar todas las pruebas unitarias y helpers afectados. Ejecuta la suite de pruebas unitarias relevante antes y después de las sustituciones; se considera terminado cuando se hayan eliminado los usos obsoletos y las pruebas conserven sus aserciones existentes sobre las excepciones y los mensajes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
ExpectedException.none() is marked as deprecated but there are so many unit tests which depends on the function.
These cases can be rewritten by using Assert.assertThrows(Class<?> class, ThrowableRunnable run)
There are also Similar deprecation; expectIllegalStateIsNotShowingComponent and expectIllegalStateIsDisabledComponent
These can be implemented as like as followings;
public static AbstractStringAssert<?> assertThatIllegalStateExceptionCauseIsDisabledComponent(ThrowingRunnable r) {
Throwable t = Assert.assertThrows(IllegalStateException.class, r);
return assertThat(t.getMessage()).contains("Expecting component").contains("to be enabled");
}
public static AbstractStringAssert<?> assertThatIllegalStateExceptionCauseIsNotShowingComponent(ThrowingRunnable r) {
Throwable t = Assert.assertThrows(IllegalStateException.class, r);
return assertThat(t.getMessage()).contains("Expecting component").contains("to be shown on the screen");
}
public static AbstractStringAssert<?> assertThatIllegalStateExceptionCauseIsNotResizableComponent(ThrowingRunnable runnable) {
Throwable t = Assert.assertThrows(IllegalStateException.class, runnable);
return assertThat(t.getMessage()).contains("Expecting component").contains("to be resizable by the user");
}
- Lenguaje dominante
- Java
- Estrellas
- 121
- Forks
- 52
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Preparar el entorno
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 assertj/assertj-swing
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
assertj/assertj-swing#279 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 48/100
assertj/assertj-swing#278 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 30/100
assertj/assertj-swing#276 · 1 comentario ·
-
how to test when headless?Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 35/100
assertj/assertj-swing#277 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
assertj/assertj-swing#274 · 2 comentarios ·
Todos los issues de assertj/assertj-swing
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
OpenAPITools/openapi-generator#25014 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 92/100
AloisSeckar/demos-java#380 ·
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
openhab/openhab-core#5847 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
appsmithorg/appsmith#42297 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 75/100