Correctly handle getopt failures

Signed-off-by: Michal Čihař <[email protected]>

Author: nijel

src/Utils/CLI.php

@@ -48,6 +48,9 @@ public function parseHighlight()
         $params = $this->getopt(
             'hq:f:', $longopts
         );
+        if ($params === false) {
+            return false;
+        }
         $this->mergeLongOpts($params, $longopts);
         if (! isset($params['f'])) {
             $params['f'] = 'cli';

tests/Utils/CLITest.php

@@ -48,6 +48,11 @@ public function highlightParams()
                 'Usage: highlight-query --query SQL [--format html|cli|text]' . "\n",
                 1,
             ),
+            array(
+                false,
+                '',
+                1,
+            ),
         );
     }
 
@@ -87,6 +92,11 @@ public function lintParams()
                 'Usage: lint-query --query SQL' . "\n",
                 1,
             ),
+            array(
+                false,
+                '',
+                1,
+            ),
         );
     }
 }