Mockito isA(Classe<T> clazz) Comment faire pour résoudre le type de sécurité?

dans mon test, j'ai la ligne suivante:

when(client.runTask(anyString(), anyString(), isA(Iterable.class)).thenReturn(...)

isA(Iterable.class) produit l'avertissement qu'il a besoin de unchecked conversion pour se conformer à Iterable<Integer> . Quelle est la syntaxe pour qui?

isA(Iterable<Integer>.class)
isA((Iterable<Integer>)Iterable.class

ne fonctionnent pas.

Des suggestions?

InformationsquelleAutor Mike | 2012-08-17
  1. 28

    Mockito/Hamcrest et classes génériques

    Oui, c'est un problème général avec Mockito/Hamcrest. Généralement à l'aide de isA() avec les classes génériques produit un avertissement.

    Il y a predifinis Mockito de rapprochement pour la plupart des classes génériques: anyList(), anyMap(), anySet() et anyCollection().

    Suggestions:

    anyIterable() dans Mockito 2.1.0

    Mockito 2.1.0 ajouté une nouvelle anyIterable() méthode pour la mise en correspondance Iterables:

    when(client.runTask(anyString(), anyString(), anyIterable()).thenReturn(...)

    Ignorer dans Eclipse

    Si vous voulez juste pour se débarrasser de l'avertissement dans Eclipse. Option existe depuis Eclipse Indigo:

    Fenêtre > Préférences > Java > Compilateur > Erreurs/Avertissements > Generic
    types > Ignorer inévitable de type générique problèmes

    Quick Fix avec @SuppressWarnings

    Je vous suggère de le faire si vous avez le problème qu'une seule fois. Personnellement, je ne me souviens pas d'avoir jamais un isA(Iterable.class).

    Comme Daniel Pryden dit, vous pouvez limiter la @SuppressWarnings à une variable locale ou une méthode d'aide.

    Utiliser un générique isA() matcher avec TypeToken

    Cela résout le problème pour de bon. Mais il a deux inconvénients:

    • La syntaxe n'est pas trop jolie et pourrait confondre certaines personnes.
    • Vous avez un supplément de dépendance à la bibliothèque fournissant la TypeToken classe. Ici j'ai utilisé la TypeToken classe de Goyave. Il y a aussi un TypeToken classe dans Gson et un GenericType de JAX-RS.

    En utilisant le générique comparateur de:

    import static com.arendvr.matchers.InstanceOfGeneric.isA;
import static org.mockito.ArgumentMatchers.argThat;

//...

when(client.runTask(anyString(), anyString(), argThat(isA(new TypeToken<Iterable<Integer>>() {}))))
            .thenReturn(...);

    Générique comparateur de classe:

    package com.arendvr.matchers;

import com.google.common.reflect.TypeToken;
import org.mockito.ArgumentMatcher;

public class InstanceOfGeneric<T> implements ArgumentMatcher<T> {
    private final TypeToken<T> typeToken;

    private InstanceOfGeneric(TypeToken<T> typeToken) {
        this.typeToken = typeToken;
    }

    public static <T> InstanceOfGeneric<T> isA(TypeToken<T> typeToken) {
        return new InstanceOfGeneric<>(typeToken);
    }

    @Override
    public boolean matches(Object item) {
        return item != null && typeToken.getRawType().isAssignableFrom(item.getClass());
    }
}
    • Un sans doute mieux de l'API (par rapport à un tas de "anyList", "anyIterable", méthodes...) serait d'avoir un T any(T) méthode. Dans JMockit, par exemple, on peut écrire la suite, tout en évitant un "unchecked" avertissement: Iterable<Integer> col = null; client.runTask(anyString, anyString, withAny(col));.
    InformationsquelleAutor Arend v. Reinersdorff
  2. 7

    Voici ce que je fais:

    //Cast from Class<Iterable> to Class<Iterable<Integer>> via the raw type.
//This is provably safe due to erasure, but will generate an unchecked warning
//nonetheless, which we suppress.
@SuppressWarnings("unchecked")
Class<Iterable<Integer>> klass 
    = (Class<Iterable<Integer>>) (Class) Iterable.class;  

//later

isA(klass) //<- now this is typesafe
    InformationsquelleAutor Daniel Pryden
  3. 3

    Vous pouvez ajouter @SuppressWarnings("unchecked") - dessus de l'instruction. Pas d'autre moyen, mais si cela vous dérange, vous pouvez déplacer le casting d'une méthode d'aide.

    • C'est-à-dire: le sucer, il n'y a aucun moyen de le faire ce qui n'implique pas un décoché exprimés.
    • Oui, bien que vous pouvez cacher dans une méthode d'aide s'il vous fait vous sentir mieux 😛
    InformationsquelleAutor Garrett Hall
  4. 2

    Il n'existe aucun moyen pour ce faire. Pour simplifier, vous ne pouvez pas initialiser cette variable sans avertissement :

    Class<Iterable<Integer>> iterableIntegerClass = ?

    Une solution pourrait être d'utiliser le pseudo-typedef antipattern,
    vous permet de créer et d'utiliser un IntegerIterable interface 

    interface IntegerIterable extends Iterable<Integer> {}

    puis 

    isA(IntegerIterable.class)

    ne plus produire d'avertissement. Mais vous devrez étendre la classe la mise en œuvre de Iterable de les laisser implémente IntegerIterable 🙂 Par exemple :

    public class IntegerArrayList extends ArrayList<Integer> implements IntegerIterable {}

    Mmm savoureux...

    Donc, je vais vous suggérons de considérer à juste papier sur les fissures en ajoutant à votre méthode :

    @SuppressWarnings("unchecked")
    InformationsquelleAutor gontard