Should assertions for properties of the superclass(es) be generated?
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 5/5
- Tempo stimato
- Più di una settimana
- Idoneità per principianti
- 25/100
Direzione di ricerca
Inizia leggendo la configurazione del generatore in pom.xml e la documentazione referenziata nell’issue, quindi esamina come vengono incluse le assertions delle superclassi. Segui la gerarchia di esempio da SomeErrorException, passando per HttpResponseException, fino a Throwable e verifica la limitazione esistente di Assertions.assertThat(Throwable). Il lavoro è completato quando sarà stato deciso e documentato un approccio per generare assertions delle proprietà ereditate senza entrare in conflitto con le assertions esistenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Consider this class:
import org.apache.http.client.HttpResponseException;
public class SomeErrorException extends HttpResponseException {
private SomeError error;
public SomeErrorException(int statusCode, String reason, SomeError error) {
super(statusCode, reason);
this.error = error;
}
public SomeError getError() {
return error;
}
}
It would be nice to be able to test SomeErrorException objects like this:
assertThat(exception).hasStatusCode(500)
.hasMessage("Internal Server Error")
.hasError(error)
However, hasStatusCode and hasMessage are not generated because they are defined in superclasses. If I include HttpResponseException in the generator configuration (in pom.xml), the hasStatusCode appear, but of course hasMessage is only present if I also include the whole hierarchy (four classes!) up to Throwable. Also, org.assertj.core.api.Assertions.assertThat(Throwable) already exists, so I cannot make my assertions a subclass (but problem with this idea are already discussed in the documentation).
Does it make sense to make it easier to include assertions for properties of superclasses?
- Lingua principale
- Java
- Stelle
- 72
- Fork
- 47
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di assertj/assertj-generator
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 35/100
assertj/assertj-generator#278 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 30/100
assertj/assertj-generator#220 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
assertj/assertj-generator#219 ·
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 35/100
assertj/assertj-generator#204 · 1 reazione ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
assertj/assertj-generator#197 · 7 commenti ·
Tutte le issue di assertj/assertj-generator
Issue simili
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
liquid-java/liquidjava#321 ·
I maintainer di solito rispondono entro 2 giorni
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
I maintainer di solito rispondono entro 1 giorno
-
rename config parameterAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
OpenConext/OpenConext-access#1015 ·
I maintainer di solito rispondono entro 1 giorno
-
68000: FDBcc branch target is 2 bytes too low (uses inst_start+2 instead of the displacement word)Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
NationalSecurityAgency/ghidra#9702 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 85/100
apache/rocketmq-dashboard#5358 ·
I maintainer di solito rispondono entro 3 giorni