fix: don't run beforeAll/afterAll in skipped block

CHANGELOG.md

@@ -4,6 +4,8 @@
 
 ### Fixes
 
+- `[jest-jasmine2]` Don't run `beforeAll` / `afterAll` in skipped describe block
+
 ### Chore & Maintenance
 
 ### Performance

e2e/skip-before-after-all/__tests__/skipDescribe.test.js

@@ -0,0 +1,32 @@
+/**
+ * Copyright (c) Facebook, Inc. and its affiliates. 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.
+ */
+let hasBeforeAllRun = false;
+let hasAfterAllRun = false;
+
+describe.skip('in describe.skip', () => {
+  describe('in describe', () => {
+    beforeAll(() => {
+      hasBeforeAllRun = true;
+    });
+
+    afterAll(() => {
+      hasAfterAllRun = true;
+    });
+
+    test('it should be skipped', () => {
+      global.fail();
+    });
+  });
+});
+
+test('in describe.skip beforeAll should not run', () => {
+  expect(hasBeforeAllRun).toBe(false);
+});
+
+test('in describe.skip afterAll should not run', () => {
+  expect(hasAfterAllRun).toBe(false);
+});

e2e/skip-before-after-all/package.json

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

packages/jest-jasmine2/src/treeProcessor.ts

@@ -23,7 +23,7 @@ export type TreeNode = {
   onException: (error: Error) => void;
   sharedUserContext: () => any;
   children?: Array<TreeNode>;
-} & Pick<Suite, 'getResult' | 'parentSuite' | 'result'>;
+} & Pick<Suite, 'getResult' | 'parentSuite' | 'result' | 'markedPending'>;
 
 export default function treeProcessor(options: Options): void {
   const {
@@ -78,7 +78,7 @@ export default function treeProcessor(options: Options): void {
     const children = node.children.map(child => ({
       fn: getNodeHandler(child, enabled),
     }));
-    if (!hasEnabledTest(node)) {
+    if (!hasEnabledTest(node) || node.markedPending) {
       return children;
     }
     return node.beforeAllFns.concat(children).concat(node.afterAllFns);