-
-
Notifications
You must be signed in to change notification settings - Fork 3k
Closed
Labels
area: repository toolingconcerning ease of contributionconcerning ease of contributionstatus: accepting prsMocha can use your help with this one!Mocha can use your help with this one!
Description
Tooling Suggestion Checklist
- I have tried restarting my IDE and the issue persists.
- I have pulled the latest
master
branch of the repository. - I have read and agree to Mocha's Code of Conduct and Contributing Guidelines
- I have searched for related issues and issues with the
faq
label, but none matched my issue. - I want to provide a PR to resolve this
Overview
Example from 9f99178 -> https://github.com/mochajs/mocha/actions/runs/7335062477/job/19972549741:
313 passing (2m)
1 pending
2 failing
1) --exit
default behavior
should not force exit after root suite completion:
Uncaught UnexpectedError:
expected true to be false
at postmortem (test/integration/options/exit.spec.js:51:9)
at /home/runner/work/mocha/mocha/test/integration/helpers.js:69:7
at ChildProcess.<anonymous> (test/integration/helpers.js:352:5)
at ChildProcess.emit (node:events:517:28)
at maybeClose (node:internal/child_process:1098:16)
at ChildProcess._handle.onexit (node:internal/child_process:303:5)
set UNEXPECTED_FULL_TRACE=true to see the full stack trace
2) --exit
when disabled
should not force exit after root suite completion:
Uncaught UnexpectedError:
expected true to be false
at postmortem (test/integration/options/exit.spec.js:51:9)
at /home/runner/work/mocha/mocha/test/integration/helpers.js:69:7
at ChildProcess.<anonymous> (test/integration/helpers.js:352:5)
at ChildProcess.emit (node:events:517:28)
at maybeClose (node:internal/child_process:1098:16)
at ChildProcess._handle.onexit (node:internal/child_process:303:5)
set UNEXPECTED_FULL_TRACE=true to see the full stack trace
Additional Info
This is surprising given that we haven't changed any code since before these started. All commits on master
are failing right now.
- Last commit with ✅: b41e985
- Two commits with no CI runs: 4b60c1a, 060f77d
- First commit with ❌: eca4fec
Edit: ah, I spoke too soon! 645469e is passing ✅. Updating this issue's title to indicate that the tests are nondeterministic. Hooray.
voxpelli
Metadata
Metadata
Assignees
Labels
area: repository toolingconcerning ease of contributionconcerning ease of contributionstatus: accepting prsMocha can use your help with this one!Mocha can use your help with this one!
Type
Projects
Status
Done