Wie zu verwenden Moq in der unit test ruft eine andere Methode in derselben Klasse

Den Fokus auf die Verspottung der Klasse unter test hat dich geblendet, um das eigentliche problem.

Aus den Kommentaren in der Klasse unter test...

• 'Datenbank-geben Sie uns die Fahrzeug-Spezifikation"
• 'externe Fertigung-system bekommt der Reifen die Spezifikation"

Sie tatsächlich setzen zwei Abhängigkeiten, die sollten in die Klasse eingefügt.

Zur Erklärung dieser Antwort können sagen, diese Abhängigkeiten wie folgt aussah.

public interface IDatabase {
    Vehicle GetVehicleByRegistrationNumber(string registrationNumber);
}

public interface IExternalManufactureSystem {
    Tyre GetTyreSpecification(long vehicleIdentifier);
}

Das würde bedeuten, dass die Selecter müsste umgestaltet werden, zu erwarten, dass diese Abhängigkeiten.

public class Selecter : ISelecter {
    private IDatabase database;
    private IExternalManufactureSystem externalManufactureSystem;

    public Selecter(IDatabase database, IExternalManufactureSystem externalManufactureSystem) {
        this.database = database;
        this.externalManufactureSystem = externalManufactureSystem;
    }

    public Vehicle GetVehicleByRegistrationNumber(string registrationNumber) {
        //'Database will give us the vehicle specification'
        var vehicle = database.GetVehicleByRegistrationNumber(registrationNumber);

        //Then we do things with the vehicle object

        //Get the tyre specification
        vehicle.TyreSpecification = GetTyreSpecification(vehicle.VehicleIdentifier);

        return vehicle;
    }

    public Tyre GetTyreSpecification(long vehicleIdentifier) {
        //'external manufacture system gets the tyre specification'
        var tyre = externalManufactureSystem.GetTyreSpecification(vehicleIdentifier);

        //Then do thing with the tyre before returning the object

        return tyre;
    }
}

Von dort aus wäre es dann eine Frage der Spott nur die Abhängigkeiten explizit benötigt, um testen Sie das Verhalten der Methode im test.

selecter.GetTyreSpecification hat keinen Zugriff auf die Datenbank so dort ist kein Grund, Sie zu verspotten und injizieren es für den test.

[TestMethod]
public void GetTyreSpecification_test() {
    //Arrange
    var vehicleIdentifier = 123456;
    var expected = new Tyre { NumberOfTyres = 4, TyreSize = 18 };

    var mockSystem = new Mock<IExternalManufactureSystem>();
    mockSystem.Setup(s => s.GetTyreSpecification(vehicleIdentifier)).Returns(expected);

    var selecter = new Selecter(null, mockSystem.Object);

    //Act
    var actual = selecter.GetTyreSpecification(vehicleIdentifier);

    //Assert
    Assert.AreEqual(expected, actual);
}

selecter.GetVehicleByRegistrationNumber muss allerdings in der Lage sein, um die Reifen-Spezifikation von der anderen Methode, so dass dieser test müssen sowohl die Abhängigkeiten verspottet, damit Sie ausgeübt werden bis zur Fertigstellung.

[TestMethod]
public void GetVehicleByRegistrationNumber_test() {
    //Arrange
    var vehicleIdentifier = 123456;
    var registrationNumber = "ABC123";
    var tyre = new Tyre { TyreSize = 18, NumberOfTyres = 4 };
    var expected = new Vehicle {
        VehicleIdentifier = vehicleIdentifier,
        RegistrationNumber = registrationNumber,
        TyreSpecification = tyre
    };

    var mockSystem = new Mock<IExternalManufactureSystem>();
    mockSystem.Setup(s => s.GetTyreSpecification(vehicleIdentifier)).Returns(tyre);

    var mockDatabase = new Mock<IDatabase>();
    mockDatabase.Setup(s => s.GetVehicleByRegistrationNumber(registrationNumber)).Returns(expected);

    var selecter = new Selecter(mockDatabase.Object, mockSystem.Object);

    //Act
    var actual = selecter.GetVehicleByRegistrationNumber(registrationNumber);

    //Assert
    Assert.IsTrue(actual.RegistrationNumber == registrationNumber);
}    

Nun mit, dass aus dem Weg, wenn zum Beispiel der Selecter Klasse hatte die GetVehicleByRegistrationNumber als virtual Methode,

public virtual Tyre GetTyreSpecification(long vehicleIdentifier) {
    //...code removed for brevity.
}

Es ist ein Weg, die Sie verwenden können, moq stub das Thema unter test und mock, die Methode für die Prüfung. Dies ist nicht immer das beste design und ist als ein code smell. Es gibt jedoch Situationen, in denen Sie am Ende in diesem speziellen Szenario.

[TestMethod]
public void GetVehicleByRegistrationNumber_test2() {
    //Arrange
    var vehicleIdentifier = 123456;
    var registrationNumber = "ABC123";
    var tyre = new Tyre { TyreSize = 18, NumberOfTyres = 4 };
    var expected = new Vehicle {
        VehicleIdentifier = vehicleIdentifier,
        RegistrationNumber = registrationNumber,
        TyreSpecification = tyre
    };        

    var mockDatabase = new Mock<IDatabase>();
    mockDatabase.Setup(s => s.GetVehicleByRegistrationNumber(registrationNumber)).Returns(expected);

    var selecter = new Mock<Selecter>(mockDatabase.Object, null) {
        CallBase = true //So that base methods that are not setup can be called.
    }

    selecter.Setup(s => s.GetTyreSpecification(vehicleIdentifier)).Returns(tyre);

    //Act
    var actual = selecter.Object.GetVehicleByRegistrationNumber(registrationNumber);

    //Assert
    Assert.IsTrue(actual.RegistrationNumber == registrationNumber);
} 

In dem obigen Beispiel, wenn selecter.Object.GetVehicleByRegistrationNumber(registrationNumber) aufgerufen wird, die Basis Selecter gehüllt durch die mock-werden aufgerufen, die wiederum wird dann rufen Sie die verspottet GetTyreSpecification wurde überschrieben durch das setup auf der verspottet Prüflings.

Neigen Sie dazu, um dies zu sehen, wenn die Tests der abstrakten Klassen implementiert Mitglieder, die Abhängigkeiten von abstrakten Mitglieder.

 var mockSelecter = new Mock<ISelecter>{ CallBase = true };
 mockSelecter.SetUp(s=>s.GetTyreSpecification(It.IsAny<long>())).Returns(tyre);