Skip to content

[jest-cli] Please update node-notifier to fix memory leak #5365

@alexilyaev

Description

@alexilyaev

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
When running tests with --notify, in some situations there's a memory leak in the terminal-notifier process which eats up all of the computer RAM.

If the current behavior is a bug, please provide the steps to reproduce and
either a repl.it demo through https://repl.it/languages/jest or a minimal
repository on GitHub that we can yarn install and yarn test.

This has been reported in:
#2999

I've started experiencing the issue with my team recently, after upgrading to Jest v22.x.x (not sure if it's related, and probably doesn't matter now).

What is the expected behavior?
Looks like the issue has been fixed in latest node-notifier:
mikaelbr/node-notifier#183

So, if we bump node-notifier, the issue should go away.

Please provide your exact Jest configuration and mention your Jest, node,
yarn/npm version and operating system.

I couldn't pin down exactly in what scenario it happens, #2999 has a repro though.

Mac OS X 10.12.6
Jest v22.1.2
Node v9.3.0

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions