cucumber / cucumber-jvm / 3784

86%

push

travis-ci

[TestNG] Fix NPE when executing empty scenario (#1887)

The `TestCaseState` defaults to undefined when a scenario has no steps. The
`TestCaseResultListener` however expects there to be an undefined step.

That the `TestCaseState` defaults to undefined is rather unexpected. The status
should be the most severe of the states so far which means that it should
default to passed. Changing this however will require a major release.

So fixing the test case result listeners is the next best thing. It also
appears that there is some duplication ongoing between TestNG and the JUnit
Platform Engine. However this duplication is hard to fix because each
platform uses its own exceptions and TestNG still supports `--non-strict`.

Fixes: https://github.com/cucumber/cucumber-jvm/issues/1885

81 of 81 new or added lines in 4 files covered. (100.0%)

6317 of 7613 relevant lines covered (82.98%)

0.83 hits per line