Utilisation de Moq pour remplacer les méthodes virtuelles dans la même classe

Question

Nous utilisons Moq pour tester nos classes de service, mais nous sommes bloqués sur la façon de tester les situations où une méthode de service appelle une autre méthode de service de la même classe. J'ai essayé de définir la méthode appelée virtuelle, mais je n'arrivais toujours pas à savoir quoi faire dans Moq. Par exemple:

public class RenewalService : IRenewalService 
{ 
    //we've already tested this 
    public virtual DateTime? GetNextRenewalDate(Guid clientId) 
    { 
     DateTime? nextRenewalDate = null; 
     //...<snip> a ton of already tested stuff... 

     return nextRenewalDate; 
    } 

    //but want to test this without needing to mock all 
    //the methods called in the GetNextRenewalDate method 
    public bool IsLastRenewalOfYear(Renewal renewal) 
    { 
     DateTime? nextRenewalDate = GetNextRenewalDate(renewal.Client.Id); 
     if (nextRenewalDate == null) 
      throw new Exceptions.DataIntegrityException("No scheduled renewal date, cannot determine if last renewal of year"); 
     if (nextRenewalDate.Value.Year != renewal.RenewDate.Year) 
      return true; 
     return false; 
    } 
} 

Dans l'exemple ci-dessus, notre méthode de GetNextRenewalDate est assez compliqué, et nous avons déjà testé l'unité. Cependant, nous voulons tester le plus simple IsLastRenewalOfYear sans avoir besoin de se moquer de tout ce qui est nécessaire pour GetNextRenewalDate. Fondamentalement, nous voulons juste se moquer de GetNextRenewalDate.

Je me rends compte que je pourrais créer une nouvelle classe qui remplace GetNextRenewalDate et tester la nouvelle classe, mais est-il possible de tirer parti de Moq pour rendre cela plus simple?

Answer 1

Vous pouvez probablement utiliser un mock partiel dans ce scénario, bien que toutes vos méthodes auraient besoin d'être virtuel:

var mock = new Moq.Mock<RenewalService>(); 
    mock.Setup(m => m.GetNextRenewalDate(It.IsAny<Guid>())).Returns(null); 
    mock.CallBase = true; 
    var results = mock.Object.IsLastRenewalOfYear(...); 

Answer 2

Modifier: Je suis maintenant d'accord que ce n'est pas la bonne réponse pour le cas d'Andrew. Je veux laisser cette réponse ici pour le fil des commentaires. S'il vous plaît ne pas voter vers le bas plus :)

Avant modifier:

En général, les cadres simulacres d'objet ne sont pas conçus pour simplifier le scénario de classe unique, ils sont conçus pour isoler votre code afin que vous puissiez tester une seule classe.

Si vous essayez d'utiliser un cadre d'objet fictif pour résoudre ce problème, le framework va simplement créer une classe dérivée et surcharger cette méthode. La seule différence sera que vous pouvez le faire en 3 lignes au lieu de 5, car vous n'aurez pas à créer la définition de la classe dérivée.

Si vous souhaitez utiliser des objets fantaisie pour isoler ce comportement, vous devez fractionner cette classe un peu. La logique GetNextRenewalDate peut vivre à l'extérieur de l'objet RenewalService.

Le fait que vous rencontriez ce problème peut montrer qu'il existe une conception plus simple ou plus fine à découvrir. Trouver un cours un peu moins concret, avec un nom comme «manager» ou «service», est souvent un indice que vous pourriez décomposer votre conception en classes plus petites et obtenir une meilleure réutilisabilité et maintenabilité.

Answer 3

var mock = new Moq.Mock<RenewalService> { CallBase = true }; 
mock.Setup(m => m.GetNextRenewalDate(It.IsAny<Guid>())).Returns(null); 
var results = mock.Object.IsLastRenewalOfYear(...);