À l'aide de MOQ pour tester un référentiel

Votre démarche est la bonne, cependant je voudrais régler quelques petites choses. J'ai fait quelques changements à votre test et se moquant de composants que vous pouvez voir ci-dessous.

De Test De L'Unité

Vous la méthode en cours de test au sein de votre Unité de Test (voir ci-dessous), n'a pas vraiment correspondre avec la méthode que vous avez défini dans votre question.

De test de l'unité de l'appel de méthode:

SubmissionVersion.DeleteNote(repo.Object, subVersion.Object, note.Id.Value);

Méthode d'essai:

public static SubmissionVersion DeleteNote
  (IRepository repository, SubmissionVersion version, Guid noteId)

Je suppose que la méthode ci-dessus fait partie d'une autre classe, je l'ai appelé comme NotesHelper - Pas le nom idéal pour le dépôt des appels, mais c'est juste pour se la compilation de travail.

Je serais personnellement séparer de votre Unité de test dans 2 tests Unitaires. Vérifier si les méthodes sont appelées, et l'autre est de vérifier si les notes ont été supprimées de la collection.

Aussi au lieu de créer un moqué SubmissionVersion, j'ai créé un fakeSubmissionVersion.
C'est parce que les routines au sein de SubmissionVersion peut pas être mockable. Vous pouvez également faire un état des tests (de préférence) pour s'assurer de la version.Les Notes.Supprimer(note); il a été appelé.

[Fact]
public void DeleteNote_Deletion_VerifyExpectedMethodsInvokecOnlyOnce()
{
    //Arrange
    var note = new Note { Id = Guid.NewGuid() };
    var fakeSubmissionVersion = new SubmissionVersion() { Notes = new List<Note>() };
    var repo = new Mock<IRepository>();
    repo.Setup(x => x.GetById<Note>(It.IsAny<Guid>())).Returns(note);

    //Act
    NotesHelper.DeleteNote(repo.Object, fakeSubmissionVersion, note.Id.Value);

    //Assert
    repo.Verify(x => x.GetById<Note>(note.Id), Times.Once());
    repo.Verify(x => x.Save(fakeSubmissionVersion), Times.Once());
    repo.Verify(x => x.Delete(note), Times.Once());
}

Le test ci-dessus peuvent être encore améliorées par la définition de chacun de Vérification dans son propre test.

[Fact]
public void DeleteNote_Deletion_VerifyDeleteMethodCalledOnlyOnce()

Cette façon, si un test échoue, nous saurons exactement la méthode qui n'a pas été appelé avec l'attente. Parfois, le fait d'avoir plusieurs vérifications ci-dessus peut être problématique. Par exemple, si la méthode n'a pas été appelé, vous ne le saurez jamais la Suppression de la méthode a été appelée ou non. C'est parce que l'exception a été levée lorsque la méthode d'enregistrement n'a pas été appelé et l'exécution du test a été résilié.

Cela aurait pour conséquence de multiples tests, mais ils sont à la fois plus lisible et maintenable. Bien sûr, vous pouvez refactoriser le code commun dans une usine ou de la méthode d'assistance.

[Fact]
public void DeleteNote_RemoveNotes_ReturnsExpectedNotes()
{
    //Arrange
    var note = new Note { Id = Guid.NewGuid() };
    var fakeSubmissionVersion = new SubmissionVersion() { Notes = new List<Note>() { note } };
    var repo = new Mock<IRepository>();
    repo.Setup(x => x.GetById<Note>(It.IsAny<Guid>())).Returns(note);

    //Act
    NotesHelper.DeleteNote(repo.Object, fakeSubmissionVersion, note.Id.Value);

    //Assert
    Assert.AreEqual(0, fakeSubmissionVersion.Notes.Count);
}

Supplémentaires Remarque: Également en fournissant un descriptif de l'Unité de la méthode d'essai noms améliore la lisibilité des Tests Unitaires.

OriginalL'auteur Spock