Skip to content

Commit ef1b7ce

Browse files
committed
increase code coverage in Psr18Client
1 parent dc8fe5a commit ef1b7ce

File tree

1 file changed

+13
-3
lines changed

1 file changed

+13
-3
lines changed

tests/Unit/Client/Psr18ClientTest.php

Lines changed: 13 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
use Psr\Http\Message\RequestInterface;
1111
use Psr\Http\Message\ResponseInterface;
1212
use Psr\Http\Message\ServerRequestFactoryInterface;
13+
use Psr\Http\Message\ServerRequestInterface;
1314
use Psr\Http\Message\StreamFactoryInterface;
1415
use Psr\Http\Message\StreamInterface;
1516
use Redmine\Client\Client;
@@ -40,20 +41,29 @@ public function shouldPassApiKeyToConstructor()
4041

4142
/**
4243
* @covers \Redmine\Client\Psr18Client
43-
* @test
4444
*/
45-
public function acceptServerRequestFactoryInConstructorForBC()
45+
public function testServerRequestFactoryIsAcceptedInConstructorForBC()
4646
{
4747
$client = new Psr18Client(
4848
$this->createMock(ClientInterface::class),
49-
$this->createMock(ServerRequestFactoryInterface::class),
49+
$this->createConfiguredMock(ServerRequestFactoryInterface::class, [
50+
'createServerRequest' => (function () {
51+
$request = $this->createMock(ServerRequestInterface::class);
52+
$request->method('withHeader')->willReturn($request);
53+
$request->method('withBody')->willReturn($request);
54+
55+
return $request;
56+
})(),
57+
]),
5058
$this->createMock(StreamFactoryInterface::class),
5159
'http://test.local',
5260
'access_token'
5361
);
5462

5563
$this->assertInstanceOf(Psr18Client::class, $client);
5664
$this->assertInstanceOf(Client::class, $client);
65+
66+
$client->requestGet('/path.xml');
5767
}
5868

5969
/**

0 commit comments

Comments
 (0)