Skip to content

test: mark test-repl-user-error-handler as flaky#64612

Open
avivkeller wants to merge 1 commit into
nodejs:mainfrom
avivkeller:mark-repl-test-flaky
Open

test: mark test-repl-user-error-handler as flaky#64612
avivkeller wants to merge 1 commit into
nodejs:mainfrom
avivkeller:mark-repl-test-flaky

Conversation

@avivkeller

Copy link
Copy Markdown
Member

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Jul 19, 2026
Signed-off-by: Aviv Keller <me@aviv.sh>
@avivkeller
avivkeller force-pushed the mark-repl-test-flaky branch from 1a813de to db343de Compare July 19, 2026 23:50
@avivkeller avivkeller added repl Issues and PRs related to the REPL subsystem. v8 engine Issues and PRs related to the V8 dependency. inspector Issues and PRs related to the V8 inspector protocol flaky-test Issues and PRs related to the tests with unstable failures on the CI. and removed needs-ci PRs that need a full CI run. labels Jul 19, 2026
@Archkon

Archkon commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

Should these tests be considered as flaky test ?

parallel/test-repl-context
parallel/test-repl-use-global

And I encounter same problem with "Uncaught Error [ERR_INSPECTOR_COMMAND]: Inspector error -32000: Promise was collected"
https://github.com/nodejs/node/actions/runs/29719200410/job/88278594681
https://github.com/nodejs/node/actions/runs/29717783702/job/88274497079
If so, could you rerun the failed github action check due to flaky test of #64604 ? Thanks !

@avivkeller

Copy link
Copy Markdown
Member Author

@avivkeller avivkeller added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jul 22, 2026
@codecov

codecov Bot commented Jul 22, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.14%. Comparing base (0bcc6ef) to head (db343de).
⚠️ Report is 30 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #64612      +/-   ##
==========================================
- Coverage   90.14%   90.14%   -0.01%     
==========================================
  Files         741      741              
  Lines      242080   242080              
  Branches    45563    45574      +11     
==========================================
- Hits       218213   218211       -2     
  Misses      15375    15375              
- Partials     8492     8494       +2     

see 31 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. flaky-test Issues and PRs related to the tests with unstable failures on the CI. inspector Issues and PRs related to the V8 inspector protocol repl Issues and PRs related to the REPL subsystem. test Issues and PRs related to the tests. v8 engine Issues and PRs related to the V8 dependency.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants