WorkerTest::testTickHandlerReturnFalse() - Code Metrics - Inspection of "Add initial library" - equip/queue - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#1)

by Adam

created 2016-07-06 20:42 UTC

WorkerTest::testTickHandlerReturnFalse() B

↳ Parent: WorkerTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	25
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	2

Metric	Value
c	2
b	0
f	2
dl	0
loc	25
rs	8.8571
cc	1
eloc	18
nc	1
nop	0

<?php

namespace Equip\Queue;

use Equip\Queue\Driver\DriverInterface;
use Equip\Queue\Exception\HandlerException;
use Equip\Queue\Serializer\JsonSerializer;
use Equip\Queue\Serializer\MessageSerializerInterface;
use Exception;

class WorkerTest extends TestCase
{
    /**
     * @var DriverInterface
     */
    private $driver;

    /**
     * @var Event
     */
    private $event;

    /**
     * @var MessageSerializerInterface
     */
    private $serializer;

    /**
     * @var Worker
     */
    private $worker;

    protected function setUp()
    {
        $this->driver = $this->createMock(DriverInterface::class);

        $this->event = $this->createMock(Event::class);

        $this->serializer = new JsonSerializer;
        $this->worker = new Worker($this->driver, $this->event);
    }

    public function testGetHandler()
    {
        $name = 'test';
        $routes = [
            $name => function () {},
        ];

        $method = static::getProtectedMethod($this->worker, 'getHandler');
        $result = $method->invoke($this->worker, $name, $routes);

        $this->assertSame($routes[$name], $result);
    }

    public function testGetHandlerNotCallable()
    {
        $this->expectException(HandlerException::class);
        $this->expectExceptionMessage('The handler for `test` is invalid.');

        $name = 'test';
        $routes = [
            $name => 'test',
        ];

        $method = static::getProtectedMethod($this->worker, 'getHandler');
        $method->invoke($this->worker, $name, $routes);
    }

    public function testGetHandlerNoHandler()
    {
        $method = static::getProtectedMethod($this->worker, 'getHandler');
        $this->assertNull($method->invoke($this->worker, 'test', []));
    }

    public function testTickPacketNull()
    {
        $queue = 'test-queue';
        $this->driver

            ->expects($this->once())
            ->method('pop')
            ->with($queue)
            ->willReturn(null);

        $method = static::getProtectedMethod($this->worker, 'tick');
        $this->assertTrue($method->invoke($this->worker, $queue));
    }

    public function testTickInvalidHandler()
    {
        $message = [
            'queue' => 'test-queue',
            'handler' => 'foo',
            'data' => ['foo' => 'bar'],
        ];

        $this->driver

            ->expects($this->once())
            ->method('pop')
            ->with($message['queue'])
            ->willReturn(json_encode($message));

        $method = static::getProtectedMethod($this->worker, 'tick');
        $this->assertTrue($method->invoke($this->worker, $message['queue']));
    }

    public function testTickHandlerException()
    {
        $message = new Message('queue', 'foo', ['foo' => 'bar']);
        $exception = new Exception;

        $this->driver

            ->expects($this->once())
            ->method('pop')
            ->with($message->queue())
            ->willReturn($this->serializer->serialize($message));

        $this->event

            ->expects($this->once())
            ->method('reject')
            ->with($message, $exception);

        $worker = new Worker(
            $this->driver,
            $this->event,
            $this->serializer,
            ['foo' => function () use ($exception) { throw $exception; }]
        );

        $method = static::getProtectedMethod($worker, 'tick');
        $this->assertTrue($method->invoke($worker, $message->queue()));
    }

    public function testTickHandlerReturnFalse()
    {
        $message = new Message('queue', 'foo', ['foo' => 'bar']);

        $this->driver

            ->expects($this->once())
            ->method('pop')
            ->with($message->queue())
            ->willReturn($this->serializer->serialize($message));

        $this->event

            ->expects($this->once())
            ->method('acknowledge')
            ->with($message);

        $worker = new Worker(
            $this->driver,
            $this->event,
            $this->serializer,
            ['foo' => function () { return false; }]
        );

        $method = static::getProtectedMethod($worker, 'tick');
        $this->assertFalse($method->invoke($worker, $message->queue()));
    }

    public function testTick()
    {
        $message = new Message('queue', 'foo', ['name' => 'foo']);

        $this->driver

            ->expects($this->once())
            ->method('pop')
            ->with($message->queue())
            ->willReturn($this->serializer->serialize($message));

        $this->event

            ->expects($this->once())
            ->method('acknowledge')
            ->with($message);

        $worker = new Worker(
            $this->driver,
            $this->event,
            $this->serializer,
            [
                'foo' => function ($data) use ($message) {
                    $this->assertSame($message->handler(), $data->handler());
                }
            ]
        );

        $method = static::getProtectedMethod($worker, 'tick');
        $this->assertTrue($method->invoke($worker, $message->queue()));
    }

    public function testConsume()
    {
        $message = new Message('queue', 'foo', ['name' => 'foo']);

        $this->driver

            ->expects($this->once())
            ->method('pop')
            ->with($message->queue())
            ->willReturn($this->serializer->serialize($message));

        $this->event

            ->expects($this->once())
            ->method('acknowledge')
            ->with($message);

        $worker = new Worker(
            $this->driver,
            $this->event,
            $this->serializer,
            ['foo' => function () { return false; }]
        );

        $worker->consume($message->queue());
    }
}


1			<?php
2
3			namespace Equip\Queue;
4
5			use Equip\Queue\Driver\DriverInterface;
6			use Equip\Queue\Exception\HandlerException;
7			use Equip\Queue\Serializer\JsonSerializer;
8			use Equip\Queue\Serializer\MessageSerializerInterface;
9			use Exception;
10
11			class WorkerTest extends TestCase
12			{
13			/**
14			* @var DriverInterface
15			*/
16			private $driver;
17
18			/**
19			* @var Event
20			*/
21			private $event;
22
23			/**
24			* @var MessageSerializerInterface
25			*/
26			private $serializer;
27
28			/**
29			* @var Worker
30			*/
31			private $worker;
32
33			protected function setUp()
34			{
35			$this->driver = $this->createMock(DriverInterface::class);
			0 ignored issues – show Documentation Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report It seems like `$this->createMock(\Equip...DriverInterface::class)` of type `object<PHPUnit_Framework_MockObject_MockObject>` is incompatible with the declared type `object<Equip\Queue\Driver\DriverInterface>` of property `$driver`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
36			$this->event = $this->createMock(Event::class);
			0 ignored issues – show Documentation Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report It seems like `$this->createMock(\Equip\Queue\Event::class)` of type `object<PHPUnit_Framework_MockObject_MockObject>` is incompatible with the declared type `object<Equip\Queue\Event>` of property `$event`. Our type inference engine has found an assignment to a property that is incompatible with the declared type of that property. Either this assignment is in error or the assigned type should be added to the documentation/type hint for that property.. Loading history...
37			$this->serializer = new JsonSerializer;
38			$this->worker = new Worker($this->driver, $this->event);
39			}
40
41			public function testGetHandler()
42			{
43			$name = 'test';
44			$routes = [
45			$name => function () {},
46			];
47
48			$method = static::getProtectedMethod($this->worker, 'getHandler');
49			$result = $method->invoke($this->worker, $name, $routes);
50
51			$this->assertSame($routes[$name], $result);
52			}
53
54			public function testGetHandlerNotCallable()
55			{
56			$this->expectException(HandlerException::class);
57			$this->expectExceptionMessage('The handler for `test` is invalid.');
58
59			$name = 'test';
60			$routes = [
61			$name => 'test',
62			];
63
64			$method = static::getProtectedMethod($this->worker, 'getHandler');
65			$method->invoke($this->worker, $name, $routes);
66			}
67
68			public function testGetHandlerNoHandler()
69			{
70			$method = static::getProtectedMethod($this->worker, 'getHandler');
71			$this->assertNull($method->invoke($this->worker, 'test', []));
72			}
73
74			public function testTickPacketNull()
75			{
76			$queue = 'test-queue';
77			$this->driver
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Driver\DriverInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
78			->expects($this->once())
79			->method('pop')
80			->with($queue)
81			->willReturn(null);
82
83			$method = static::getProtectedMethod($this->worker, 'tick');
84			$this->assertTrue($method->invoke($this->worker, $queue));
85			}
86
87			public function testTickInvalidHandler()
88			{
89			$message = [
90			'queue' => 'test-queue',
91			'handler' => 'foo',
92			'data' => ['foo' => 'bar'],
93			];
94
95			$this->driver
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Driver\DriverInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
96			->expects($this->once())
97			->method('pop')
98			->with($message['queue'])
99			->willReturn(json_encode($message));
100
101			$method = static::getProtectedMethod($this->worker, 'tick');
102			$this->assertTrue($method->invoke($this->worker, $message['queue']));
103			}
104
105			public function testTickHandlerException()
106			{
107			$message = new Message('queue', 'foo', ['foo' => 'bar']);
108			$exception = new Exception;
109
110			$this->driver
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Driver\DriverInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
111			->expects($this->once())
112			->method('pop')
113			->with($message->queue())
114			->willReturn($this->serializer->serialize($message));
115
116			$this->event
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Event>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
117			->expects($this->once())
118			->method('reject')
119			->with($message, $exception);
120
121			$worker = new Worker(
122			$this->driver,
123			$this->event,
124			$this->serializer,
125			['foo' => function () use ($exception) { throw $exception; }]
126			);
127
128			$method = static::getProtectedMethod($worker, 'tick');
129			$this->assertTrue($method->invoke($worker, $message->queue()));
130			}
131
132			public function testTickHandlerReturnFalse()
133			{
134			$message = new Message('queue', 'foo', ['foo' => 'bar']);
135
136			$this->driver
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Driver\DriverInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
137			->expects($this->once())
138			->method('pop')
139			->with($message->queue())
140			->willReturn($this->serializer->serialize($message));
141
142			$this->event
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Event>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
143			->expects($this->once())
144			->method('acknowledge')
145			->with($message);
146
147			$worker = new Worker(
148			$this->driver,
149			$this->event,
150			$this->serializer,
151			['foo' => function () { return false; }]
152			);
153
154			$method = static::getProtectedMethod($worker, 'tick');
155			$this->assertFalse($method->invoke($worker, $message->queue()));
156			}
157
158			public function testTick()
159			{
160			$message = new Message('queue', 'foo', ['name' => 'foo']);
161
162			$this->driver
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Driver\DriverInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
163			->expects($this->once())
164			->method('pop')
165			->with($message->queue())
166			->willReturn($this->serializer->serialize($message));
167
168			$this->event
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Event>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
169			->expects($this->once())
170			->method('acknowledge')
171			->with($message);
172
173			$worker = new Worker(
174			$this->driver,
175			$this->event,
176			$this->serializer,
177			[
178			'foo' => function ($data) use ($message) {
179			$this->assertSame($message->handler(), $data->handler());
180			}
181			]
182			);
183
184			$method = static::getProtectedMethod($worker, 'tick');
185			$this->assertTrue($method->invoke($worker, $message->queue()));
186			}
187
188			public function testConsume()
189			{
190			$message = new Message('queue', 'foo', ['name' => 'foo']);
191
192			$this->driver
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Driver\DriverInterface>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
193			->expects($this->once())
194			->method('pop')
195			->with($message->queue())
196			->willReturn($this->serializer->serialize($message));
197
198			$this->event
			0 ignored issues – show Bug introduced 2016-07-06 19:32 UTC by Report Bug Copy Issue Report The method `expects()` does not seem to exist on `object<Equip\Queue\Event>`. This check looks for calls to methods that do not seem to exist on a given type. It looks for the method on the type itself as well as in inherited classes or implemented interfaces. This is most likely a typographical error or the method has been renamed. Loading history...
199			->expects($this->once())
200			->method('acknowledge')
201			->with($message);
202
203			$worker = new Worker(
204			$this->driver,
205			$this->event,
206			$this->serializer,
207			['foo' => function () { return false; }]
208			);
209
210			$worker->consume($message->queue());
211			}
212			}
213

equip / queue

Pull Request — master (#1)

WorkerTest::testTickHandlerReturnFalse() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like