Sollte ich einen $ this> doSomething() Aufruf im Komponententest verspotten?

Question

Ich bin ziemlich neu in der Testwelt und versuche, mich mit Spott und Spott zu beschäftigen. Ich weiß, wie man bestimmte Klassen verspottet, die gerufen werden. Zum Glück macht Laravel das sehr einfach.

Im Laufe der Zeit habe ich gelernt, dass Sie Ihre Methoden z. B. in die Controller-Klassen in kleinere Stücke teilen. So macht jede dieser kleineren Methoden mehr oder weniger nur eine Sache und das ist, wie ich hörte, viel besser zum Testen.

Lässt meine Controller-Klasse das sieht aus wie nehmen:

class MyController extends Controller 
{ 
    public function myMethod($request) 
    { 
    $this->validateRequest($request); 
    $response = App\ModelClass::doSomething(); 
    return response()->json($response) 
    } 

    protected function validateRequest($request) 
    { 
    // do some validation here 
    } 
} 

Wie ich es verstehe, sind Unit-Tests verwendet, um die Methoden isoliert zu überprüfen und ich sollte den Code in so wenig Brocken wie möglich teilen Sie es zu machen einfacher, sie zu prüfen (so eine Methode macht nur eine Sache usw.)

Ist es eine gute Praxis zu testen, in diesem Beispiel myMethod isoliert und prüfen, ob es die anderen Methoden ruft? Ich weiß, das funktioniert nicht, aber kann ich es ähnlich ähnlich machen? Und sollte ich es so machen?

\Mockery::mock('App\Http\Controllers\MyController') 
    ->shouldReceive('validateRequest') 
    ->once(); 

Answer 1

Leider können Sie nicht öffentliche Methoden abfangen. Dies ist ein Teil eines Komponententests und sollte daher ordnungsgemäß ausgeführt werden. Wenn Sie innerhalb dieser Methode Teil einer Logik abfangen wollen Sie Refactoring es zu einer Serviceklasse denken sollte und dann kann man es so nennen:

public function myMethod($request) 
{ 
    app(RequestService::class)->validateRequest($request); 
    $response = App\ModelClass::doSomething(); 
    return response()->json($response) 
} 

dann können Sie verspotten es mögen:

App::shouldReceive('make->validateRequest') (...)