InjiTest::testArrayWithCallback() - Code Metrics - Inspection of "Unused Code remove" - injitools/cms-Inji - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( edbfbd...89a048 )

by Alexey

created 2016-12-06 06:12 UTC

InjiTest::testArrayWithCallback() A

↳ Parent: InjiTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	5
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	4
nc	1
nop	0
dl	0
loc	5
rs	9.4285
c	0
b	0
f	0

<?php

class InjiTest extends PHPUnit_Framework_TestCase {

    /**
     * @covers Inji::listen
     * @covers Inji::event
     * @covers Inji::unlisten
     */
    public function testNotSavingEvent() {
        \Inji::$inst->listen('testEvent', 'testCallback', function() {
            return true;
        });
        $this->assertEquals(true, Inji::$inst->event('testEvent'));
        \Inji::$inst->unlisten('testEvent', 'testCallback');
    }

    /**
     * @covers Inji::listen
     * @covers Inji::event
     * @covers Inji::unlisten
     */
    public function testSavingEvent() {
        \Inji::$inst->listen('testEvent2', 'testCallback', 'InjiTestcallback', true);
function acceptsInteger($int) { }

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

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertEquals(true, Inji::$inst->event('testEvent2'));
        \Inji::$inst->unlisten('testEvent2', 'testCallback', true);
    }

    /**
     * @covers Inji::listen
     * @covers Inji::event
     */
    public function testArrayWithCallback() {
        $callbackOptions = ['callback' => 'InjiTestcallbackArray', 'data' => 'data'];
        \Inji::$inst->listen('testEvent3', 'testCallback', $callbackOptions);
        $this->assertEquals($callbackOptions, Inji::$inst->event('testEvent3')['callbackOptions']);
    }
}

function InjiTestcallback() {
    return true;
}

function InjiTestcallbackArray($event, $callbackOptions) {
    return ['event' => $event, 'callbackOptions' => $callbackOptions];
}


1			<?php
2
3			class InjiTest extends PHPUnit_Framework_TestCase {
4
5			/**
6			* @covers Inji::listen
7			* @covers Inji::event
8			* @covers Inji::unlisten
9			*/
10			public function testNotSavingEvent() {
11			\Inji::$inst->listen('testEvent', 'testCallback', function() {
12			return true;
13			});
14			$this->assertEquals(true, Inji::$inst->event('testEvent'));
15			\Inji::$inst->unlisten('testEvent', 'testCallback');
16			}
17
18			/**
19			* @covers Inji::listen
20			* @covers Inji::event
21			* @covers Inji::unlisten
22			*/
23			public function testSavingEvent() {
24			\Inji::$inst->listen('testEvent2', 'testCallback', 'InjiTestcallback', true);
			0 ignored issues – show Documentation introduced 2016-11-30 13:58 UTC by Report Bug Copy Issue Report `'InjiTestcallback'` is of type `string`, but the function expects a `array\|object<Closure>`. 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...
25			$this->assertEquals(true, Inji::$inst->event('testEvent2'));
26			\Inji::$inst->unlisten('testEvent2', 'testCallback', true);
27			}
28
29			/**
30			* @covers Inji::listen
31			* @covers Inji::event
32			*/
33			public function testArrayWithCallback() {
34			$callbackOptions = ['callback' => 'InjiTestcallbackArray', 'data' => 'data'];
35			\Inji::$inst->listen('testEvent3', 'testCallback', $callbackOptions);
36			$this->assertEquals($callbackOptions, Inji::$inst->event('testEvent3')['callbackOptions']);
37			}
38			}
39
40			function InjiTestcallback() {
41			return true;
42			}
43
44			function InjiTestcallbackArray($event, $callbackOptions) {
45			return ['event' => $event, 'callbackOptions' => $callbackOptions];
46			}
47

injitools / cms-Inji

Push — master ( edbfbd...89a048 )

InjiTest::testArrayWithCallback() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like