TDD avec MS Test

Question

Comme tous les bons programmeurs, j'essaie d'obtenir certaines choses directement lorsque j'utilise TDD avec MS Test. Je suis la disposition de base Arrange, Act, Assert, et quelque chose semble trop compliqué pour mon code Act. Je pars du principe qu'il ne devrait y avoir qu'une action dans la ligne de la Loi. Donc, étant donné mon exemple de code ci-dessous, est-ce que je sors de la piste en effectuant d'abord une action et en testant ensuite son état? Merci pour la contribution.

[TestMethod] 
    public void TheCountOfAllRecordsIsGreaterThanZero() 
    { 
     //Arrange 
     var auditLog = new AuditMasterClass(); 

     //Act 

     //Create a new record in a local list of objects 
     auditLog.LogAction("MyPersonName", DateTime.Now, "Stuff", "MoreStuff", 
            "Desc", 
            "Comments", true, false, 
            "UndoStatement"); 

     //Return the count of objects in the local list 
     var count = auditLog.GetCommentCount(); 

     //Assert 
     Assert.IsTrue(count > 0); 
    } 

Answer 1

Le test semble bien pour moi - je ne serais pas trop dogmatique ici, mais si elle vous fait vous sentir mieux, vous pouvez marquer la ligne: var count = auditLog.GetCommentCount(); dans le cadre de la phase d'affirmation;)

Une chose Je voudrais changer dans le test est l'assertion réelle - utiliser Assert.AreNotEqual(0, count) ou Assert.IsTrue(count > 0, string.Format("Count was not greater than 0, it was {0}", count)) - de cette façon, vous obtiendrez un meilleur message d'erreur au cas où l'assertion échoue.

Answer 2

Je ne vois aucun problème avec ce que vous avez fait. Je tendance à inline la variable count et simplement

Assert.IsTrue(auditLog.GetCommentCount() > 0); 

mais ce n'est pas significativement différent. Votre test, tel qu'il est écrit, indique que vous attendez que LogAction() soit appelé avec un ensemble particulier de paramètres, il y aura un ou plusieurs commentaires dans le journal. C'est clair. Une chose que j'aime faire est d'affirmer le contraire de mon affirmation avant l'action, donc je sais que l'action est vraiment ce qui a provoqué la condition. Ce serait, bien sûr, être:

Assert.IsTrue(auditLog.GetCommentCount() == 0); 

juste avant votre acte.

Answer 3

Oui, il n'y a généralement qu'une seule action par test. Invoquer un getter peut être considéré comme n'étant pas une action et donc appartenant à la partie Assert du test.

Dans le test TheCountOfAllRecordsIsGreaterThanZero ci-dessus, la création d'un nouvel enregistrement fait partie de la section Arrangement - ou le test est mal nommé et peut être TheCountOfAllRecordsIncreasedUponLogAction.

Je voudrais également souligner qu'une action par cas de test peut signifier plusieurs lignes de code. L'idée n'est pas d'écrire un scénario entier avec une longue séquence d'actions.