Fix Scenario.isFailed

Adapt `Scenario.isFailed()` to change in `Scenario.getStatus()`.

Fixes cucumber#1215.

Author: olivierlemasle

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

@@ -69,7 +69,7 @@ public Result.Type getStatus() {
 
     @Override
     public boolean isFailed() {
-        return "failed".equals(getStatus());
+        return getStatus() == Result.Type.FAILED;
     }
 
     @Override

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

@@ -12,8 +12,7 @@
 
 import static java.util.Arrays.asList;
 import static org.hamcrest.CoreMatchers.sameInstance;
-import static org.junit.Assert.assertEquals;
-import static org.junit.Assert.assertThat;
+import static org.junit.Assert.*;
 import static org.mockito.Mockito.mock;
 import static org.mockito.Mockito.verify;
 import static org.mockito.Mockito.when;
@@ -37,13 +36,15 @@ public void passed_failed_pending_undefined_skipped_is_failed() throws Exception
         s.add(new Result(Result.Type.UNDEFINED, 0L, null));
         s.add(new Result(Result.Type.SKIPPED, 0L, null));
         assertEquals(Result.Type.FAILED, s.getStatus());
+        assertTrue(s.isFailed());
     }
 
     @Test
     public void passed_and_skipped_is_skipped_although_we_cant_have_skipped_without_undefined_or_pending() throws Exception {
         s.add(new Result(Result.Type.PASSED, 0L, null));
         s.add(new Result(Result.Type.SKIPPED, 0L, null));
         assertEquals(Result.Type.SKIPPED, s.getStatus());
+        assertFalse(s.isFailed());
     }
 
     @Test
@@ -53,6 +54,7 @@ public void passed_pending_undefined_skipped_is_pending() throws Exception {
         s.add(new Result(Result.Type.PENDING, 0L, null));
         s.add(new Result(Result.Type.SKIPPED, 0L, null));
         assertEquals(Result.Type.UNDEFINED, s.getStatus());
+        assertFalse(s.isFailed());
     }
 
     @Test
@@ -61,6 +63,7 @@ public void passed_undefined_skipped_is_undefined() throws Exception {
         s.add(new Result(Result.Type.UNDEFINED, 0L, null));
         s.add(new Result(Result.Type.SKIPPED, 0L, null));
         assertEquals(Result.Type.UNDEFINED, s.getStatus());
+        assertFalse(s.isFailed());
     }
 
     @Test