Pointer null sur objet testé

Version imprimable

Bonjour,

Voici le test suivant qui fonctionne :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
 
package user;
 
import junit.framework.TestCase;
 
import org.junit.AfterClass;
import org.junit.BeforeClass;
import org.junit.Test;
 
public class UtilisateurTest extends TestCase
{
 
    @Test public void testUtilisateur()
    {    
        Utilisateur user = new Utilisateur(1, "toto");
        assertEquals(user.getId().intValue() == 1, true);
    }
 
}

Voici le même test en utilisant @Before et @After :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
package user;
 
import junit.framework.TestCase;
 
import org.junit.AfterClass;
import org.junit.BeforeClass;
import org.junit.Test;
 
public class UtilisateurTest extends TestCase
{
    public Utilisateur user;
 
    @BeforeClass public void runBeforeEachTest()
    {
        user = new Utilisateur(1, "toto");
    }
 
    @Test public void testUtilisateur()
    {    
        assertEquals(user.getId().intValue() == 1, true);
    }
 
    @AfterClass public void runAfterEachTest()
    {
        user = null;
    }
 
    /*public static junit.framework.Test suite()
    {
        return new JUnit4TestAdapter(UtilisateurTest.class);
    }
    */
}

Et bien l'exécution de celui-ci soulève l'exception :

Code:

1
2
3
 
java.lang.NullPointerException
    at user.UtilisateurTest.testUtilisateur(UtilisateurTest.java:20)

... comme si @BeforeClass et @AfterClass n'étaient pas exécutés.

Nota : @Before et @After génère le même soucis.

Pourquoi ? ;)

Merci. ;)

Pour information, voici le main de test :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
package user;
 
import junit.framework.Test;
import junit.framework.TestSuite;
 
public class AllTests
{
    public static Test suite()
    {
        TestSuite suite = new TestSuite("Test for user");
        //$JUnit-BEGIN$
        suite.addTestSuite(UtilisateurTest.class);
        //$JUnit-END$
        return suite;
    }
 
}

28/03/2007, 15h31
Mister Nono

En fait les méthode @Before et @After ne sont pas appelées automatiquement.

Pourquoi ? :?
28/03/2007, 15h44
maxf1
Parce qu'avant chaque methode et après chaque méthode il faut implémenter les methodes spécifiques et que chaque object entre tes 2 methodes n'est pas "sauvagarder".

Donc dans ton cas il fait bien le before, mais ensuite il réinitialise tout, fait ton test => donc ca plante vu qu'il a tout réinitiliser :

Donc ton before doit implementer :
Code:

1 2 3 protected void setUp() throws Exception { //la tu met le code qui est dans ton before }
Et ton after :
Code:

1 2 3 protected void tearDown() throws Exception { //la tu met le code qui est dans ton after }

Merci maxf1, cela faisait longtemps que tu ne m'avais pas écrit (GAG :D).

Voici donc une classe de test qui fonctionne :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
package user;

import junit.framework.TestCase;

import org.junit.After;
import org.junit.Before;
import org.junit.Test;

public class UtilisateurTest extends TestCase
{
    public Utilisateur user;
    
    @Before public void runBeforeEachTest()
    {
        
    }

    @Test public void testUtilisateur()
    {    
        System.out.println("TEST");
        assertEquals(user.getId().intValue() == 1, true);
    }
    
    @After public void runAfterEachTest()
    {
       
    }
    
    protected void setUp() throws Exception {
        System.out.println("BEFORE");
        user = new Utilisateur(2, "toto");
    }
    
    protected void tearDown() throws Exception {
        System.out.println("AFTER");
        user = null;
    }
    
}

... mais à quoi servent donc les 2 méthodes vident (en gras) ? :?

A+ ;)

28/03/2007, 21h44
maxf1

Pour faire une reponse simple : A RIEN.

Ce sont des methodes qui vont etre executer comme celle la :
testUtilisateur.

EN gros il va passer dans l'ordre dans :

- setUp, puis runBeforeEachTest, puis tearDown
- setUp, puis testUtilisateur, puis tearDown
- setUp, puis runAfterEachTest, puis tearDown

J'espere que tu a compris. C'est toi qui a inventé les méthodes en gras ou tu as vu ca sur un tuto?
29/03/2007, 09h00
Mister Nono

Bonjour maxf1,

Pour te répondre, j'ai suivi le tutorial suivant : http://www.instrumentalservices.com/...d=45&Itemid=52

En fait, les 2 méthodes en gras ne sont jamais exécutées contrairement à ce qui est expliqué dans le tutorial.

A+ ;)
29/03/2007, 09h17
maxf1

Humm ok je ne connaissait pas cette syntaxe c'est pour ca que je la trouvé bizarre. Utilises-tu bien le JUnit 4.0? Car il me semble que ce doit etre une amélioration dans 4.0 de mettre @Before devant la méthode que tu veux appeler avant (afin de pouvoir nommer la methode comme tu le souhaites à la place de SetUp).

Et Idem pour after.

Edit de plus il me semble que tu n'es pas obligé et c'est surement ca qui fou la m..d. c'est que ta classe extends TestCase hors sur le tuto la classe n'étend pas TestCase.

Si t'y arrives avec tes méthodes fait nous signes ca peut-etre pas mal. Perso j'utilise toujours 3.8 donc je ne connaissais pas les nouveautés de 4.0 mais grace a toi j'en ai vu un peu la couleur.
29/03/2007, 09h22
Mister Nono

J'utilise bien JUnit 4.3.

Tu as compris la même chose que moi sur le tutorial, mais le soucis c'est qu'à l'exécution ces 2 méthodes ne sont pas executées.

Et je cherche donc où se trouve l'astuce manquante. :?

A+ ;)
29/03/2007, 09h44
maxf1

Tu as essaye en n'étendant pas ta classe de TestCase car c'est surement ca le souci!! Etant donnée que ca étant TestCase il veut les methodes setUp et tearDown pour avant et après.

Hors dans le tuto ta classe ne doit pas étendre TestCase. Pour que l'interpreteur JUnit regarde les @before et @after (pour les majuscule je ne sais pas du tout si c'est obligatoire).

Citation:

Envoyé par maxf1

Tu as essaye en n'étendant pas ta classe de TestCase car c'est surement ca le souci!! Etant donnée que ca étant TestCase il veut les methodes setUp et tearDown pour avant et après.

Hors dans le tuto ta classe ne doit pas étendre TestCase. Pour que l'interpreteur JUnit regarde les @before et @after (pour les majuscule je ne sais pas du tout si c'est obligatoire).

Oui, maintenant cela fonctionne mais l'appel de la classe dans le TestSuite ne fonctionne plus puisqu'il attend un TestCase (en caractère gras) :
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 package user; import junit.framework.Test; import junit.framework.TestSuite; public class AllTests { public static Test suite() { TestSuite suite = new TestSuite("Test for user"); //$JUnit-BEGIN$ suite.addTestSuite(UtilisateurTest.class); //$JUnit-END$ return suite; } }
On tourne en rond dans cette histoire. :?

29/03/2007, 10h38
maxf1

m'enfin on tourne pas trop en rond c'est juste que maintenant ta classe de test marche suivant le model 4.x

Le probleme est qu'apparement testSuite ne marche pas avec 4.x puisque les classes n'implémente pas testCase. Donc il y a 2 solutions:

tu extends testCase mais tes methodes doivent obligatoirement s'appeler setUp et tearDown et dans ce cas tu peux utiliser testSuite.

tu n'etends pas testCase et tu peux appeler tes methodes comme tu le souhaite avec @before et @after mais dans ce cas tu dois trouver un parade a testSuite (etant donné que je suis toujours en 3.8 depuis tout à l'heure je n'ai pas ce probleme). La nouveauté c'est bien mais quand c'est trop neuf y manque des trucs..... :P