assertj / assertj/assertj-swing
Improve GuiActionRunner
- Lingua principale
- Java
- Stelle
- 121
- Fork
- 52
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Descrizione
At the end of GuiActionRunner there is the following code:
```java
if (caughtException instanceof RuntimeException) {
appendStackTraceInCurrentThreadToThrowable(caughtException, "execute");
throw (RuntimeException) caughtException;
}
if (caughtException instanceof Error) {
caughtException.fillInStackTrace();
throw (Error) caughtException;
}
```
I think it is not reasonable to fill in the stack trace for the Errors, throwing away the original stack trace: Errors should be treated the same as RuntimeExceptions. Or - if there is some reason for doing so - then at least don't throw away the stack trace of AssertionErrors, they are normal and expected during testing.
Guida per i contributori
Apri la guida per i contributori
Direzione di ricerca
Individuate GuiActionRunner ed esaminate la gestione, alla fine del metodo, di RuntimeException e Error. Esaminate i test esistenti, quindi concentratevi su AssertionError e sulla preservazione dello stack trace originale; il lavoro è completato quando il comportamento selezionato è coperto e la suite di test passa.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- java
- Ambito
- testing
- Tipo di issue
- Bug
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Stato di attività
- Ferma
- Chiarezza
- Abbastanza chiara
- Idoneità per principianti
- 35/100