Fix/after all asserts bug

CHANGELOG.md

@@ -41,6 +41,8 @@
 
 ### Fixes
 
+* `[jest-jasmine2]` Added assertion error handling inside `afterAll hook`
+  ([#5884](https://github.com/facebook/jest/pull/5884))
 * `[jest-runner]` Assign `process.env.JEST_WORKER_ID="1"` when in runInBand mode
   ([#5860](https://github.com/facebook/jest/pull/5860))
 * `[jest-cli]` Add descriptive error message when trying to use

integration-tests/__tests__/lifecycles.js

@@ -0,0 +1,18 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ * @flow
+ */
+'use strict';
+
+const runJest = require('../runJest');
+
+test('suite with invalid assertions in afterAll', () => {
+  const {stderr, status} = runJest('lifecycles');
+
+  expect(status).toBe(1);
+  expect(stderr).toMatch(/afterAll just failed!/);
+});

integration-tests/lifecycles/__tests__/index.js

@@ -0,0 +1,12 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ */
+
+afterAll(() => {
+  throw new Error('afterAll just failed!');
+});
+test('one', () => {});
+test('two', () => {});

integration-tests/lifecycles/index.js

@@ -0,0 +1,9 @@
+/**
+ * Copyright (c) 2014-present, Facebook, Inc. All rights reserved.
+ *
+ * This source code is licensed under the MIT license found in the
+ * LICENSE file in the root directory of this source tree.
+ *
+ */
+
+module.exports = () => {};

integration-tests/lifecycles/package.json

@@ -0,0 +1,5 @@
+{
+  "jest": {
+    "testEnvironment": "node"
+  }
+}

packages/jest-jasmine2/src/jasmine/js_api_reporter.js

@@ -54,7 +54,14 @@ export default function JsApiReporter(options: Object) {
 
   let executionTime;
 
+  function validateAfterAllExceptions({failedExpectations}) {
+    if (failedExpectations && failedExpectations.length > 0) {
+      throw failedExpectations[0];
+    }
+  }
+
   this.jasmineDone = function(runDetails) {
+    validateAfterAllExceptions(runDetails);
     this.finished = true;
     this.runDetails = runDetails;
     executionTime = timer.elapsed();