GenericEventListenerCollectionTest::testCanConstruct() - Code Metrics - onoi/event-dispatcher - Measure and Improve Code Quality continuously with Scrutinizer

testCanConstruct() A
last analyzed 2019-01-27 01:52 UTC

↳ Parent: GenericEventListenerCollectionTest

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	12
rs	9.8666
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php

namespace Onoi\EventDispatcher\Tests\Listener;

use Onoi\EventDispatcher\Listener\GenericEventListenerCollection;
use Onoi\EventDispatcher\Listener\GenericCallbackEventListener;

/**
 * @covers \Onoi\EventDispatcher\Listener\GenericEventListenerCollection
 *
 * @group onoi-event-dispatcher
 *
 * @license GNU GPL v2+
 * @since 1.0
 *
 * @author mwjames
 */
class GenericEventListenerCollectionTest extends \PHPUnit_Framework_TestCase {

	public function testCanConstruct() {

		$this->assertInstanceOf(
			'\Onoi\EventDispatcher\EventListenerCollection',
			new GenericEventListenerCollection()
		);

		$this->assertInstanceOf(
			'\Onoi\EventDispatcher\Listener\GenericEventListenerCollection',
			new GenericEventListenerCollection()
		);
	}

	public function testRegisterListener() {

		$eventListener = $this->getMockBuilder( '\Onoi\EventDispatcher\EventListener' )
			->disableOriginalConstructor()
			->getMock();

		$instance = new GenericEventListenerCollection();
		$instance->registerListener( 'FOO', $eventListener );

		$expected = array(
			'foo' => array( $eventListener )
		);

		$this->assertEquals(
			$expected,
			$instance->getCollection()
		);
	}

	public function testTryRegisterListenerUsingInvalidEventIdentifierThrowsException() {

		$eventListener = $this->getMockBuilder( '\Onoi\EventDispatcher\EventListener' )
			->disableOriginalConstructor()
			->getMock();

		$instance = new GenericEventListenerCollection();

		$this->setExpectedException( 'InvalidArgumentException' );
		$instance->registerListener( new \stdClass, $eventListener );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
	}

	public function testRegisterCallback() {

		$callback = function() { return 'doSomething'; };

		$instance = new GenericEventListenerCollection();
		$instance->registerCallback( 'fOo', $callback );

		$expected = array(
			'foo' => array( new GenericCallbackEventListener( $callback ) )
		);

		$this->assertEquals(
			$expected,
			$instance->getCollection()
		);
	}

	public function testTryRegisterCallbackUsingInvalidEventIdentifierThrowsException() {

		$callback = function() { return 'doSomething'; };

		$instance = new GenericEventListenerCollection();

		$this->setExpectedException( 'InvalidArgumentException' );
		$instance->registerCallback( new \stdClass, $callback );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
	}

	public function testTryRegisterCallbackUsingInvalidCallbackThrowsException() {

		$eventListener = $this->getMockBuilder( '\Onoi\EventDispatcher\EventListener' )
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
			->disableOriginalConstructor()
			->getMock();

		$instance = new GenericEventListenerCollection();

		$this->setExpectedException( 'RuntimeException' );
		$instance->registerCallback( 'foo', new \stdClass );
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
	}

}


1			<?php
2
3			namespace Onoi\EventDispatcher\Tests\Listener;
4
5			use Onoi\EventDispatcher\Listener\GenericEventListenerCollection;
6			use Onoi\EventDispatcher\Listener\GenericCallbackEventListener;
7
8			/**
9			* @covers \Onoi\EventDispatcher\Listener\GenericEventListenerCollection
10			*
11			* @group onoi-event-dispatcher
12			*
13			* @license GNU GPL v2+
14			* @since 1.0
15			*
16			* @author mwjames
17			*/
18			class GenericEventListenerCollectionTest extends \PHPUnit_Framework_TestCase {
19
20			public function testCanConstruct() {
21
22			$this->assertInstanceOf(
23			'\Onoi\EventDispatcher\EventListenerCollection',
24			new GenericEventListenerCollection()
25			);
26
27			$this->assertInstanceOf(
28			'\Onoi\EventDispatcher\Listener\GenericEventListenerCollection',
29			new GenericEventListenerCollection()
30			);
31			}
32
33			public function testRegisterListener() {
34
35			$eventListener = $this->getMockBuilder( '\Onoi\EventDispatcher\EventListener' )
36			->disableOriginalConstructor()
37			->getMock();
38
39			$instance = new GenericEventListenerCollection();
40			$instance->registerListener( 'FOO', $eventListener );
41
42			$expected = array(
43			'foo' => array( $eventListener )
44			);
45
46			$this->assertEquals(
47			$expected,
48			$instance->getCollection()
49			);
50			}
51
52			public function testTryRegisterListenerUsingInvalidEventIdentifierThrowsException() {
53
54			$eventListener = $this->getMockBuilder( '\Onoi\EventDispatcher\EventListener' )
55			->disableOriginalConstructor()
56			->getMock();
57
58			$instance = new GenericEventListenerCollection();
59
60			$this->setExpectedException( 'InvalidArgumentException' );
61			$instance->registerListener( new \stdClass, $eventListener );
			0 ignored issues – show Documentation introduced 2018-01-15 22:50 UTC by Report Bug Copy Issue Report `new \stdClass()` is of type `object<stdClass>`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
62			}
63
64			public function testRegisterCallback() {
65
66			$callback = function() { return 'doSomething'; };
67
68			$instance = new GenericEventListenerCollection();
69			$instance->registerCallback( 'fOo', $callback );
70
71			$expected = array(
72			'foo' => array( new GenericCallbackEventListener( $callback ) )
73			);
74
75			$this->assertEquals(
76			$expected,
77			$instance->getCollection()
78			);
79			}
80
81			public function testTryRegisterCallbackUsingInvalidEventIdentifierThrowsException() {
82
83			$callback = function() { return 'doSomething'; };
84
85			$instance = new GenericEventListenerCollection();
86
87			$this->setExpectedException( 'InvalidArgumentException' );
88			$instance->registerCallback( new \stdClass, $callback );
			0 ignored issues – show Documentation introduced 2018-01-15 22:50 UTC by Report Bug Copy Issue Report `new \stdClass()` is of type `object<stdClass>`, but the function expects a `string`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
89			}
90
91			public function testTryRegisterCallbackUsingInvalidCallbackThrowsException() {
92
93			$eventListener = $this->getMockBuilder( '\Onoi\EventDispatcher\EventListener' )
			0 ignored issues – show Unused Code introduced 2018-01-15 22:50 UTC by Report Bug Copy Issue Report `$eventListener` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
94			->disableOriginalConstructor()
95			->getMock();
96
97			$instance = new GenericEventListenerCollection();
98
99			$this->setExpectedException( 'RuntimeException' );
100			$instance->registerCallback( 'foo', new \stdClass );
			0 ignored issues – show Documentation introduced 2018-01-15 22:50 UTC by Report Bug Copy Issue Report `new \stdClass()` is of type `object<stdClass>`, but the function expects a `callable`. It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { } $x = '123'; // string "123" // Instead of acceptsInteger($x); // we recommend to use acceptsInteger((integer) $x); Loading history...
101			}
102
103			}
104

onoi / event-dispatcher

testCanConstruct() A last analyzed 2019-01-27 01:52 UTC

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like

testCanConstruct() A
last analyzed 2019-01-27 01:52 UTC