Changes as per PR comments.

cucumber · Apr 7, 2018 · 2e30f73 · 2e30f73
1 parent 80c4c7e
commit 2e30f73
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/core/src/main/java/cucumber/runtime/Stats.java b/core/src/main/java/cucumber/runtime/Stats.java
@@ -94,7 +94,7 @@ public List<Throwable> getErrors() {
 
     public byte exitStatus(boolean isStrict) {
         byte result = 0x0;
-        if (!failedScenarios.isEmpty() || (isStrict && (!pendingScenarios.isEmpty() || !undefinedScenarios.isEmpty() || !ambiguousScenarios.isEmpty()))) {
+        if (!failedScenarios.isEmpty() || !ambiguousScenarios.isEmpty() || (isStrict && (!pendingScenarios.isEmpty() || !undefinedScenarios.isEmpty()))) {
             result |= ERRORS;
         }
         return result;

diff --git a/core/src/main/resources/cucumber/api/cli/USAGE.txt b/core/src/main/resources/cucumber/api/cli/USAGE.txt
@@ -19,7 +19,7 @@ Options:
   -n, --name REGEXP                      Only run scenarios whose names match REGEXP.
   -d, --[no-]-dry-run                    Skip execution of glue code.
   -m, --[no-]-monochrome                 Don't colour terminal output.
-  -s, --[no-]-strict                     Treat undefined, pending and ambiguous steps as errors.
+  -s, --[no-]-strict                     Treat undefined and pending steps as errors.
       --snippets [underscore|camelcase]  Naming convention for generated snippets.
                                          Defaults to underscore.
   -v, --version                          Print version.

diff --git a/core/src/test/java/cucumber/runtime/RuntimeTest.java b/core/src/test/java/cucumber/runtime/RuntimeTest.java
@@ -208,7 +208,7 @@ public void non_strict_with_ambiguous_scenarios() {
         Runtime runtime = createNonStrictRuntime();
         runtime.getEventBus().send(testCaseFinishedWithStatus(Result.Type.AMBIGUOUS));
 
-        assertEquals(0x0, runtime.exitStatus());
+        assertEquals(0x1, runtime.exitStatus());
     }
 
     @Test