Skip to content

Commit 92fd79e

Browse files
author
Michał Kopacz
committed
tests from moddleware has mocked factory
1 parent e086e68 commit 92fd79e

File tree

2 files changed

+20
-8
lines changed

2 files changed

+20
-8
lines changed

src/RequestIdProviderFactory.php

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,9 @@
55
use PhpMiddleware\RequestId\Generator\GeneratorInterface;
66
use Psr\Http\Message\ServerRequestInterface;
77

8+
/**
9+
* @codeCoverageIgnore
10+
*/
811
class RequestIdProviderFactory implements RequestIdProviderFactoryInterface
912
{
1013

test/RequestIdMiddlewareTest.php

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -2,11 +2,10 @@
22

33
namespace PhpMiddlewareTestTest\RequestId;
44

5-
use PhpMiddleware\RequestId\Exception\InvalidRequestId;
65
use PhpMiddleware\RequestId\Exception\MissingRequestId;
7-
use PhpMiddleware\RequestId\Generator\GeneratorInterface;
8-
use PhpMiddleware\RequestId\OverridePolicy\OverridePolicyInterface;
96
use PhpMiddleware\RequestId\RequestIdMiddleware;
7+
use PhpMiddleware\RequestId\RequestIdProviderFactoryInterface;
8+
use PhpMiddleware\RequestId\RequestIdProviderInterface;
109
use PHPUnit_Framework_TestCase;
1110
use Psr\Http\Message\ServerRequestInterface;
1211
use Zend\Diactoros\Response;
@@ -16,9 +15,13 @@ class RequestIdMiddlewareTest extends PHPUnit_Framework_TestCase
1615
{
1716
public function testEmmitRequestIdToResponse()
1817
{
19-
$this->generator->expects($this->once())->method('generateRequestId')->willReturn('123456789');
18+
$requestIdProviderFactory = $this->getMock(RequestIdProviderFactoryInterface::class);
19+
$requestIdProvider = $this->getMock(RequestIdProviderInterface::class);
2020

21-
$middleware = new RequestIdMiddleware($this->generator);
21+
$requestIdProviderFactory->method('create')->willReturn($requestIdProvider);
22+
$requestIdProvider->method('getRequestId')->willReturn('123456789');
23+
24+
$middleware = new RequestIdMiddleware($requestIdProviderFactory);
2225
$request = new ServerRequest();
2326
$response = new Response();
2427
$calledOut = false;
@@ -41,9 +44,13 @@ public function testEmmitRequestIdToResponse()
4144

4245
public function testNotEmmitRequestIdToResponse()
4346
{
44-
$this->generator->expects($this->once())->method('generateRequestId')->willReturn('123456789');
47+
$requestIdProviderFactory = $this->getMock(RequestIdProviderFactoryInterface::class);
48+
$requestIdProvider = $this->getMock(RequestIdProviderInterface::class);
49+
50+
$requestIdProviderFactory->method('create')->willReturn($requestIdProvider);
51+
$requestIdProvider->method('getRequestId')->willReturn('123456789');
4552

46-
$middleware = new RequestIdMiddleware($this->generator, true, null);
53+
$middleware = new RequestIdMiddleware($requestIdProviderFactory, null);
4754
$request = new ServerRequest();
4855
$response = new Response();
4956
$calledOut = false;
@@ -68,7 +75,9 @@ public function testTryToGetRequestIdBeforeRunMiddleware()
6875
{
6976
$this->setExpectedException(MissingRequestId::class);
7077

71-
$middleware = new RequestIdMiddleware($this->generator);
78+
$requestIdProviderFactory = $this->getMock(RequestIdProviderFactoryInterface::class);
79+
80+
$middleware = new RequestIdMiddleware($requestIdProviderFactory);
7281
$middleware->getRequestId();
7382
}
7483
}

0 commit comments

Comments
 (0)