phingofficial / phing

src/Phing/PropertyHelper.php

Spacing +1 added lines, -1 removed lines

@@ -214,7 +214,7 @@
         while (false !== strpos($sb, '${')) {
             $sb = preg_replace_callback(
                 '/\$\{([^\$}]+)\}/',
-                function ($matches) use ($keys) {
+                function($matches) use ($keys) {
                     $propertyName = $matches[1];
 
                     $replacement = null;

src/Phing/Listener/DefaultLogger.php

Spacing +1 added lines, -1 removed lines

@@ -342,7 +342,7 @@
     public static function formatTime(float $seconds): string
     {
         /** @var float|int $number */
-        $getPlural = function ($number): string {
+        $getPlural = function($number): string {
             return 1 == $number ? '' : 's';
         };
         $chunks = [];

tests/Phing/Task/Optional/PHPStan/PHPStanAnalyseCommandBuilderTest.php

Indentation -16 removed lines

@@ -57,19 +57,3 @@
 
         $this->builder->build($task);
         $expectedCommand = <<< 'CMD'
-            Executing 'phpstan' with arguments:
-            'analyse'
-            '--configuration=anyConfiguration'
-            '--level=anyLevel'
-            '--no-progress'
-            '--debug'
-            '--autoload-file=anyAutoloadFile'
-            '--error-format=anyErrorFormat'
-            '--memory-limit=anyMemoryLimit'
-            'path1 path2'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Switch Indentation -16 removed lines

@@ -57,19 +57,3 @@
 
         $this->builder->build($task);
         $expectedCommand = <<< 'CMD'
-            Executing 'phpstan' with arguments:
-            'analyse'
-            '--configuration=anyConfiguration'
-            '--level=anyLevel'
-            '--no-progress'
-            '--debug'
-            '--autoload-file=anyAutoloadFile'
-            '--error-format=anyErrorFormat'
-            '--memory-limit=anyMemoryLimit'
-            'path1 path2'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Spacing -16 removed lines

@@ -57,19 +57,3 @@
 
         $this->builder->build($task);
         $expectedCommand = <<< 'CMD'
-            Executing 'phpstan' with arguments:
-            'analyse'
-            '--configuration=anyConfiguration'
-            '--level=anyLevel'
-            '--no-progress'
-            '--debug'
-            '--autoload-file=anyAutoloadFile'
-            '--error-format=anyErrorFormat'
-            '--memory-limit=anyMemoryLimit'
-            'path1 path2'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Braces -16 removed lines

@@ -57,19 +57,3 @@
 
         $this->builder->build($task);
         $expectedCommand = <<< 'CMD'
-            Executing 'phpstan' with arguments:
-            'analyse'
-            '--configuration=anyConfiguration'
-            '--level=anyLevel'
-            '--no-progress'
-            '--debug'
-            '--autoload-file=anyAutoloadFile'
-            '--error-format=anyErrorFormat'
-            '--memory-limit=anyMemoryLimit'
-            'path1 path2'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Upper-Lower-Casing -16 removed lines

@@ -57,19 +57,3 @@
 
         $this->builder->build($task);
         $expectedCommand = <<< 'CMD'
-            Executing 'phpstan' with arguments:
-            'analyse'
-            '--configuration=anyConfiguration'
-            '--level=anyLevel'
-            '--no-progress'
-            '--debug'
-            '--autoload-file=anyAutoloadFile'
-            '--error-format=anyErrorFormat'
-            '--memory-limit=anyMemoryLimit'
-            'path1 path2'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

tests/Phing/Task/Optional/PHPStan/PHPStanCommandBuilderTest.php

Indentation -50 removed lines

@@ -49,53 +49,3 @@
         $this->builder->build($task);
 
         $cmd = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($cmd, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-
-    public function testItFailsWhenExecutableNotSet(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('');
-
-        $this->expectException(BuildException::class);
-
-        $this->builder->build($task);
-    }
-
-    public function testItHandlesCommonOptions(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('anyExecutable');
-        $task->setCommand('anyCommand');
-
-        $task->setHelp(true);
-        $task->setQuiet(true);
-        $task->setVersion(true);
-        $task->setANSI(true);
-        $task->setNoANSI(true);
-        $task->setNoInteraction(true);
-        $task->setVerbose(true);
-
-        $this->builder->build($task);
-
-        $expectedCommand = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            '--help'
-            '--quiet'
-            '--version'
-            '--ansi'
-            '--no-ansi'
-            '--no-interaction'
-            '--verbose'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Switch Indentation -50 removed lines

@@ -49,53 +49,3 @@
         $this->builder->build($task);
 
         $cmd = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($cmd, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-
-    public function testItFailsWhenExecutableNotSet(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('');
-
-        $this->expectException(BuildException::class);
-
-        $this->builder->build($task);
-    }
-
-    public function testItHandlesCommonOptions(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('anyExecutable');
-        $task->setCommand('anyCommand');
-
-        $task->setHelp(true);
-        $task->setQuiet(true);
-        $task->setVersion(true);
-        $task->setANSI(true);
-        $task->setNoANSI(true);
-        $task->setNoInteraction(true);
-        $task->setVerbose(true);
-
-        $this->builder->build($task);
-
-        $expectedCommand = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            '--help'
-            '--quiet'
-            '--version'
-            '--ansi'
-            '--no-ansi'
-            '--no-interaction'
-            '--verbose'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Spacing -50 removed lines

@@ -49,53 +49,3 @@
         $this->builder->build($task);
 
         $cmd = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($cmd, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-
-    public function testItFailsWhenExecutableNotSet(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('');
-
-        $this->expectException(BuildException::class);
-
-        $this->builder->build($task);
-    }
-
-    public function testItHandlesCommonOptions(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('anyExecutable');
-        $task->setCommand('anyCommand');
-
-        $task->setHelp(true);
-        $task->setQuiet(true);
-        $task->setVersion(true);
-        $task->setANSI(true);
-        $task->setNoANSI(true);
-        $task->setNoInteraction(true);
-        $task->setVerbose(true);
-
-        $this->builder->build($task);
-
-        $expectedCommand = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            '--help'
-            '--quiet'
-            '--version'
-            '--ansi'
-            '--no-ansi'
-            '--no-interaction'
-            '--verbose'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Braces -50 removed lines

@@ -49,53 +49,3 @@
         $this->builder->build($task);
 
         $cmd = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($cmd, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-
-    public function testItFailsWhenExecutableNotSet(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('');
-
-        $this->expectException(BuildException::class);
-
-        $this->builder->build($task);
-    }
-
-    public function testItHandlesCommonOptions(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('anyExecutable');
-        $task->setCommand('anyCommand');
-
-        $task->setHelp(true);
-        $task->setQuiet(true);
-        $task->setVersion(true);
-        $task->setANSI(true);
-        $task->setNoANSI(true);
-        $task->setNoInteraction(true);
-        $task->setVerbose(true);
-
-        $this->builder->build($task);
-
-        $expectedCommand = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            '--help'
-            '--quiet'
-            '--version'
-            '--ansi'
-            '--no-ansi'
-            '--no-interaction'
-            '--verbose'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Upper-Lower-Casing -50 removed lines

@@ -49,53 +49,3 @@
         $this->builder->build($task);
 
         $cmd = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($cmd, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-
-    public function testItFailsWhenExecutableNotSet(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('');
-
-        $this->expectException(BuildException::class);
-
-        $this->builder->build($task);
-    }
-
-    public function testItHandlesCommonOptions(): void
-    {
-        $task = new PHPStanTask();
-        $task->setExecutable('anyExecutable');
-        $task->setCommand('anyCommand');
-
-        $task->setHelp(true);
-        $task->setQuiet(true);
-        $task->setVersion(true);
-        $task->setANSI(true);
-        $task->setNoANSI(true);
-        $task->setNoInteraction(true);
-        $task->setVerbose(true);
-
-        $this->builder->build($task);
-
-        $expectedCommand = <<<'CMD'
-            Executing 'anyExecutable' with arguments:
-            'anyCommand'
-            '--help'
-            '--quiet'
-            '--version'
-            '--ansi'
-            '--no-ansi'
-            '--no-interaction'
-            '--verbose'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

tests/Phing/Task/Optional/PHPStan/PHPStanListCommandBuilderTest.php

Indentation -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'list'
-            '--format=anyFormat'
-            '--raw'
-            'anyNamespace'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Switch Indentation -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'list'
-            '--format=anyFormat'
-            '--raw'
-            'anyNamespace'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Spacing -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'list'
-            '--format=anyFormat'
-            '--raw'
-            'anyNamespace'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Braces -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'list'
-            '--format=anyFormat'
-            '--raw'
-            'anyNamespace'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Upper-Lower-Casing -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'list'
-            '--format=anyFormat'
-            '--raw'
-            'anyNamespace'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

tests/Phing/Task/Optional/PHPStan/PHPStanHelpCommandBuilderTest.php

Indentation -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'help'
-            '--format=anyFormat'
-            '--raw'
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Switch Indentation -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'help'
-            '--format=anyFormat'
-            '--raw'
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Spacing -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'help'
-            '--format=anyFormat'
-            '--raw'
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Braces -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'help'
-            '--format=anyFormat'
-            '--raw'
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

Upper-Lower-Casing -11 removed lines

@@ -53,14 +53,3 @@
         $this->builder->build($task);
 
         $expectedCommand = <<<'CMD'
-            Executing 'phpstan' with arguments:
-            'help'
-            '--format=anyFormat'
-            '--raw'
-            'anyCommand'
-            The ' characters around the executable and arguments are not part of the command.
-            CMD;
-
-        $this->assertEquals($expectedCommand, str_replace("\r", '', $task->getCommandline()->describeCommand()));
-    }
-}

tests/Phing/Task/System/EchoXMLTest.php

Indentation -12 removed lines

@@ -50,15 +50,3 @@
         $this->assertStringEqualsFile(
             $this->getProject()->getProperty('file'),
             <<< 'XML'
-                <project name="failure" default="" basedir=".">
-                  <fail message="foo=bar"/>
-                </project>
-                XML
-        );
-    }
-
-    public function testEmpty(): void
-    {
-        $this->expectBuildException('testEmpty', 'No nested XML specified');
-    }
-}

Switch Indentation -12 removed lines

@@ -50,15 +50,3 @@
         $this->assertStringEqualsFile(
             $this->getProject()->getProperty('file'),
             <<< 'XML'
-                <project name="failure" default="" basedir=".">
-                  <fail message="foo=bar"/>
-                </project>
-                XML
-        );
-    }
-
-    public function testEmpty(): void
-    {
-        $this->expectBuildException('testEmpty', 'No nested XML specified');
-    }
-}

Spacing -12 removed lines

@@ -50,15 +50,3 @@
         $this->assertStringEqualsFile(
             $this->getProject()->getProperty('file'),
             <<< 'XML'
-                <project name="failure" default="" basedir=".">
-                  <fail message="foo=bar"/>
-                </project>
-                XML
-        );
-    }
-
-    public function testEmpty(): void
-    {
-        $this->expectBuildException('testEmpty', 'No nested XML specified');
-    }
-}

Braces -12 removed lines

@@ -50,15 +50,3 @@
         $this->assertStringEqualsFile(
             $this->getProject()->getProperty('file'),
             <<< 'XML'
-                <project name="failure" default="" basedir=".">
-                  <fail message="foo=bar"/>
-                </project>
-                XML
-        );
-    }
-
-    public function testEmpty(): void
-    {
-        $this->expectBuildException('testEmpty', 'No nested XML specified');
-    }
-}

Upper-Lower-Casing -12 removed lines

@@ -50,15 +50,3 @@
         $this->assertStringEqualsFile(
             $this->getProject()->getProperty('file'),
             <<< 'XML'
-                <project name="failure" default="" basedir=".">
-                  <fail message="foo=bar"/>
-                </project>
-                XML
-        );
-    }
-
-    public function testEmpty(): void
-    {
-        $this->expectBuildException('testEmpty', 'No nested XML specified');
-    }
-}

tests/Phing/Filter/StripWhitespaceTest.php

Indentation -9 removed lines

@@ -47,12 +47,3 @@
         $resultFile = $this->getProject()->resolveFile('result/stripwhitespace.test');
 
         $expected = <<<'EXPECTED'
-            <?php
-            class { public function __construct() { return ''; } }
-            EXPECTED;
-
-        $result = file_get_contents($resultFile->getAbsolutePath());
-
-        $this->assertEquals($expected, $result, "Files don't match!");
-    }
-}

Switch Indentation -9 removed lines

@@ -47,12 +47,3 @@
         $resultFile = $this->getProject()->resolveFile('result/stripwhitespace.test');
 
         $expected = <<<'EXPECTED'
-            <?php
-            class { public function __construct() { return ''; } }
-            EXPECTED;
-
-        $result = file_get_contents($resultFile->getAbsolutePath());
-
-        $this->assertEquals($expected, $result, "Files don't match!");
-    }
-}

Spacing -9 removed lines

@@ -47,12 +47,3 @@
         $resultFile = $this->getProject()->resolveFile('result/stripwhitespace.test');
 
         $expected = <<<'EXPECTED'
-            <?php
-            class { public function __construct() { return ''; } }
-            EXPECTED;
-
-        $result = file_get_contents($resultFile->getAbsolutePath());
-
-        $this->assertEquals($expected, $result, "Files don't match!");
-    }
-}

Braces -9 removed lines

@@ -47,12 +47,3 @@
         $resultFile = $this->getProject()->resolveFile('result/stripwhitespace.test');
 
         $expected = <<<'EXPECTED'
-            <?php
-            class { public function __construct() { return ''; } }
-            EXPECTED;
-
-        $result = file_get_contents($resultFile->getAbsolutePath());
-
-        $this->assertEquals($expected, $result, "Files don't match!");
-    }
-}

Upper-Lower-Casing -9 removed lines

@@ -47,12 +47,3 @@
         $resultFile = $this->getProject()->resolveFile('result/stripwhitespace.test');
 
         $expected = <<<'EXPECTED'
-            <?php
-            class { public function __construct() { return ''; } }
-            EXPECTED;
-
-        $result = file_get_contents($resultFile->getAbsolutePath());
-
-        $this->assertEquals($expected, $result, "Files don't match!");
-    }
-}

tests/Phing/Task/Optional/SonarConfigurationFileParserTest.php

Spacing +2 added lines, -2 removed lines

@@ -179,7 +179,7 @@ 
         $fh = fopen($tmpFile, 'w');
 
         if (false !== $fh) {
-            register_shutdown_function(static function () use ($tmpFile) {
+            register_shutdown_function(static function() use ($tmpFile) {
                 unlink($tmpFile);
             });
 
@@ -211,7 +211,7 @@ 
 
         $fh = fopen($tmpFile, 'w');
         if (false !== $fh) {
-            register_shutdown_function(static function () use ($tmpFile) {
+            register_shutdown_function(static function() use ($tmpFile) {
                 unlink($tmpFile);
             });