consolidation / robo

tests/integration/GenerateMarkdownDocTest.php

Spacing +7 added lines, -7 removed lines

@@ -33,13 +33,13 @@ 
 
         $collection = $this->collectionBuilder();
         $taskGenerator = $collection->taskGenDoc("TestedRoboTask.md");
-        $taskGenerator->filterClasses(function (\ReflectionClass $r) {
+        $taskGenerator->filterClasses(function(\ReflectionClass $r) {
             return !($r->isAbstract() || $r->isTrait()) && $r->implementsInterface('Robo\Contract\TaskInterface');
         })->prepend("# TestedRoboTask Tasks");
         $taskGenerator->docClass('TestedRoboTask');
 
         $taskGenerator->filterMethods(
-            function (\ReflectionMethod $m) {
+            function(\ReflectionMethod $m) {
                 if ($m->isConstructor() || $m->isDestructor() || $m->isStatic()) {
                     return false;
                 }
@@ -65,21 +65,21 @@ 
                 return !in_array($m->name, $undocumentedMethods) && $m->isPublic(); // methods are not documented
             }
         )->processClassSignature(
-            function ($c) {
+            function($c) {
                 return "## " . preg_replace('~Task$~', '', $c->getShortName()) . "\n";
             }
         )->processClassDocBlock(
-            function (\ReflectionClass $c, $doc) {
+            function(\ReflectionClass $c, $doc) {
                 $doc = preg_replace('~@method .*?(.*?)\)~', '* `$1)` ', $doc);
-                $doc = str_replace('\\'.$c->name, '', $doc);
+                $doc = str_replace('\\' . $c->name, '', $doc);
                 return $doc;
             }
         )->processMethodSignature(
-            function (\ReflectionMethod $m, $text) {
+            function(\ReflectionMethod $m, $text) {
                 return str_replace('#### *public* ', '* `', $text) . '`';
             }
         )->processMethodDocBlock(
-            function (\ReflectionMethod $m, $text) {
+            function(\ReflectionMethod $m, $text) {
 
                 return $text ? ' ' . trim(strtok($text, "\n"), "\n") : '';
             }

tests/integration/ExecTest.php

Spacing +3 added lines, -3 removed lines

@@ -68,7 +68,7 @@ 
         $task = $this->taskExec('env | wc -l')->interactive(false);
         $result = $task->run();
         $this->assertTrue($result->wasSuccessful());
-        $start_count = (int) $result->getMessage();
+        $start_count = (int)$result->getMessage();
         $this->assertGreaterThan(0, $start_count);
 
         // Verify that we get the same amount of environment variables with
@@ -78,7 +78,7 @@ 
         $this->assertTrue($result->wasSuccessful());
         $this->assertEquals(
             $start_count,
-            (int) $result->getMessage());
+            (int)$result->getMessage());
 
         // Now run the same command, but this time add another environment
         // variable, and see if our count increases by one.
@@ -86,6 +86,6 @@ 
         $task->env('FOO', 'BAR');
         $result = $task->run();
         $this->assertTrue($result->wasSuccessful());
-        $this->assertEquals($start_count + 1, (int) $result->getMessage());
+        $this->assertEquals($start_count + 1, (int)$result->getMessage());
     }
 }

tests/integration/ForEachTest.php

Spacing +7 added lines, -7 removed lines

@@ -59,8 +59,8 @@ 
         // set iterable in the __constructor
         $result = $this
             ->taskForEach($items)
-            ->withBuilder(function (CollectionBuilder $builder, $key, $value) use (&$actual) {
-                $builder->addCode(function () use ($key, $value, &$actual) {
+            ->withBuilder(function(CollectionBuilder $builder, $key, $value) use (&$actual) {
+                $builder->addCode(function() use ($key, $value, &$actual) {
                     $actual[] = "$key = $value";
 
                     return 0;
@@ -82,7 +82,7 @@ 
         // set iterable with deferTaskConfiguration()
         $result = $this
             ->collectionBuilder()
-            ->addCode(function (Data $data) use ($items) {
+            ->addCode(function(Data $data) use ($items) {
                 $data['items'] = $items;
 
                 return 0;
@@ -91,8 +91,8 @@ 
                 $this
                     ->taskForEach()
                     ->deferTaskConfiguration('setIterable', 'items')
-                    ->withBuilder(function (CollectionBuilder $builder, $key, $value) use (&$actual) {
-                        $builder->addCode(function () use ($key, $value, &$actual) {
+                    ->withBuilder(function(CollectionBuilder $builder, $key, $value) use (&$actual) {
+                        $builder->addCode(function() use ($key, $value, &$actual) {
                             $actual[] = "$key = $value";
 
                             return 0;
@@ -111,8 +111,8 @@ 
         // call the __constructor() without argument
         $result = $this
             ->taskForEach()
-            ->withBuilder(function (CollectionBuilder $builder, $key, $value) use (&$actual) {
-                $builder->addCode(function () use ($key, $value, &$actual) {
+            ->withBuilder(function(CollectionBuilder $builder, $key, $value) use (&$actual) {
+                $builder->addCode(function() use ($key, $value, &$actual) {
                     $actual++;
 
                     return 0;

tests/integration/PackExtractTest.php

Indentation +1 added lines, -1 removed lines

@@ -41,7 +41,7 @@
     public function testPackExtract($archiveType)
     {
         if ((version_compare(PHP_VERSION, '7.4.0') >= 0) && (getenv('TRAVIS'))) {
-          $this->markTestSkipped('Zip libraries apparently not available on Travis CI with PHP 7.4 image.');
+            $this->markTestSkipped('Zip libraries apparently not available on Travis CI with PHP 7.4 image.');
         }
 
         // Archive directory and then extract it again with Archive and Extract tasks

tests/integration/WriteFileTest.php

Indentation +17 added lines, -17 removed lines

@@ -27,10 +27,10 @@ 
     {
         // write lines with WriteToFile task
         $result = $this->taskWriteToFile('blogpost.md')
-           ->line('****')
-           ->line('hello world')
-           ->line('****')
-           ->run();
+            ->line('****')
+            ->line('hello world')
+            ->line('****')
+            ->run();
         $this->assertTrue($result->wasSuccessful(), $result->getMessage());
         $this->assertFileExists('blogpost.md');
         $contents = file_get_contents('blogpost.md');
@@ -46,9 +46,9 @@ 
     public function appendToFile()
     {
         $result = $this->taskWriteToFile('a.txt')
-           ->append()
-           ->line('hello world')
-           ->run();
+            ->append()
+            ->line('hello world')
+            ->run();
         $this->assertTrue($result->wasSuccessful(), $result->getMessage());
         $this->assertFileExists('a.txt');
         $contents = file_get_contents('a.txt');
@@ -62,7 +62,7 @@ 
     public function testWouldChange()
     {
         $writeTask = $this->taskWriteToFile('a.txt')
-           ->append();
+            ->append();
         $this->assertEquals(false, $writeTask->wouldChange(), "No changes to test file.");
         $writeTask->line('hello world');
         $this->assertEquals(true, $writeTask->wouldChange(), "Test file would change.");
@@ -90,15 +90,15 @@ 
     {
         // append lines with WriteToFile task, but only if pattern does not match
         $result = $this->taskWriteToFile('blogpost.md')
-           ->line('****')
-           ->line('hello world')
-           ->line('****')
-           ->appendUnlessMatches('/hello/', 'Should not add this')
-           ->appendUnlessMatches('/goodbye/', 'Should add this')
-           ->appendIfMatches('/hello/', ' and should also add this')
-           ->appendIfMatches('/goodbye/', ' but should not add this')
-           ->appendIfMatches('/should/', '!')
-           ->run();
+            ->line('****')
+            ->line('hello world')
+            ->line('****')
+            ->appendUnlessMatches('/hello/', 'Should not add this')
+            ->appendUnlessMatches('/goodbye/', 'Should add this')
+            ->appendIfMatches('/hello/', ' and should also add this')
+            ->appendIfMatches('/goodbye/', ' but should not add this')
+            ->appendIfMatches('/should/', '!')
+            ->run();
         $this->assertTrue($result->wasSuccessful(), $result->getMessage());
         $this->assertFileExists('blogpost.md');
         $contents = file_get_contents('blogpost.md');

tests/integration/FlattenDirTest.php

Spacing +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
         $this->fixtures->createAndCdToSandbox();
 
         $result = $this->taskFlattenDir('some/deeply/nested/*.re')
-            ->includeParents([1,1])
+            ->includeParents([1, 1])
             ->parentDir('some')
             ->to('flattened')
             ->run();

tests/integration/GenerateTaskTest.php

Indentation +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
         $result = $this->taskGenTask('Symfony\Component\Filesystem\Filesystem', 'FilesystemStack')->run();
         $this->assertTrue($result->wasSuccessful(), $result->getMessage());
         $this->assertContains(
-          'protected function _chgrp($files, $group, $recursive = false)',
-          $result->getMessage());
+            'protected function _chgrp($files, $group, $recursive = false)',
+            $result->getMessage());
     }
 
 }

tests/integration/AssetsTest.php

Indentation +14 added lines, -14 removed lines

@@ -25,29 +25,29 @@
     public function testCssMinification()
     {
         if (version_compare(PHP_VERSION, '7.2.0') >= 0) {
-          $this->markTestSkipped('natxet/cssmin uses deprecated "each()" function (PHP 7.2+)');
+            $this->markTestSkipped('natxet/cssmin uses deprecated "each()" function (PHP 7.2+)');
         }
 
         $this->fixtures->createAndCdToSandbox();
 
-				$sampleCss = $this->fixtures->dataFile('sample.css');
-				$outputCss = 'minifiedSample.css';
+                $sampleCss = $this->fixtures->dataFile('sample.css');
+                $outputCss = 'minifiedSample.css';
 
-				$initialFileSize = filesize($sampleCss);
+                $initialFileSize = filesize($sampleCss);
 
-				$result = $this->taskMinify($sampleCss)
-				  ->to('minifiedSample.css')
-				  ->run();
+                $result = $this->taskMinify($sampleCss)
+                    ->to('minifiedSample.css')
+                    ->run();
         $this->assertTrue($result->wasSuccessful(), $result->getMessage());
 
-				$this->assertFileExists($outputCss);
-				$minifiedFileSize = filesize($outputCss);
-				$outputCssContents = file_get_contents($outputCss);
+                $this->assertFileExists($outputCss);
+                $minifiedFileSize = filesize($outputCss);
+                $outputCssContents = file_get_contents($outputCss);
 
-				$this->assertLessThan($initialFileSize, $minifiedFileSize, 'Minified file is smaller than the source file');
-				$this->assertGreaterThan(0, $minifiedFileSize, 'Minified file is not empty');
-				$this->assertContains('body', $outputCssContents, 'Minified file has some content from the source file');
-				$this->assertNotContains('Sample css file', $outputCssContents, 'Minified file does not contain comment from source file');
+                $this->assertLessThan($initialFileSize, $minifiedFileSize, 'Minified file is smaller than the source file');
+                $this->assertGreaterThan(0, $minifiedFileSize, 'Minified file is not empty');
+                $this->assertContains('body', $outputCssContents, 'Minified file has some content from the source file');
+                $this->assertNotContains('Sample css file', $outputCssContents, 'Minified file does not contain comment from source file');
     }
 
 }

tests/integration/CollectionTest.php

Spacing +11 added lines, -11 removed lines

@@ -161,7 +161,7 @@ 
             ->rollback(
                 $this->taskDeleteDir('j')
             )
-            ->rollbackCode(function () {
+            ->rollbackCode(function() {
                 throw new AbortTasksException('Aborting rollback.');
             })
             ->taskFilesystemStack()
@@ -220,7 +220,7 @@ 
                 ->mkdir('stuff')
             ->taskForEach($processList)
                 ->withBuilder(
-                    function ($builder, $key, $value) {
+                    function($builder, $key, $value) {
                         return $builder
                             ->taskFilesystemStack()
                                 ->touch("stuff/{$value}.txt");
@@ -277,32 +277,32 @@ 
 
     public function testRollbackInCorrectOrder()
     {
-        $expected_order = [6,5,4,3,2,1];
+        $expected_order = [6, 5, 4, 3, 2, 1];
         $actual_order = [];
         $collection = $this->collectionBuilder();
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 1;
         });
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 2;
         });
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 3;
         });
         // Add a nested collection with rollbacks.
         $nested_collection = $this->collectionBuilder();
-        $nested_collection->rollbackCode(function () use (&$actual_order) {
+        $nested_collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 4;
         });
-        $nested_collection->rollbackCode(function () use (&$actual_order) {
+        $nested_collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 5;
         });
         $collection->addTask($nested_collection);
 
-        $collection->rollbackCode(function () use (&$actual_order) {
+        $collection->rollbackCode(function() use (&$actual_order) {
             $actual_order[] = 6;
         });
-        $collection->addCode(function () {
+        $collection->addCode(function() {
             return Result::EXITCODE_ERROR;
         });
         $result = $collection->run();
@@ -505,7 +505,7 @@ 
         $collection = $this->collectionBuilder();
 
         $collection->addCode(
-            function () {
+            function() {
                 throw new \RuntimeException('Error');
             }
         );