Skip to content

Improve reporter message for pre-declaration failures#3444

Closed
NoeFabris wants to merge 1 commit into
avajs:mainfrom
NoeFabris:fix-reporter-worker-failed-no-tests
Closed

Improve reporter message for pre-declaration failures#3444
NoeFabris wants to merge 1 commit into
avajs:mainfrom
NoeFabris:fix-reporter-worker-failed-no-tests

Conversation

@NoeFabris

Copy link
Copy Markdown

Summary

  • report explicitly when a test worker fails before any tests are declared
  • cover the worker-failed reporter path with a focused regression test

Test plan

  • npx tap test-tap/reporters/default-worker-failed.js --disable-coverage
  • npx xo lib/reporters/default.js test-tap/reporters/default-worker-failed.js

Addresses one reporter item from #2501.

@NoeFabris NoeFabris force-pushed the fix-reporter-worker-failed-no-tests branch from 7458379 to 603903b Compare May 11, 2026 02:01
@avajs avajs locked as spam and limited conversation to collaborators May 11, 2026
@sindresorhus

Copy link
Copy Markdown
Member

We don't accept fully AI-generated PRs.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants