ExecTest::testInheritEnv() - Code Metrics - Inspection of "Add PHP 7.4 testing and dependency updates." - consolidation/robo - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#924)

by Greg

created 2020-02-11 01:23 UTC

ExecTest::testInheritEnv() A

↳ Parent: ExecTest

Complexity

Conditions	2
Paths	2

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	31
rs	9.424
c	0
b	0
f	0
cc	2
nc	2
nop	0

<?php
namespace Robo;

use PHPUnit\Framework\TestCase;
use Robo\Traits\TestTasksTrait;

class ExecTest extends TestCase
{
    use TestTasksTrait;
    use Task\Base\loadTasks;

    public function setup()
    {
        $this->initTestTasksTrait();
    }

    public function testExecLsCommand()
    {
        $command = strncasecmp(PHP_OS, 'WIN', 3) == 0 ? 'dir' : 'ls';
        $res = $this->taskExec($command)->interactive(false)->run();
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.
}
        $this->assertContains(
            'src',
            $res->getMessage());
        $this->assertContains(
            'codeception.yml',
            $res->getMessage());
    }

    public function testMultipleEnvVars()
    {
        $task = $this->taskExec('env')->interactive(false);
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.
}
        $task->env('FOO', 'BAR');
        $task->env('BAR', 'BAZ');
        $result = $task->run();
        // Verify that the text contains our environment variable.
        $this->assertContains(
            'FOO=BAR',
            $result->getMessage());
        $this->assertContains(
            'BAR=BAZ',
            $result->getMessage());

        // Now verify that we can reset a value that was previously set.
        $task = $this->taskExec('env')->interactive(false);
        $task->env('FOO', 'BAR');
        $task->env('FOO', 'BAZ');
        $result = $task->run();
        // Verify that the text contains the most recent environment variable.
        $this->assertContains(
            'FOO=BAZ',
            $result->getMessage());
    }

    public function testInheritEnv()
    {
        // Symfony < 3.2.1 does not inherit environment variables, so there's
        // nothing to test if the function doesn't exist.
        if (!method_exists('Symfony\Component\Process\Process', 'inheritEnvironmentVariables')) {
            throw new \PHPUnit_Framework_SkippedTestError(
                'Inheriting of environment variables is not supported.'
            );
        }
        // With no environment variables set, count how many environment
        // variables are present.
        $task = $this->taskExec('env | wc -l')->interactive(false);
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.
}
        $result = $task->run();
        $start_count = (int) $result->getMessage();
        $this->assertGreaterThan(0, $start_count);

        // Verify that we get the same amount of environment variables with
        // another exec call.
        $task = $this->taskExec('env | wc -l')->interactive(false);
        $result = $task->run();
        $this->assertEquals(
            $start_count,
            (int) $result->getMessage());

        // Now run the same command, but this time add another environment
        // variable, and see if our count increases by one.
        $task = $this->taskExec('env | wc -l')->interactive(false);
        $task->env('FOO', 'BAR');
        $result = $task->run();
        $this->assertEquals($start_count + 1, (int) $result->getMessage());
    }
}


1			<?php
2			namespace Robo;
3
4			use PHPUnit\Framework\TestCase;
5			use Robo\Traits\TestTasksTrait;
6
7			class ExecTest extends TestCase
8			{
9			use TestTasksTrait;
10			use Task\Base\loadTasks;
11
12			public function setup()
13			{
14			$this->initTestTasksTrait();
15			}
16
17			public function testExecLsCommand()
18			{
19			$command = strncasecmp(PHP_OS, 'WIN', 3) == 0 ? 'dir' : 'ls';
20			$res = $this->taskExec($command)->interactive(false)->run();
			0 ignored issues – show Bug introduced 2020-02-08 05:06 UTC by Report Bug Copy Issue Report The method `interactive` does only exist in `Robo\Task\Base\Exec`, but not in `Robo\Collection\CollectionBuilder`. 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...
21			$this->assertContains(
22			'src',
23			$res->getMessage());
24			$this->assertContains(
25			'codeception.yml',
26			$res->getMessage());
27			}
28
29			public function testMultipleEnvVars()
30			{
31			$task = $this->taskExec('env')->interactive(false);
			0 ignored issues – show Bug introduced 2020-02-08 05:06 UTC by Report Bug Copy Issue Report The method `interactive` does only exist in `Robo\Task\Base\Exec`, but not in `Robo\Collection\CollectionBuilder`. 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...
32			$task->env('FOO', 'BAR');
33			$task->env('BAR', 'BAZ');
34			$result = $task->run();
35			// Verify that the text contains our environment variable.
36			$this->assertContains(
37			'FOO=BAR',
38			$result->getMessage());
39			$this->assertContains(
40			'BAR=BAZ',
41			$result->getMessage());
42
43			// Now verify that we can reset a value that was previously set.
44			$task = $this->taskExec('env')->interactive(false);
45			$task->env('FOO', 'BAR');
46			$task->env('FOO', 'BAZ');
47			$result = $task->run();
48			// Verify that the text contains the most recent environment variable.
49			$this->assertContains(
50			'FOO=BAZ',
51			$result->getMessage());
52			}
53
54			public function testInheritEnv()
55			{
56			// Symfony < 3.2.1 does not inherit environment variables, so there's
57			// nothing to test if the function doesn't exist.
58			if (!method_exists('Symfony\Component\Process\Process', 'inheritEnvironmentVariables')) {
59			throw new \PHPUnit_Framework_SkippedTestError(
60			'Inheriting of environment variables is not supported.'
61			);
62			}
63			// With no environment variables set, count how many environment
64			// variables are present.
65			$task = $this->taskExec('env \| wc -l')->interactive(false);
			0 ignored issues – show Bug introduced 2020-02-08 05:06 UTC by Report Bug Copy Issue Report The method `interactive` does only exist in `Robo\Task\Base\Exec`, but not in `Robo\Collection\CollectionBuilder`. 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...
66			$result = $task->run();
67			$start_count = (int) $result->getMessage();
68			$this->assertGreaterThan(0, $start_count);
69
70			// Verify that we get the same amount of environment variables with
71			// another exec call.
72			$task = $this->taskExec('env \| wc -l')->interactive(false);
73			$result = $task->run();
74			$this->assertEquals(
75			$start_count,
76			(int) $result->getMessage());
77
78			// Now run the same command, but this time add another environment
79			// variable, and see if our count increases by one.
80			$task = $this->taskExec('env \| wc -l')->interactive(false);
81			$task->env('FOO', 'BAR');
82			$result = $task->run();
83			$this->assertEquals($start_count + 1, (int) $result->getMessage());
84			}
85			}
86

consolidation / robo

Pull Request — master (#924)

ExecTest::testInheritEnv() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like