ListCommandTest - Code Metrics - ktomk/n98-magerun2 - Measure and Improve Code Quality continuously with Scrutinizer

ListCommandTest A
last analyzed 2016-05-29 11:20 UTC

↳ Parent: Project

Complexity

Total Complexity

Size/Duplication

Total Lines	24
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

Metric	Value
wmc	1
c	0
b	0
f	0
lcom	0
cbo	3
dl	0
loc	24
rs	10

1 Method

Rating	Name	Duplication	Size	Complexity
A	testExecute()	0	21	1

<?php

namespace N98\Magento\Command\Script\Repository;

use Symfony\Component\Console\Tester\CommandTester;
use N98\Magento\Command\PHPUnit\TestCase;

class ListCommandTest extends TestCase
{
    public function testExecute()
    {
        $application = $this->getApplication();
        $config = $application->getConfig();
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
        $config['script']['folders'][] = __DIR__ . '/_scripts';
        $application->setConfig($config);
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

        $application->add(new RunCommand());
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}
        $command = $this->getApplication()->find('script:repo:list');
class A
{
    public function foo() { }
}

class B extends A
{
    public function bar() { }
}

/**
 * @param A|B $x
 */
function someFunction($x)
{
    $x->foo(); // This call is fine as the method exists in A and B.
    $x->bar(); // This method only exists in B and might cause an error.
}

        $commandTester = new CommandTester($command);
        $commandTester->execute(
            array(
                'command' => $command->getName(),
            )
        );

        $this->assertContains('Cache Flush Command Test (Hello World)', $commandTester->getDisplay());
        $this->assertContains('Foo command', $commandTester->getDisplay());
        $this->assertContains('Bar command', $commandTester->getDisplay());
    }
}


ListCommandTest A
last analyzed 2016-05-29 11:20 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

1			<?php
2
3			namespace N98\Magento\Command\Script\Repository;
4
5			use Symfony\Component\Console\Tester\CommandTester;
6			use N98\Magento\Command\PHPUnit\TestCase;
7
8			class ListCommandTest extends TestCase
9			{
10			public function testExecute()
11			{
12			$application = $this->getApplication();
13			$config = $application->getConfig();
			0 ignored issues – show Bug introduced 2016-01-03 16:06 UTC by Report Bug Copy Issue Report The method `getConfig` does only exist in `N98\Magento\Application`, but not in `PHPUnit_Framework_MockObject_MockObject`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
14			$config['script']['folders'][] = __DIR__ . '/_scripts';
15			$application->setConfig($config);
			0 ignored issues – show Bug introduced 2016-01-03 16:06 UTC by Report Bug Copy Issue Report The method `setConfig` does only exist in `N98\Magento\Application`, but not in `PHPUnit_Framework_MockObject_MockObject`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
16
17			$application->add(new RunCommand());
			0 ignored issues – show Bug introduced 2016-01-03 16:06 UTC by Report Bug Copy Issue Report The method `add` does only exist in `N98\Magento\Application`, but not in `PHPUnit_Framework_MockObject_MockObject`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
18			$command = $this->getApplication()->find('script:repo:list');
			0 ignored issues – show Bug introduced 2016-01-03 16:06 UTC by Report Bug Copy Issue Report The method `find` does only exist in `N98\Magento\Application`, but not in `PHPUnit_Framework_MockObject_MockObject`. It seems like the method you are trying to call exists only in some of the possible types. Let’s take a look at an example: class A { public function foo() { } } class B extends A { public function bar() { } } /** * @param A\|B $x / function someFunction($x) { $x->foo(); // This call is fine as the method exists in A and B. $x->bar(); // This method only exists in B and might cause an error. } Available Fixes Add an additional type-check: /* * @param A\|B $x / function someFunction($x) { $x->foo(); if ($x instanceof B) { $x->bar(); } } Only allow a single type to be passed if the variable comes from a parameter: function someFunction(B $x) { /* ... */ } Loading history...
19
20			$commandTester = new CommandTester($command);
21			$commandTester->execute(
22			array(
23			'command' => $command->getName(),
24			)
25			);
26
27			$this->assertContains('Cache Flush Command Test (Hello World)', $commandTester->getDisplay());
28			$this->assertContains('Foo command', $commandTester->getDisplay());
29			$this->assertContains('Bar command', $commandTester->getDisplay());
30			}
31			}
32

ktomk / n98-magerun2

ListCommandTest A last analyzed 2016-05-29 11:20 UTC

Complexity

Size/Duplication

Coupling/Cohesion

Importance

1 Method

Available Fixes

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like

ListCommandTest A
last analyzed 2016-05-29 11:20 UTC