Skip to content

🛠 Repo: Nondeterministic test failures on the main branch #5052

@JoshuaKGoldberg

Description

@JoshuaKGoldberg

Tooling Suggestion Checklist

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.

Edit: ah, I spoke too soon! 645469e is passing ✅. Updating this issue's title to indicate that the tests are nondeterministic. Hooray.

Metadata

Metadata

Assignees

Type

No type

Projects

Status

Done

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions