Merge pull request #21 from jdobson/master

Call RunNotifier's fireTestFinished() method for failed tests
master
Wouter Groeneveld 8 years ago
commit c74aac7548

@ -228,6 +228,7 @@ public class JasmineTestRunner extends Runner {
notifier.fireTestFinished(spec.getDescription());
} else if (spec.isFailed(rhinoContext)) {
notifier.fireTestFailure(spec.getJunitFailure(rhinoContext));
notifier.fireTestFinished(spec.getDescription());
} else {
throw new IllegalStateException("Unexpected spec status received: " + spec);
}

@ -32,6 +32,7 @@ public class JasmineFailingSpecsTest {
ArgumentCaptor<Description> descriptionCaptor = ArgumentCaptor.forClass(Description.class);
verify(notifierMock).fireTestStarted(descriptionCaptor.capture());
verify(notifierMock).fireTestFailure(failureCaptor.capture());
verify(notifierMock).fireTestFinished(descriptionCaptor.capture());
verifyNoMoreInteractions(notifierMock);
Failure failure = failureCaptor.getValue();
@ -50,6 +51,7 @@ public class JasmineFailingSpecsTest {
ArgumentCaptor<Description> descriptionCaptor = ArgumentCaptor.forClass(Description.class);
verify(notifierMock).fireTestStarted(descriptionCaptor.capture());
verify(notifierMock).fireTestFailure(failureCaptor.capture());
verify(notifierMock).fireTestFinished(descriptionCaptor.capture());
verifyNoMoreInteractions(notifierMock);
Failure failure = failureCaptor.getValue();