Dato il seguente esempio (utilizzando JUnit con i corrispondenti Hamcrest):
Map<String, Class<? extends Serializable>> expected = null;
Map<String, Class<java.util.Date>> result = null;
assertThat(result, is(expected));
Questo non viene compilato con la assertThat
firma del metodo JUnit di:
public static <T> void assertThat(T actual, Matcher<T> matcher)
Il messaggio di errore del compilatore è:
Error:Error:line (102)cannot find symbol method
assertThat(java.util.Map<java.lang.String,java.lang.Class<java.util.Date>>,
org.hamcrest.Matcher<java.util.Map<java.lang.String,java.lang.Class
<? extends java.io.Serializable>>>)
Tuttavia, se cambio la assertThat
firma del metodo in:
public static <T> void assertThat(T result, Matcher<? extends T> matcher)
Quindi la compilazione funziona.
Quindi tre domande:
- Perché esattamente la versione attuale non viene compilata? Anche se capisco vagamente le questioni di covarianza qui, non potrei certamente spiegarlo se dovessi.
- C'è qualche svantaggio nel cambiare il
assertThat
metodo inMatcher<? extends T>
? Ci sono altri casi che potrebbero rompersi se lo facessi? - C'è qualche punto per la generalizzazione del
assertThat
metodo in JUnit? LaMatcher
classe non sembra richiederlo, dal momento che JUnit chiama il metodo match, che non viene digitato con alcun generico, e sembra solo un tentativo di forzare un tipo di sicurezza che non fa nulla, poiché laMatcher
volontà non lo fa partita e il test fallirà a prescindere. Nessuna operazione pericolosa coinvolta (o almeno così sembra).
Per riferimento, ecco l'implementazione JUnit di assertThat
:
public static <T> void assertThat(T actual, Matcher<T> matcher) {
assertThat("", actual, matcher);
}
public static <T> void assertThat(String reason, T actual, Matcher<T> matcher) {
if (!matcher.matches(actual)) {
Description description = new StringDescription();
description.appendText(reason);
description.appendText("\nExpected: ");
matcher.describeTo(description);
description
.appendText("\n got: ")
.appendValue(actual)
.appendText("\n");
throw new java.lang.AssertionError(description.toString());
}
}