JUnit - assertSame

Question

Quelqu'un peut-il me dire pourquoi assertSame() échoue lorsque j'utilise des valeurs> 127?

 import static org.junit.Assert.*; 

     ... 

     @Test 
     public void StationTest1() { 
      .. 

      assertSame(4, 4);   // OK 
      assertSame(10, 10);  // OK 
      assertSame(100, 100);  // OK 
      assertSame(127, 127);  // OK 
      assertSame(128, 128);   // raises an junit.framework.AssertionFailedError! 
      assertSame(((int) 128),((int) 128)); // also junit.framework.AssertionFailedError! 
     } 

J'utilise JUnit 4.8.1.

Answer 1

La raison en est l'autoboxing de Java. Vous utilisez la méthode:

public static void assertSame(Object expected, Object actual) 

Il fonctionne uniquement avec des objets. Lorsque vous passez int s à cette méthode, Java appelle automatiquement

Integer.valueOf(int i) 

avec ces valeurs. Donc, la distribution à int n'a aucun effet.

Pour les valeurs inférieures à 128 Java a un cache, donc assertSame() compare l'objet Integer avec lui-même. Pour les valeurs supérieures à 127, Java crée de nouvelles instances, donc assertSame() compare un objet Integer avec un autre. Parce qu'ils ne sont pas la même instance, la méthode assertSame() renvoie false.

Vous devez utiliser la méthode:

public static void assertEquals(Object expected, Object actual) 

à la place. Cette méthode utilise la méthode equals() de Object.

Answer 2

assertSame prend deux arguments Object, et donc le compilateur doit autoboxer vos littéraux int en Integer.

Cela équivaut à

assertSame(Integer.valueOf(128), Integer.valueOf(128)); 

Maintenant, pour des valeurs comprises entre -128 et 127, la machine virtuelle Java en cache les résultats de Integer.valueOf, de sorte que vous obtenez le même objet Integer retour à chaque fois. Pour les valeurs en dehors de cette plage, vous récupérez de nouveaux objets.

Donc pour assertSame(127, 127), JUnit compare les mêmes objets, donc cela fonctionne. Pour assertSame(128, 128), vous obtenez différents objets, donc cela échoue.

Juste une autre raison de faire attention à l'autoboxing.