Completed
Pull Request — master (#924)
by Greg
02:07
created
src/Collection/CollectionBuilder.php 1 patch
Indentation   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -168,16 +168,16 @@
 block discarded – undo
168 168
         return $this;
169 169
     }
170 170
 
171
-  /**
172
-   * Add arbitrary code to execute as a task.
173
-   *
174
-   * @see \Robo\Collection\CollectionInterface::addCode
175
-   *
176
-   * @param callable $code
177
-   * @param int|string $name
178
-   *
179
-   * @return $this
180
-   */
171
+    /**
172
+     * Add arbitrary code to execute as a task.
173
+     *
174
+     * @see \Robo\Collection\CollectionInterface::addCode
175
+     *
176
+     * @param callable $code
177
+     * @param int|string $name
178
+     *
179
+     * @return $this
180
+     */
181 181
     public function addCode(callable $code, $name = \Robo\Collection\CollectionInterface::UNNAMEDTASK)
182 182
     {
183 183
         $this->getCollection()->addCode($code, $name);
Please login to merge, or discard this patch.
tests/_bootstrap.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -6,9 +6,9 @@
 block discarded – undo
6 6
     'debug' => true,
7 7
     'cacheDir' => '/tmp',
8 8
     'includePaths' => [
9
-        __DIR__.'/../src',
10
-        __DIR__.'/../vendor/symfony/process',
11
-        __DIR__.'/../vendor/symfony/console',
12
-        __DIR__.'/../vendor/henrikbjorn/lurker/src',
9
+        __DIR__ . '/../src',
10
+        __DIR__ . '/../vendor/symfony/process',
11
+        __DIR__ . '/../vendor/symfony/console',
12
+        __DIR__ . '/../vendor/henrikbjorn/lurker/src',
13 13
     ]
14 14
 ]);
Please login to merge, or discard this patch.
tests/cli/GenTaskCest.php 1 patch
Indentation   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -7,7 +7,7 @@
 block discarded – undo
7 7
     {
8 8
         $result = $I->taskGenTask('Symfony\Component\Filesystem\Filesystem', 'FilesystemStack')->run();
9 9
         $I->assertContains(
10
-          'protected function _chgrp($files, $group, $recursive = false)',
11
-          $result->getMessage());
10
+            'protected function _chgrp($files, $group, $recursive = false)',
11
+            $result->getMessage());
12 12
     }
13 13
 }
Please login to merge, or discard this patch.
tests/src/Traits/TestTasksTrait.php 1 patch
Doc Comments   +16 added lines patch added patch discarded remove patch
@@ -17,6 +17,9 @@  discard block
 block discarded – undo
17 17
     protected $capturedOutput;
18 18
     protected $logger;
19 19
 
20
+    /**
21
+     * @return \League\Container\ContainerInterface
22
+     */
20 23
     public function initTestTasksTrait($commandClass = null, $container = null, $input = null)
21 24
     {
22 25
         if (!$container) {
@@ -47,6 +50,9 @@  discard block
 block discarded – undo
47 50
         return $container;
48 51
     }
49 52
 
53
+    /**
54
+     * @return null|OutputInterface
55
+     */
50 56
     public function capturedOutputStream()
51 57
     {
52 58
         if (!$this->testPrinter) {
@@ -66,6 +72,9 @@  discard block
 block discarded – undo
66 72
         return $this->capturedOutput;
67 73
     }
68 74
 
75
+    /**
76
+     * @param string $value
77
+     */
69 78
     public function assertOutputContains($value)
70 79
     {
71 80
         $output = $this->accumulate();
@@ -73,6 +82,9 @@  discard block
 block discarded – undo
73 82
         $this->assertContains($value, $output);
74 83
     }
75 84
 
85
+    /**
86
+     * @param string $value
87
+     */
76 88
     public function assertOutputNotContains($value)
77 89
     {
78 90
         $output = $this->accumulate();
@@ -80,6 +92,9 @@  discard block
 block discarded – undo
80 92
         $this->assertNotContains($value, $output);
81 93
     }
82 94
 
95
+    /**
96
+     * @param string $value
97
+     */
83 98
     public function assertOutputEquals($value)
84 99
     {
85 100
         $output = $this->accumulate();
@@ -90,6 +105,7 @@  discard block
 block discarded – undo
90 105
     /**
91 106
      * Make our output comparisons more platform-agnostic by converting
92 107
      * CRLF (Windows) or raw CR (confused output) to a LF (unix/Mac).
108
+     * @param string $output
93 109
      */
94 110
     protected function simplify($output)
95 111
     {
Please login to merge, or discard this patch.
tests/src/Fixtures.php 3 patches
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -46,7 +46,7 @@
 block discarded – undo
46 46
      */
47 47
     public function mktmpdir($basedir = false)
48 48
     {
49
-        $tempfile = tempnam($basedir ?: $this->testDir ?: sys_get_temp_dir(),'robo-tests');
49
+        $tempfile = tempnam($basedir ?: $this->testDir ?: sys_get_temp_dir(), 'robo-tests');
50 50
         unlink($tempfile);
51 51
         mkdir($tempfile);
52 52
         $this->tmpDirs[] = $tempfile;
Please login to merge, or discard this patch.
Braces   +1 added lines, -2 removed lines patch added patch discarded remove patch
@@ -27,8 +27,7 @@
 block discarded – undo
27 27
         foreach ($this->tmpDirs as $tmpDir) {
28 28
             try {
29 29
                 $fs->remove($tmpDir);
30
-            }
31
-            catch (\Exception $e) {
30
+            } catch (\Exception $e) {
32 31
                 // Ignore problems with removing fixtures.
33 32
             }
34 33
         }
Please login to merge, or discard this patch.
Doc Comments   +4 added lines, -1 removed lines patch added patch discarded remove patch
@@ -39,7 +39,7 @@  discard block
 block discarded – undo
39 39
      * Create a new temporary directory.
40 40
      *
41 41
      * @param string|bool $basedir Where to store the temporary directory
42
-     * @return type
42
+     * @return string
43 43
      */
44 44
     public function mktmpdir($basedir = false)
45 45
     {
@@ -61,6 +61,9 @@  discard block
 block discarded – undo
61 61
         return $targetSandbox;
62 62
     }
63 63
 
64
+    /**
65
+     * @param string $filename
66
+     */
64 67
     public function dataFile($filename)
65 68
     {
66 69
         return $this->fixturesDir() . '/' . $filename;
Please login to merge, or discard this patch.
tests/functional/PackExtractTest.php 1 patch
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -41,7 +41,7 @@
 block discarded – undo
41 41
     public function testPackExtract($archiveType)
42 42
     {
43 43
         if ((version_compare(PHP_VERSION, '7.4.0') >= 0) && (getenv('TRAVIS'))) {
44
-          $this->markTestSkipped('Zip libraries apparently not available on Travis CI with PHP 7.4 image.');
44
+            $this->markTestSkipped('Zip libraries apparently not available on Travis CI with PHP 7.4 image.');
45 45
         }
46 46
 
47 47
         // Archive directory and then extract it again with Archive and Extract tasks
Please login to merge, or discard this patch.
src/Task/File/loadTasks.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@
 block discarded – undo
4 4
 trait loadTasks
5 5
 {
6 6
     /**
7
-     * @param array|\Iterator $files
7
+     * @param string[] $files
8 8
      *
9 9
      * @return \Robo\Task\File\Concat|\Robo\Collection\CollectionBuilder
10 10
      */
Please login to merge, or discard this patch.
src/Task/Filesystem/loadTasks.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -4,7 +4,7 @@
 block discarded – undo
4 4
 trait loadTasks
5 5
 {
6 6
     /**
7
-     * @param string|string[] $dirs
7
+     * @param string[] $dirs
8 8
      *
9 9
      * @return \Robo\Task\Filesystem\CleanDir|\Robo\Collection\CollectionBuilder
10 10
      */
Please login to merge, or discard this patch.
tests/functional/FlattenDirTest.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,7 @@
 block discarded – undo
44 44
         $this->fixtures->createAndCdToSandbox();
45 45
 
46 46
         $result = $this->taskFlattenDir('some/deeply/nested/*.re')
47
-            ->includeParents([1,1])
47
+            ->includeParents([1, 1])
48 48
             ->parentDir('some')
49 49
             ->to('flattened')
50 50
             ->run();
Please login to merge, or discard this patch.