lewismj / cucumber

Cucumber Test Framework & Plugin for SBT (BDD Testing in Scala)
Other
41 stars 18 forks source link

CucumberTestFramework returns successful despite test failures. #47

Closed dpratt747 closed 5 years ago

dpratt747 commented 5 years ago

Inside CucumberRunner:


            info(Console.RED + s"$shortName .. failed")
            CucumberRunner.numFailures.incrementAndGet()
            handle(new OptionalThrowable(), Status.Success)```

- Would it not be better if this was Status.Failure? or something that more accurately reflects the failed scenario