throw exception when unsupported options are passed

when options starting with a dash are passed which aren't handled specifically, the usages are printed to system.out and an exception is thrown

Issue cucumber#463

Author: Klaus

core/src/main/java/cucumber/runtime/RuntimeOptions.java

@@ -75,7 +75,7 @@ private void parse(List<String> args) {
             String arg = args.remove(0).trim();
 
             if (arg.equals("--help") || arg.equals("-h")) {
-                System.out.println(USAGE);
+                printUsage();
                 System.exit(0);
             } else if (arg.equals("--version") || arg.equals("-v")) {
                 System.out.println(VERSION);
@@ -100,6 +100,9 @@ private void parse(List<String> args) {
                 String nextArg = args.remove(0);
                 Pattern patternFilter = Pattern.compile(nextArg);
                 parsedFilters.add(patternFilter);
+            } else if (arg.startsWith("-")) {
+                printUsage();
+                throw new CucumberException("Unknown option: " + arg);
             } else {
                 PathWithLines pathWithLines = new PathWithLines(arg);
                 featurePaths.add(pathWithLines.path);
@@ -112,6 +115,11 @@ private void parse(List<String> args) {
         }
     }
 
+    private void printUsage()
+    {
+        System.out.println(USAGE);
+    }
+
     public List<CucumberFeature> cucumberFeatures(ResourceLoader resourceLoader) {
         return load(resourceLoader, featurePaths, filters);
     }

core/src/test/java/cucumber/runtime/RuntimeOptionsTest.java

@@ -11,6 +11,7 @@
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
 import static org.junit.Assert.assertTrue;
+import static org.junit.Assert.fail;
 
 public class RuntimeOptionsTest {
     @Test
@@ -153,4 +154,15 @@ public void allows_removal_of_strict_in_cucumber_options_property() {
         RuntimeOptions runtimeOptions = new RuntimeOptions(properties, "--strict");
         assertFalse(runtimeOptions.strict);
     }
+
+    @Test
+    public void fail_on_unsupported_options() {
+        try {
+            new RuntimeOptions(new Properties(), "-concreteUnsupportedOption", "somewhere", "somewhere_else");
+            fail();
+        } catch (CucumberException e) {
+            assertEquals("Unknown option: -concreteUnsupportedOption", e.getMessage());
+        }
+    }
+
 }

core/src/test/java/cucumber/runtime/RuntimeTest.java

@@ -164,11 +164,11 @@ public void strict_with_errors() {
     }
 
     private Runtime createStrictRuntime() {
-        return createRuntime("-g anything", "--strict");
+        return createRuntime("-g", "anything", "--strict");
     }
 
     private Runtime createNonStrictRuntime() {
-        return createRuntime("-g anything");
+        return createRuntime("-g", "anything");
     }
 
     private Runtime createRuntime(String... runtimeArgs) {