Bonne pratique pour ajouter des méthodes protégées pour des tests unitaires?

Question

Dire que j'ai la classe suivante:

public class ProblematicObject { 
    public static void methodToTest() { 
     SomeDependency dep = DependencyGetter.getDependency(SomeDependency.class); 
     dep.doStuff(); 
    } 
} 

Est-ce une pratique acceptable de modifier et ajouter des méthodes à cette classe pour le plaisir de faire plus propre de tests unitaires (aka éviter PowerMock)? Voici ce que la classe ci-dessus serait ressembler à:

public class ProblematicObject { 

    // new 
    private static SomeDependency dep; 

    // updated 
    public static void methodToTest() { 
     getSomeDependency().doStuff(); 
    } 

    // new 
    private SomeDependency getSomeDependency() { 
     if (this.dep == null) { 
      return DependencyGetter.getDependency(SomeDependency.class); 
     } 
     return dep; 
    } 

    // new, only used for testing, not in any impl code 
    @Deprecated 
    protected void setDependencyGetter(SomeDependency dep) { 
     this.dep = dep; 
    } 

} 

Je me souviens avoir lu quelque part que l'ajout de méthodes pour le bien des tests (au lieu de refactoring la classe problématique) est mal vu.

Answer 1

Dans cet exemple particulier, vous améliorez la conception de votre classe en la rendant testable.

Ce nouveau test unitaire vous a fait réfléchir du point de vue du client ProblematicObject et vous a permis d'injecter cette dépendance.