Merge pull request Roave#73 from ciaranmcnulty/fix-from-option-parsing

Check that from is actually provided

Author: Ocramius

src/Command/AssertBackwardsCompatible.php

@@ -148,7 +148,7 @@ public function execute(InputInterface $input, OutputInterface $output) : int
         // @todo fix flaky assumption about the path of the source repo...
         $sourceRepo = CheckedOutRepository::fromPath(getcwd());
 
-        $fromRevision = $input->hasOption('from')
+        $fromRevision = $input->hasParameterOption('from')
             ? $this->parseRevisionFromInput($input, $sourceRepo)
             : $this->determineFromRevisionFromRepository($sourceRepo, $stdErr);
 

test/unit/Command/AssertBackwardsCompatibleTest.php

@@ -110,6 +110,7 @@ public function testExecuteWhenRevisionsAreProvidedAsOptions() : void
         $toSha   = sha1('toRevision', false);
 
         $this->input->expects(self::any())->method('hasOption')->willReturn(true);
+        $this->input->expects(self::any())->method('hasParameterOption')->willReturn(true);
         $this->input->expects(self::any())->method('getOption')->willReturnMap([
             ['from', $fromSha],
             ['to', $toSha],
@@ -160,6 +161,7 @@ public function testExecuteReturnsNonZeroExitCodeWhenChangesAreDetected() : void
         $toSha   = sha1('toRevision', false);
 
         $this->input->expects(self::any())->method('hasOption')->willReturn(true);
+        $this->input->expects(self::any())->method('hasParameterOption')->willReturn(true);
         $this->input->expects(self::any())->method('getOption')->willReturnMap([
             ['from', $fromSha],
             ['to', $toSha],
@@ -222,6 +224,7 @@ public function testProvidingMarkdownOptionWritesMarkdownOutput() : void
         $toSha   = sha1('toRevision', false);
 
         $this->input->expects(self::any())->method('hasOption')->willReturn(true);
+        $this->input->expects(self::any())->method('hasParameterOption')->willReturn(true);
         $this->input->expects(self::any())->method('getOption')->willReturnMap([
             ['from', $fromSha],
             ['to', $toSha],
@@ -349,6 +352,7 @@ public function testExecuteFailsIfCheckedOutRepositoryDoesNotExist() : void
         $toSha   = sha1('toRevision', false);
 
         $this->input->expects(self::any())->method('hasOption')->willReturn(true);
+        $this->input->expects(self::any())->method('hasParameterOption')->willReturn(true);
         $this->input->expects(self::any())->method('getOption')->willReturnMap([
             ['from', $fromSha],
             ['to', $toSha],