Cas de test Junit pour la méthode privée

Normalement, lorsque vous ne parvenez pas à tester votre code, vous devriez penser à le réécrire, afin qu'il soit testable. Les méthodes privées ne peuvent pas être appelées de manière régulière, en dehors de votre classe, même pas dans un test JUnit.

Il y a plusieurs choses que vous pouvez faire pour le rendre testable:

1. Réécrivez votre code pour diviser la méthode en plus petites. Doit toujours être fait, le cas échéant.

2. Vous pouvez rendre le package de méthode privé (supprimer le modificateur privé), afin que votre classe de test, qui doit être dans le même package, puisse y accéder.

3. Vous pouvez rendre la méthode protégée, ainsi vous pouvez hériter de votre classe dans votre test et appeler indirectement cette méthode sur votre classe héritée.

4. Vous pouvez appeler une méthode, qui appelle la méthode, que vous souhaitez tester et faire vos affirmations en conséquence.

Si vous avez tendance à changer la visibilité de la méthode en package private ou protected, vous pouvez annoter votre méthode comme @VisibleForTesting , afin que des outils comme Sonar et d'autres membres de l'équipe sachent pourquoi elle n'est pas privée.

/ p>

J'espère que cela aide.

Tout d'abord, deux ou trois choses:

1. Pourquoi loading_status est-il statique? Fondamentalement, 2 instances de cette classe utiliseraient la même variable loading_status. loading_status ne doit pas être statique ou checkLoadingStatus doit être statique également.
2. loading_status est terriblement nommé. Si vous voulez le garder statique, vous devez le nommer LOADING_STATUS. Si ce n'est pas statique, appelons-le loadingStatus. C'est la convention Java.
3. Vous devez créer un type d'énumération approprié pour loading_status au lieu qu'il s'agisse d'un entier.
4. while (loading_status == IN_PROGRESS); est tout simplement une mauvaise pratique. Au moins ce que vous pouvez faire est: while (loading_status == IN_PROGRESS) {Thread.sleep (100); }; Cependant, un délai d'attente est une meilleure pratique. Quelque chose comme:

src
+-- main
+-- +-- com/app/MyClass.java
+-- test
    +-- com/app/MyClassTest.java // this test class will able to access package-private methods in MyClass

Et enfin je suppose que quelque chose mettra à jour loadingStatus à terminé ou quelque chose. Je suppose que vous utilisez également synchronisé. Si le chargement se produit sur 2 threads différents, vous vous retrouverez dans un dead-lock. Si checkLoadingStatus entrez d'abord le bloc synchronisé, donc lorsque le chargement est terminé, le thread de chargement ne pourra jamais entrer dans le bloc synchronisé, car le checkLoadingStatus maintient le verrou.

Dernier point mais non des moindres pour répondre à votre question , si vous voulez garder votre méthode privée, vous pouvez l'invoquer par réflexion, mais c'est encore une mauvaise pratique. En règle générale, vous devez éviter de tester les méthodes privées et de tester les méthodes qui les invoquent. Si toutefois vous avez vraiment besoin de tester unitaire une méthode particulière, rendez-la privée au lieu de privée, puis vous pouvez créer un test unitaire dans le même package où se trouve la classe qui contient votre méthode. Et vous pouvez ajouter un commentaire à la méthode indiquant qu'elle n'est visible que pour les tests unitaires. Exemple de votre structure de code dans ce cas:

long timeoutMillis = 5000L; // Should come from a configuration or something
long endTimeMillis = System.currentTimeMillis() + timeoutMillis;
while (loadingStatus == IN_PROGRESS) {
    long remainingMillis = end - System.currentTimeMillis();
    if (remaining <= 0) {
        // Timeout while loading
        loadingStatus = LOADING_ERROR;
        break;
    }
    Thread.sleep(50);
}
// ...

Votre méthode privée fonctionne selon la valeur de la variable privée interne loading_status , donc lors de vos tests, vous devriez simplement pouvoir changer cette variable. Pour ce faire, vous pouvez procéder comme suit:

<dependency>
  <groupId>org.powermock</groupId>
  <artifactId>powermock-api-mockito</artifactId>
  <version>1.6.4</version>
</dependency>

En supposant que vous devez inclure la dépendance mockito sur votre projet:

par exemple Maven

package com.test

import com.test.SchemaMaintenance;
import org.junit.Test;
import org.mockito.internal.util.reflection.Whitebox;

public class SchemaMaintenanceTest {

    @Test
    public void TestSchema() throws Exception {
        SchemaMaintenance schema = new SchemaMaintenance();
        Whitebox.setInternalState(schema,"loading_status",2);
        schema.startSchemaMaintenance();
    }

}