EventTest::testAcknowledge() - 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 19:43 UTC

EventTest::testAcknowledge() A

↳ Parent: EventTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	18
Code Lines	12

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	2

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

<?php

namespace Equip\Queue;

use Exception;
use League\Event\EmitterInterface;

class EventTest extends TestCase
{
    /**
     * @var EmitterInterface
     */
    private $emitter;

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

    protected function setUp()
    {
        $this->emitter = $this->createMock(EmitterInterface::class);

        $this->event = new Event($this->emitter);
    }

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

        $this->emitter

            ->expects($this->exactly(2))
            ->method('emit')
            ->withConsecutive(
                [Event::QUEUE_ACKNOWLEDGE, $message],
                [sprintf('%s.%s', Event::QUEUE_ACKNOWLEDGE, $message->handler()), $message]
            );

        $this->event->acknowledge($message);
    }

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

        $exception = new Exception;

        $this->emitter

            ->expects($this->exactly(2))
            ->method('emit')
            ->withConsecutive(
                [Event::QUEUE_REJECT, $message, $exception],
                [sprintf('%s.%s', Event::QUEUE_REJECT, $message->handler()), $message, $exception]
            );

        $this->event->reject($message, $exception);
    }
}


1			<?php
2
3			namespace Equip\Queue;
4
5			use Exception;
6			use League\Event\EmitterInterface;
7
8			class EventTest extends TestCase
9			{
10			/**
11			* @var EmitterInterface
12			*/
13			private $emitter;
14
15			/**
16			* @var Event
17			*/
18			private $event;
19
20			protected function setUp()
21			{
22			$this->emitter = $this->createMock(EmitterInterface::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(\Leagu...mitterInterface::class)` of type `object<PHPUnit_Framework_MockObject_MockObject>` is incompatible with the declared type `object<League\Event\EmitterInterface>` of property `$emitter`. 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...
23			$this->event = new Event($this->emitter);
24			}
25
26			public function testAcknowledge()
27			{
28			$message = new Message(
29			'queue',
30			'handler',
31			['foo' => 'bar']
32			);
33
34			$this->emitter
			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<League\Event\EmitterInterface>`. 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...
35			->expects($this->exactly(2))
36			->method('emit')
37			->withConsecutive(
38			[Event::QUEUE_ACKNOWLEDGE, $message],
39			[sprintf('%s.%s', Event::QUEUE_ACKNOWLEDGE, $message->handler()), $message]
40			);
41
42			$this->event->acknowledge($message);
43			}
44
45			public function testReject()
46			{
47			$message = new Message(
48			'queue',
49			'handler',
50			['foo' => 'bar']
51			);
52
53			$exception = new Exception;
54
55			$this->emitter
			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<League\Event\EmitterInterface>`. 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...
56			->expects($this->exactly(2))
57			->method('emit')
58			->withConsecutive(
59			[Event::QUEUE_REJECT, $message, $exception],
60			[sprintf('%s.%s', Event::QUEUE_REJECT, $message->handler()), $message, $exception]
61			);
62
63			$this->event->reject($message, $exception);
64			}
65			}
66

equip / queue

Pull Request — master (#1)

EventTest::testAcknowledge() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like