Merge pull request #20 from localheinz/fix/output

Fix: Use current output rather than NullOutput
ergebnis · Jan 18, 2018 · 0197481 · 0197481
2 parents aea2b72 + d614bd4
commit 0197481
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/Command/NormalizeCommand.php b/src/Command/NormalizeCommand.php
@@ -120,14 +120,14 @@ protected function execute(Console\Input\InputInterface $input, Console\Output\O
             return 0;
         }
 
-        return $this->updateLocker();
+        return $this->updateLocker($output);
     }
 
-    private function updateLocker(): int
+    private function updateLocker(Console\Output\OutputInterface $output): int
     {
         return $this->getApplication()->run(
             new Console\Input\StringInput('update --lock --no-plugins --no-scripts'),
-            new Console\Output\NullOutput()
+            $output
         );
     }
 }
diff --git a/test/Unit/Command/NormalizeCommandTest.php b/test/Unit/Command/NormalizeCommandTest.php
@@ -524,7 +524,7 @@ public function testExecuteSucceedsIfLockerIsLockedAndLockerCouldBeUpdatedAfterN
                         return 'update --lock --no-plugins --no-scripts' === (string) $input;
                     })
                 ),
-                Argument::type(Console\Output\NullOutput::class)
+                Argument::type(Console\Output\OutputInterface::class)
             )
             ->shouldBeCalled()
             ->willReturn(0);