ConfigShowCommandTest::testIncorrectParameter() - Code Metrics - Inspection of "CAMN-10" - SierraTecnologia/fabrica - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a5a038...08ae9a )

by Ricardo

created 2021-02-26 20:52 UTC

ConfigShowCommandTest::testIncorrectParameter() A

↳ Parent: ConfigShowCommandTest

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	6
rs	10
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php

/**
 * This file is part of Fabrica.
 *
 * (c) Alexandre Salomé <[email protected]>
 * (c) Julien DIDIER <[email protected]>
 *
 * This source file is subject to the GPL license that is bundled
 * with this source code in the file LICENSE.
 */

namespace Fabrica\Bundle\CoreBundle\Tests\Command;

use Fabrica\Bundle\CoreBundle\Test\CommandTestCase;

class ConfigShowCommandTest extends CommandTestCase
{
    protected $client;

    protected function setUp(): void
    {
        $this->client = self::createClient();
        $this->client->startIsolation();
    }

    public function tearDown(): void
    {
        $this->client->stopIsolation();
    }

    public function testNoParameter()
    {
        list($statusCode ,$output) = $this->runCommand($this->client, "fabrica:config:show");
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;

        $this->assertContains('repository_path  '.$this->getRepositoryPath(), $output);
        $this->assertRegexp('/\n$/', $output, 'Ends with an empty line');
    }

    public function testCorrectParameter()
    {
        list($statusCode ,$output) = $this->runCommand($this->client, "fabrica:config:show repository_path");
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;

        $this->assertEquals($this->getRepositoryPath()."\n", $output);
    }

    public function testIncorrectParameter()
    {
        list($statusCode ,$output) = $this->runCommand($this->client, "fabrica:config:show foobarbaz");
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;

        $this->assertNotEquals(0, $statusCode);
    }

    private function getRepositoryPath()
    {
        return self::createClient()->getKernel()->getContainer()->getParameter('repository_path');
    }
}


1			<?php
2
3			/**
4			* This file is part of Fabrica.
5			*
6			* (c) Alexandre Salomé <[email protected]>
7			* (c) Julien DIDIER <[email protected]>
8			*
9			* This source file is subject to the GPL license that is bundled
10			* with this source code in the file LICENSE.
11			*/
12
13			namespace Fabrica\Bundle\CoreBundle\Tests\Command;
14
15			use Fabrica\Bundle\CoreBundle\Test\CommandTestCase;
16
17			class ConfigShowCommandTest extends CommandTestCase
18			{
19			protected $client;
20
21			protected function setUp(): void
22			{
23			$this->client = self::createClient();
24			$this->client->startIsolation();
25			}
26
27			public function tearDown(): void
28			{
29			$this->client->stopIsolation();
30			}
31
32			public function testNoParameter()
33			{
34			list($statusCode ,$output) = $this->runCommand($this->client, "fabrica:config:show");
			0 ignored issues – show Unused Code introduced 2020-07-25 16:42 UTC by Report Bug Copy Issue Report The assignment to `$statusCode` is unused. Consider omitting it like so `list($first,,$third)`. This checks looks for assignemnts to variables using the `list(...)` function, where not all assigned variables are subsequently used. Consider the following code example. <?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c; Only the variables `$a` and `$c` are used. There was no need to assign `$b`. Instead, the list call could have been. list($a,, $c) = returnThreeValues(); Loading history...
35
36			$this->assertContains('repository_path '.$this->getRepositoryPath(), $output);
37			$this->assertRegexp('/\n$/', $output, 'Ends with an empty line');
38			}
39
40			public function testCorrectParameter()
41			{
42			list($statusCode ,$output) = $this->runCommand($this->client, "fabrica:config:show repository_path");
			0 ignored issues – show Unused Code introduced 2020-07-25 16:42 UTC by Report Bug Copy Issue Report The assignment to `$statusCode` is unused. Consider omitting it like so `list($first,,$third)`. This checks looks for assignemnts to variables using the `list(...)` function, where not all assigned variables are subsequently used. Consider the following code example. <?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c; Only the variables `$a` and `$c` are used. There was no need to assign `$b`. Instead, the list call could have been. list($a,, $c) = returnThreeValues(); Loading history...
43
44			$this->assertEquals($this->getRepositoryPath()."\n", $output);
45			}
46
47			public function testIncorrectParameter()
48			{
49			list($statusCode ,$output) = $this->runCommand($this->client, "fabrica:config:show foobarbaz");
			0 ignored issues – show Unused Code introduced 2020-07-25 16:42 UTC by Report Bug Copy Issue Report The assignment to `$output` is unused. Consider omitting it like so `list($first,,$third)`. This checks looks for assignemnts to variables using the `list(...)` function, where not all assigned variables are subsequently used. Consider the following code example. <?php function returnThreeValues() { return array('a', 'b', 'c'); } list($a, $b, $c) = returnThreeValues(); print $a . " - " . $c; Only the variables `$a` and `$c` are used. There was no need to assign `$b`. Instead, the list call could have been. list($a,, $c) = returnThreeValues(); Loading history...
50
51			$this->assertNotEquals(0, $statusCode);
52			}
53
54			private function getRepositoryPath()
55			{
56			return self::createClient()->getKernel()->getContainer()->getParameter('repository_path');
57			}
58			}
59

SierraTecnologia / fabrica

Push — master ( a5a038...08ae9a )

ConfigShowCommandTest::testIncorrectParameter() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like