Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CLI --watch accomulates error handlers #1692

Closed
Krinkle opened this issue May 1, 2022 · 0 comments
Closed

CLI --watch accomulates error handlers #1692

Krinkle opened this issue May 1, 2022 · 0 comments
Labels
Component: CLI good first issue Straight-forward to start and receive help with. help welcome Type: Bug Something isn't working right.

Comments

@Krinkle
Copy link
Member

Krinkle commented May 1, 2022

Tell us about your runtime:

  • QUnit version: 2.18.2
  • Environment: Node

What actually happened?

Screenshot

What did you expect to happen?

Ultimately, to work correctly and not cause the memory leak warning.

Concretely, we probably either need to keep only one handler for the Node environment and use whichever the last initialised QUnit instance was from the same shared lexical scope (or global), or to properly remove the lister after each run, and re-add it on restart.

@Krinkle Krinkle added Type: Bug Something isn't working right. help welcome Component: CLI good first issue Straight-forward to start and receive help with. labels May 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: CLI good first issue Straight-forward to start and receive help with. help welcome Type: Bug Something isn't working right.
Development

No branches or pull requests

1 participant