CODEWITHSUNDEEP
node.jstddjestjs
gandra404
gandra404

Reputation: 6081

How to fix err Jest has detected the following 3 open handles potentially keeping Jest from exiting

Just starting to work on some node app using jest for testing. express-generator used for scaffolding.
On first test I get following error:

Jest has detected the following 3 open handles potentially keeping Jest from exiting

Steps to reproduce: 

git clone [email protected]:gandra/node-jest-err-demo.git   
cd node-jest-err-demo       
npm install   
cp .env.example .env    
npm run test

npx envinfo --preset jest output: 

npx: installed 1 in 1.896s

  System:
    OS: macOS High Sierra 10.13.4
    CPU: x64 Intel(R) Core(TM) i7-6820HQ CPU @ 2.70GHz   Binaries:
    Node: 9.3.0 - /usr/local/bin/node
    Yarn: 1.5.1 - /usr/local/bin/yarn
    npm: 5.7.1 - /usr/local/bin/npm   npmPackages:
    jest: ^23.1.0 => 23.1.0

Any idea how to fix it?

Here is related issue on github: https://github.com/facebook/jest/issues/6446

Upvotes: 27

Views: 59347

Answers (4)

mikemaccana
mikemaccana

Reputation: 123410

Update April 2023 - new openHandlesTimeout option

jest 29.5.0 has added a new openHandlesTimeout option

openHandlesTimeout [number] Default: 1000

Print a warning indicating that there are probable open handles if Jest does not exit cleanly this number of milliseconds after it completes. Use 0 to disable the warning.

  • You should still ideally close any open handles, however in many cases the handles (like TCP sockets) will close on their own, they just take a little longer than jest expects.
  • If you are currently adding await sleep(3 * SECONDS) or similar for handles to close, you can now just set openHandlesTimeout in your config instead and avoid these hacks.
  • If you don't care about the warning just set openHandlesTimeout to 0.

In my own case, I had:

Jest has detected the following 1 open handle potentially keeping Jest from exiting:

  ●  TLSWRAP

      46 |     options.headers = headers;
      47 |   }
    > 48 |   const response = await fetch(uri, options);
         |                          ^
      49 |
      50 |   let contentType = CONTENT_TYPES.JSON;
      51 |   if (forceResponseContentType) {

      at node_modules/node-fetch/lib/index.js:1468:15

Adding:

// https://jestjs.io/docs/configuration#openhandlestimeout-number
openHandlesTimeout: 2 * SECONDS,

to my jest.config.js makes the issue go away

(SECONDS is a constant of 1000 to avoid magic numbers).

Upvotes: 8

Liger
Liger

Reputation: 191

I was having same issue with Mongoose but in my case calling disconnect in afterAll did not make a change. But adding a promise based setTimeout in beforeAll did work. But then I am not a fan of timers and managed to fix it like this 👇

beforeAll(async () => {
  await mongoose.disconnect();
  await mongoose.connect(MONGODB_URL, MONGODB_OPTIONS);
});

Above workaround feels more natural to me, hope it helps.

Upvotes: 1

tksilicon
tksilicon

Reputation: 4426

Here's what I did to solve this problem.

    afterAll(async () => {
  await new Promise(resolve => setTimeout(() => resolve(), 10000)); // avoid jest open handle error
});

Then I set jest.setTimeout in the particular test that was giving issues.

describe('POST /customers', () => {
  jest.setTimeout(30000);
  test('It creates a customer', async () => {
    const r = Math.random()
      .toString(36)
      .substring(7);
    const response = await request(server)
      .post('/customers')
      .query({
        name: r,
        email: `${r}@${r}.com`,
        password: 'beautiful',
      });
    // console.log(response.body);
    expect(response.body).toHaveProperty('customer');
    expect(response.body).toHaveProperty('accessToken');
    expect(response.statusCode).toBe(200);
  });
});

As answered above, close any other open handles.

Upvotes: 9

Estus Flask
Estus Flask

Reputation: 222760

detectOpenHandles option is used to detect open handles, it should be normally used. The error warns about potentially open handles:

Jest has detected the following 4 open handles potentially keeping Jest from exiting

Even if the handles will be closed, the error will still appear.

The actual problem with this application is that database connection isn't really closed:

if (process.env.NODE_ENV === 'test') {
  mongoose.connection.close(function () {
    console.log('Mongoose connection disconnected');
  });
}

For some reason NODE_ENV is dev, despite that the documentation states that it's expected to be test.

Closing database connection immediately on application start may cause problems in units that actually use database connection. As explained in the guide, MongoDB connection should be at the end of test. Since default Mongoose connection is used, it can be:

afterAll(() => mongoose.disconnect());

Upvotes: 14

Related Questions