EventTest::testEventHandlerTraitMethods() - Code Metrics - Inspection of "oops! refactor back... seems its working now :)" - 2amigos/mailer-library - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( d068c5...99fa9d )

by Antonio

created 2015-12-29 23:35 UTC

EventTest::testEventHandlerTraitMethods() B

↳ Parent: EventTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	26
Code Lines	18

Duplication

Lines	0
Ratio	0 %

Metric	Value
dl	0
loc	26
rs	8.8571
cc	1
eloc	18
nc	1
nop	0

<?php
namespace Da\Mailer\Test\Event;

use Da\Mailer\Event\Event;
use Da\Mailer\Event\EventHandlerTrait;
use PHPUnit_Framework_TestCase;

class EventTest extends PHPUnit_Framework_TestCase
{
    /**
     * @expectedException \Da\Mailer\Exception\InvalidCallbackArgumentException
     */
    public function testInvalidCallbackArgumentException()
    {
        $event = new Event('not a callback');
$myVar = 'Value';
$higher = false;

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

    public function testEventHandlerTraitMethods()
    {
        $triggered = null;
        $data = null;
        $handler = function (Event $event) use (&$triggered, &$data) {
            $triggered = $event;
            $data = $event->getData();
        };
        $event = new Event($handler);
        $eventsTester = new EventsTester();

        $eventsTester->attach('eventName', $event);

        $eventsTester->trigger('eventName', ['x-wings']);

        $this->assertEquals($triggered, $event);
        $this->assertEquals($data, ['x-wings']);

        $triggered = null;
        $data = null;
        $eventsTester->detach('eventName');
        $eventsTester->trigger('eventName');

        $this->assertNull($triggered);
        $this->assertNull($data);
    }
}

class EventsTester
{
    use EventHandlerTrait;
}


1			<?php
2			namespace Da\Mailer\Test\Event;
3
4			use Da\Mailer\Event\Event;
5			use Da\Mailer\Event\EventHandlerTrait;
6			use PHPUnit_Framework_TestCase;
7
8			class EventTest extends PHPUnit_Framework_TestCase
9			{
10			/**
11			* @expectedException \Da\Mailer\Exception\InvalidCallbackArgumentException
12			*/
13			public function testInvalidCallbackArgumentException()
14			{
15			$event = new Event('not a callback');
			0 ignored issues – show Unused Code introduced 2015-12-29 23:38 UTC by Report Bug Copy Issue Report `$event` 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...
16			}
17
18			public function testEventHandlerTraitMethods()
19			{
20			$triggered = null;
21			$data = null;
22			$handler = function (Event $event) use (&$triggered, &$data) {
23			$triggered = $event;
24			$data = $event->getData();
25			};
26			$event = new Event($handler);
27			$eventsTester = new EventsTester();
28
29			$eventsTester->attach('eventName', $event);
30
31			$eventsTester->trigger('eventName', ['x-wings']);
32
33			$this->assertEquals($triggered, $event);
34			$this->assertEquals($data, ['x-wings']);
35
36			$triggered = null;
37			$data = null;
38			$eventsTester->detach('eventName');
39			$eventsTester->trigger('eventName');
40
41			$this->assertNull($triggered);
42			$this->assertNull($data);
43			}
44			}
45
46			class EventsTester
47			{
48			use EventHandlerTrait;
49			}
50

2amigos / mailer-library

Push — master ( d068c5...99fa9d )

EventTest::testEventHandlerTraitMethods() B

Complexity

Size

Duplication

Duplication Side-by-Side

Filter issues like