Wie kann ich behaupten, dass ein Gerät getestet, die Methode True zurück?

Dies ist meine Domäne-Klasse

public partial class Department
{
    public int DepartmentId { get; set; }
    [Required]
    public string DepartmentCode { get; set; }
    [Required]
    public string DepartmentFullName { get; set; }
    public bool Status { get; set; }
    public System.DateTime CreatedOn { get; set; }
}

In meiner MVC Anwendung, dies ist, wie mein DepartmentService Klasse aussieht.

public class DepartmentService : IDepartmentService
{
    private IUnitOfWork _UoW;
    private IRepository<Department> repository;
    public DepartmentService(IUnitOfWork UoW)
    {
        _UoW = UoW;
        repository = _UoW.GetRepository<Department>();
    }

    public IList<Department> GetAllDepartments()
    {
        return repository.GetAll();
    }

    public bool SaveDepartment(Department newDepartment)
    {
        try
        {
            repository.Add(newDepartment);
            _UoW.Save();
        }
        catch (Exception)
        {
            throw;
        }
        return true;
    }
}

Schrieb ich einen unit-test für GetAllDepartments Methoden, wie unten beschrieben.

[Test]
public void When_GetAllDepartments_Is_Called_RepositoryGetAll_ShouldBeCalled()
{
    //Arrange
    var mockUnitOfWork = new Mock<IUnitOfWork>();
    var mockRepository = new Mock<IRepository<Department>>();
    mockUnitOfWork.Setup(x => x.GetRepository<Department>())
                  .Returns(mockRepository.Object);
    var sut = new DepartmentService(mockUnitOfWork.Object);

    //Act
    sut.GetAllDepartments();

    //Assert
    mockRepository.Verify(x => x.GetAll());
}

Ich testen wollen SaveDepartment Methode, wo, Wann-Abteilung ist erfolgreich gespeichert, es sollte true zurückgeben. Ich bin nicht in der Lage zu schreiben unit-Tests für diese.

Möchte ich auch testen, wenn DepartmentCode oder DepartmentFullName leer ist, und falls das Speichern versucht, wird eine Ausnahme geworfen werden sollte.

Dies ist, was ich habe, so weit.

[Test]
public void ShouldSucessfully_SaveNewDepartment()
{
    //Arrange
    var mockUnitOfWork = new Mock<IUnitOfWork>();
    var mockRepository = new Mock<IRepository<Department>>();
    Department newDept = new Department { 
          CreatedOn = DateTime.Now,
          Status = true, 
          DepartmentFullName = "DFN", 
          DepartmentCode = "DC" };
    mockUnitOfWork.Setup(x => x.GetRepository<Department>())
                  .Returns(mockRepository.Object);
    var sut = new DepartmentService(mockUnitOfWork.Object);

    //Act
    sut.SaveDepartment(newDept);

    //Assert 
    //???
}

bool result = sut.SaveDepartment(newDept); Assert.IsTrue(result);?
Nicht sut.SaveDepartment(newDept); tatsächlich fügen Sie einen Datensatz in der DB ? Ich denke, dass ich nicht wollen, dass dies passiert. Oder sollte es passieren, in unit-Tests ?
Sie wies die mock unitofwork, Euch wieder ein mock-repository. Solche repository nicht handeln auf die Datenbank
Aktualisiert haben die OP. Wie kann ich testen, für einen Misserfolg ? Denn wenn DepartmentCode oder FullName fehlt, dann sollte die Ausnahme geworfen werden.
Sorry, konnte Sie markieren als Antwort, wenn Sie zufrieden sind, indem Sie keine Antwort?

InformationsquelleAutor NoobDeveloper | 2013-09-25

c#moq nunit unit-testing

Zunächst - verschieben Sie alle gemeinsam arrangieren-code, um das SetUp-Methode:

private Mock<IUnitOfWork> mockUnitOfWork;
private Mock<IRepository<Department>> mockRepository;
private DepartmentService sut;

[SetUp]
public void SetUp()
{
    mockUnitOfWork = new Mock<IUnitOfWork>();
    mockRepository = new Mock<IRepository<Department>>();
    mockUnitOfWork.Setup(x => x.GetRepository<Department>())
                  .Returns(mockRepository.Object);
    sut = new DepartmentService(mockUnitOfWork.Object);
}

//tests will be here

Machen, dass die tests viel einfacher zu Lesen und zu verwalten. Nächste kleben nicht an der Umsetzung, wenn die Benennung des tests:

When_GetAllDepartments_Is_Called_RepositoryGetAll_ShouldBeCalled

Was ist, wenn Sie umbenennen repository-Methode FindAll? Was ist, wenn service-Methode umbenannt werden? Test obsolet. Und niemand wird wissen, dass. Sie sollten beschreiben, WAS Ihr SUT stattdessen tun sollte, WIE:

[Test]
public void ShouldGetAllDepartments()
{
    var expected = new List<Department>{ CreateDepartment(), CreateDepartment()};
    mockRepository.Setup(r => r.GetAll()).Returns(expected);
    var actual = sut.GetAllDepartments();
    Assert.That(actual, Is.EqualTo(expected));
    mockRepository.VerifyAll();
}

Wie Sie sehen können, testen Sie Namen geändert. Auch ich überprüfen, verschiedene Dinge sind hier - nicht nur-repository genannt wurde (dass der check entfernt werden können, tatsächlich), aber das service liefert genau die gleichen Abteilungen, die es bekommt aus dem repository. Das ist, was service bedeutet. Und der zweite test:

[Test]
public void ShouldSucessfullySaveNewDepartment()
{
    var department = CreateDepartment();
    mockRepository.Setup(r => r.Add(department));
    var result = sut.SaveDepartment(department);
    Assert.True(result);
    mockRepository.VerifyAll();
    mockUnitOfWork.Verify(u => u.Save());
}

Es überprüft, ob folgende service-Verhalten: der Dienst sollte pass im repository exakt gleichen Abteilung Instanz, die übergeben wurde, um Dienst, und es sollte true zurückgeben, es fordert auch sparen Arbeitseinheit zum senden von Daten.

BTW Wie man oben sehen kann, die ich verwenden helper-Methode um tests mehr sauber:

private Department CreateDepartment()
{
    return new Department { 
        CreatedOn = DateTime.Now, 
        Status = true, 
        DepartmentFullName = "DFN", 
        DepartmentCode = "DC" 
    };
}

- Und bonus - überprüfen, dass der Dienst nicht speichert-Abteilung, die bereits vorhanden ist:

[Test]
public void ShouldNotSaveExistingDepartment()
{
    mockUnitOfWork.Setup(u => u.Save()).Throws<NonUniqueEntityException>();
    var result = sut.SaveDepartment(CreateDepartment());
    Assert.False(result);
    mockUnitOfWork.VerifyAll();
}

Wie Sie sehen können, die das erwartete Verhalten ist ganz einfach - bei der Einheit von Arbeit wirft NonUniqueEntityException während Abteilung speichern, sollte der Dienst "false" zurück. Ja, ich denke, es ist besser, um den Wert false zurück. Hier ist der service-code, die macht dieser test pass:

public bool SaveDepartment(Department department)
{
    try
    {
        repository.Add(department);
        _UoW.Save();
        return true;
    }
    catch (NonUniqueEntityException e)
    {
        //log exception
        return false;
    }            
}

InformationsquelleAutor Sergey Berezovskiy

Entfernen Sie zuerst die redundanten try...catch.
Es gibt keinen Punkt, in dem abfangen einer Ausnahme nur zu rethrow es.
Soweit der test, den Sie fragte, für hier ist das snippet:

    [Test]
    public void ShouldSucessfully_SaveNewDepartment()
    {
        //Arrange
        var mockUnitOfWork = new Mock<IUnitOfWork>();
        var mockRepository = new Mock<IRepository<Department>>();
        Department newDept = new Department { CreatedOn = DateTime.Now, Status = true, DepartmentFullName = "DFN", DepartmentCode = "DC" };
        mockUnitOfWork.Setup(x => x.GetRepository<Department>()).Returns(mockRepository.Object);

        var sut = new DepartmentService(mockUnitOfWork.Object);

        //Act
        bool result = sut.SaveDepartment(newDept);

        //Assert 
        Assert.That(result, Is.True);
    }

   [Test]
    public void ShouldThrowExceptionWhenExceptionThrownInternally_SaveNewDepartment()
    {
        //Arrange
        var mockUnitOfWork = new Mock<IUnitOfWork>();
        var mockRepository = new Mock<IRepository<Department>>();
        mockUnitOfWork.Setup(x => x.GetRepository<Department>()).Returns(mockRepository.Object);
        mockUnitOfWork.Setup(uow => uow.Save()).Throws<Exception>();

        var sut = new DepartmentService(mockUnitOfWork.Object);

        //Act
        TestDelegate action = () => sut.SaveDepartment(new Department());

        //Assert 
        Assert.Throws<Exception>(action);
    }

Möchte ich auch testen, wenn DepartmentCode oder DepartmentFullName leer ist, und falls das Speichern versucht, wird eine Ausnahme geworfen werden sollte.

Es hängt davon ab, wer Sie erwarten, um zu prüfen, für diese Werte: wenn Sie möchten, verlassen Sie die Datenbank oder unitofwork zu prüfen, für diese Felder ist dies nicht der Vorrichtung zu testen, die für bestimmte Fehlerbedingungen möchten Sie einen unit-test. Hier sollten Sie nur testen, was passiert, wenn uow wirft Ausnahme.

Wenn Sie planen, fügen Sie diese check-in der Abteilung service als können Sie Folgendes tun:

[Test]
    public void ShouldThrowExceptionWhenDepartmentCodeIsNull_SaveNewDepartment()
    {
        //Arrange
        var mockUnitOfWork = new Mock<IUnitOfWork>();
        Department newDept = new Department
                                 {
                                     CreatedOn = DateTime.Now, 
                                     Status = true, DepartmentFullName = "DFN", 
                                     DepartmentCode = null
                                 };

        var sut = new DepartmentService(mockUnitOfWork.Object);

        //Act
        TestDelegate action = () => sut.SaveDepartment(newDept);

        //Assert 
        Assert.Throws<ArgumentException>(action);
    }

vorausgesetzt, Sie verändert Ihre Methode wie folgt:

public bool SaveDepartment(Department newDepartment)
    {
        if (string.IsNullOrEmpty(newDepartment.DepartmentCode))
        {
            throw new ArgumentException("DepartmentCode must be not null");
        }

        repository.Add(newDepartment);
        _UoW.Save();
        return true;
    }

Danke. Ich habe nicht verstanden, warum brauchen wir diese Zeile code.mockUnitOfWork.Setup(Unit of work, uow => uow.Save()).Wirft<Ausnahme>(); Ausnahme sollte nur geworfen werden, wenn DepartmentCode oder DepartmentFullName leer ist. In Ihrem code, Objekt newDept hat alle seine Eigenschaften richtig eingestellt.
Wenn Sie erwarten, dass deine uow.Save () - Methode fehl, irgendwie sollten Sie test das erwartete Verhalten, wenn diese Methode fehlschlägt, egal, der Grund. In diesem besonderen Fall ist es cleare nicht die Eigenschaften im test. Ich 'll Bearbeiten Sie das Beispiel. Aber das wichtigste ist, dass, wenn Sie planen, testen Sie explizit die Werte in SaveDepartment sollten Sie die Letzte Fixture Methode, die ich schrieb. Wenn Sie planen, verlassen Sie sich auf die Datenbank-überprüfung auf null-Werte sollten Sie ein integration test auf der uow-und repository zusammen.

InformationsquelleAutor ilmatte

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.