Partial Mocking als Code riechen?

Question

Warum ist so viel Hass über "teilweise Spott" und den Code, der es erfordert?

Hier ist ein (theoretisches) Beispiel-Implementierung:

public ComplexResult1 operationA(Stimulus a) { 
    { 
     ... 
     result = ...; 
    } 
    auditTheChange(a); 
} 
public ComplexResult2 operationB(Stimulus b) { 
    { 
     ... 
     result = ...; 
    } 
    auditTheChange(b); 
    return result; 
} 
void auditTheChange(Stimulus stim) { 
    // do a bunch of stuff to record the change 
    // and interact with another outside service 
} 

Nun, in meinem Verständnis dieses gut Überarbeitete Code.

Wenn ich Unit-Test operationA will und operationB, und sicherzustellen, dass die Überwachung in jedem Szenario geschieht, aber ohne die Besonderheiten des Prüfungs Code zu testen ist, würde ich teilweise spöttisch verwenden.

Was sehe ich nicht, was dazu führt, dass so viele Projekte (EasyMock, Mockito usw.) Refactoring empfehlen?

Answer 1

Wenn Auditing wirklich eine interne Funktion der Klasse ist, sollte der Code als Teil des Komponententests getestet werden. Warum behandelt Ihre Klasse sowohl komplexe Vorgänge als auch Auditing? Könnte das Auditing in eine separate Klasse verschoben werden?

Wenn dies der Fall ist, führen Sie das Auditing als Kollaborateur mit dieser Klasse ein und mock-out. Wenn nicht, testen Sie es.

Sie können teilweise Mocks verwenden, aber in diesem Fall denke ich, es ist ein Hinweis darauf, dass die Klasse zu viel tut.