test: optimize test

Reasno · Reasno · commit 0683b2dbf65b · 2020-04-09T23:19:32.000+08:00
diff --git a/tests/Cases/GoTaskProxyTest.php b/tests/Cases/GoTaskProxyTest.php
@@ -27,11 +27,10 @@ class GoTaskProxyTest extends TestCase
     public function testCall()
     {
         $goTask = Mockery::mock(GoTask::class);
-        $goTask->shouldReceive('call')->once()->withArgs(['a', 'b', 0]);
-        $goTask->shouldReceive('call')->once()->withArgs(['GoProxyStub.Test', 'mmm', 1]);
+        $goTask->shouldReceive('call')->once()->withArgs(['a', 'b', 0])->andReturn('ok');
+        $goTask->shouldReceive('call')->once()->withArgs(['GoProxyStub.Test', 'mmm', 1])->andReturn('ok');
         $proxy = new GoProxyStub($goTask);
-        $proxy->call('a', 'b', 0);
-        $proxy->test('mmm', 1);
-        $this->assertFalse(false);
+        $this->assertEquals('ok', $proxy->call('a', 'b', 0));
+        $this->assertEquals('ok', $proxy->test('mmm', 1));
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -27,11 +27,10 @@ class GoTaskProxyTest extends TestCase`
`27`	`27`	`public function testCall()`
`28`	`28`	`{`
`29`	`29`	`$goTask = Mockery::mock(GoTask::class);`
`30`		`- $goTask->shouldReceive('call')->once()->withArgs(['a', 'b', 0]);`
`31`		`- $goTask->shouldReceive('call')->once()->withArgs(['GoProxyStub.Test', 'mmm', 1]);`
	`30`	`+ $goTask->shouldReceive('call')->once()->withArgs(['a', 'b', 0])->andReturn('ok');`
	`31`	`+ $goTask->shouldReceive('call')->once()->withArgs(['GoProxyStub.Test', 'mmm', 1])->andReturn('ok');`
`32`	`32`	`$proxy = new GoProxyStub($goTask);`
`33`		`- $proxy->call('a', 'b', 0);`
`34`		`- $proxy->test('mmm', 1);`
`35`		`- $this->assertFalse(false);`
	`33`	`+ $this->assertEquals('ok', $proxy->call('a', 'b', 0));`
	`34`	`+ $this->assertEquals('ok', $proxy->test('mmm', 1));`
`36`	`35`	`}`
`37`	`36`	`}`