afterAll hide errors

[evandrocoan]

🐛 Bug Report

https://repl.it/@evandrocoan/EnviousMagnificentApplicationpackage

const sum = require('./sum');

describe("Test question and answer audios", () => {
  beforeAll(async () => {
    console.log("beforeAll");
  });

  afterAll(async () => {
    console.supererror();
    console.log("afterAll");
  });

  test('Timeout on the wrong place...', async () => {
    expect(sum(1, 2)).toBe(3);
  });
});

Actual behavior

 jest
 PASS  ./sum.test.js
  Test question and answer audios
    ✓ Timeout on the wrong place... (5ms)

  console.log sum.test.js:5
    beforeAll

Test Suites: 1 passed, 1 total
Tests:       1 passed, 1 total
Snapshots:   0 total
Time:        6.199s
Ran all test suites.

Expected behavior

 jest
 FAIL  ./sum.test.js
  Test question and answer audios
    ✕ Timeout on the wrong place... (3ms)

  ● Test question and answer audios › Timeout on the wrong place...

    TypeError: console.supererror is not a function

       7 | 
       8 |   afterAll(async () => {
    >  9 |     console.supererror();
         |             ^
      10 |     console.log("afterAll");
      11 |   });
      12 | 

      at Object.<anonymous> (sum.test.js:9:13)

Test Suites: 1 failed, 1 total
Tests:       1 failed, 1 total
Snapshots:   0 total
Time:        6.657s
Ran all test suites.
exit status 1

envinfo

 npx envinfo --preset jest
npx: installed 1 in 4.22s

  System:
    OS: Linux 4.15 Debian GNU/Linux 9 (stretch) 9 (stretch)
    CPU: (4) x64 Intel(R) Xeon(R) CPU @ 2.30GHz
  Binaries:
    Node: 12.16.2 - /usr/local/bin/node
    Yarn: 1.22.4 - /usr/local/bin/yarn
    npm: 6.14.4 - /usr/local/bin/npm
  npmPackages:
    jest: ^24.9.0 => 24.9.0 

Related:

1. afterAll hook hides errors. afterAll hook hides errors. #3266
2. afterAll inside a describe hides errors afterAll inside a describe hides errors #6692
3. beforeAll() executed in parallel with test case in case of timeout beforeAll() executed in parallel with test case in case of timeout #9527
4. A timeout in a beforeEach continues executing the dependant test cases A timeout in a beforeEach continues executing the dependant test cases #8442
5. beforeAll async hook hides errors beforeAll async hook hides errors #8688

[SimenB]

@evandrocoan I cannot reproduce this. Running the repl you provide it looks like what you outline in "Expected behavior".

If you meant to have the error in afterAll instead I can reproduce using jest-jasmine (default), but it's fixed with jest-circus, which will become the default in Jest 27.

Can you confirm?

[evandrocoan]

Yes. The code I had on Repl.it was different from the code I have here on the first post. I think I changed it on Repl.it thinking it was another Repl.it. The correct code is this:

const sum = require('./sum');

describe("Test question and answer audios", () => {
  afterAll(async () => {
    console.zupererror();
    console.log("afterAll");
  });

  beforeAll(async () => {
    console.log("beforeAll");
  });

  test('Timeout on the wrong place...', async () => {
    expect(sum(1, 2)).toBe(3);
  });
});

Which has an error on the afterAll() but the error is hidden:

[chasen-bettinger]

Why was this closed?

[evandrocoan]

Because he said it will be fixed:

If you meant to have the error in afterAll instead I can reproduce using jest-jasmine (default), but it's fixed with jest-circus, which will become the default in Jest 27.

But I am reopening it until the fix is actually released.

[chasen-bettinger]

Because he said it will be fixed:

If you meant to have the error in afterAll instead I can reproduce using jest-jasmine (default), but it's fixed with jest-circus, which will become the default in Jest 27.

But I am reopening it until the fix is actually released.

Thanks, I was confused because another thread said circus was going to be default in 25. Was elegantly surprised to see that jasmine was still the default in 26.

I can personally report that jest-circus has fixed this issue for me.

[jayarjo]

Related: #9368

[jayarjo]

So what would be the procedure to fix this behavior at the moment? @chasen-bettinger mind to share? Do I simply need to install jest-circus?

[chasen-bettinger]

@jayarjo

Follow the Installation and Configuration steps here: https://www.npmjs.com/package/jest-circus

[SimenB]

feel free to send a PR. Or use jest-cricus - it will be default in 27: #10686

[github-actions]

This issue is stale because it has been open for 1 year with no activity. Remove stale label or comment or this will be closed in 30 days.

[github-actions]

This issue was closed because it has been stalled for 30 days with no activity. Please open a new issue if the issue is still relevant, linking to this one.

[github-actions]

This issue has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.
Please note this issue tracker is not a help forum. We recommend using StackOverflow or our discord channel for questions.