Test, ob Methode in ClassA von einer anderen Methode in ClassA aufgerufen wurde

Question

Es kann getestet werden, ob eine Methode mit Moq and dependency injection aufgerufen wurde. Ist es jedoch möglich, zu testen, ob eine Methode in einer Klasse eine andere in derselben Klasse aufruft?

Zum Beispiel möchte ich testen, dass, wenn ich eine bestimmte Ausnahme protokollieren, dass eine Informationsnachricht auch protokolliert wird.

Die Methode ist:

public void Error(string message, Exception exception, long logId = 0) 
{ 
    var int32 = (int)logId; 
    Info("Id was converted to an int so that it would fit in the log: " + logId, int32); 
    Error(message, exception, int32); 
} 

Dies war bei Einheit mein Versuch, es zu testen. Der Test schlägt fehl, gibt es einen Weg, wie es gemacht werden kann?

void logging_an_error_with_a_long_id_also_logs_info() 
{ 
    var mock = new Mock<ILogger>(); 
    var testedClass = new Logger(); 
    var counter = 0; 

    testedClass.Error("test" + counter++, new Exception("test" + counter), Int64.MaxValue); 

    mock.Verify(m => m.Info(It.IsAny<string>(), It.IsAny<int>())); 
} 

Da die Info und Error Methoden in der gleichen Klasse (Klasse A) sind, ich glaube nicht, dass ich KlasseA als Abhängigkeit in KlasseA passieren kann. Muss es nicht getestet werden?

Answer 1

Das Beste, was Sie tun können, ist Infovirtual. Auf diese Weise können Sie eine Mock<Logger> erstellen, CallBase = true festlegen und überprüfen, dass Info aufgerufen wurde.

var mock = new Mock<Logger> 
{ 
    CallBase = true 
}; 

mock.Object.Error("test" + counter++, new Exception("test" + counter), Int64.MaxValue); 

mock.Verify(m => m.Info(It.IsAny<string>(), It.IsAny<int>())); 

Auf diese Weise sind Sie immer noch die tatsächliche Umsetzung von Error Aufruf, aber Sie haben verwendet Moq die Info Methode zur Überprüfung aufgerufen wurde.

Answer 2

Es fühlt sich an, als ob Sie versuchen, das Falsche zu testen. Es ist nicht wirklich wichtig, dass die Info-Methode für Ihre Klasse von der Error-Methode aufgerufen wird. Wichtig ist, dass das Verhalten der Info-Methode auftritt. Wie es passiert, ist ein Implementierungsdetail der Klasse.

Wenn ich ein Mathe-Klasse mit zwei Funktionen hatte:

public int Mult(int x, int y) { 
    return x*y; 
} 

public int Sqr(int x) { 
    return Mult(x,y); 
} 

würde ich nicht auf die Mult Funktion rief Aufruf Sqr testen, würde ich Sqr(4)==16 testen. Es spielt keine Rolle, ob diese Berechnung in der Sqr-Methode oder in einer anderen Methode der Klasse stattfindet.

Während @ Andrew's solution ist wahrscheinlich, was Sie wollen, spöttelt die Klasse, die Sie testen, zu eng gekoppelten, brüchigen Tests führen.

Wenn es unpraktisch ist, den Aufruf zu testen, indem man seine Nebenwirkungen beobachtet, dann kann es ein Zeichen sein, dass die Implementierung ein bisschen Refactoring verwenden könnte.