Skip to content

Properly aggregate HTML report from multiple workers#700

Closed
DenysKuchma wants to merge 1 commit into2.xfrom
feature/codeceptjs-workers-html-isMainThread
Closed

Properly aggregate HTML report from multiple workers#700
DenysKuchma wants to merge 1 commit into2.xfrom
feature/codeceptjs-workers-html-isMainThread

Conversation

@DenysKuchma
Copy link
Copy Markdown
Contributor

Problem

HTML report was broken with CodeceptJS workers - each worker created its own report causing data loss.

Solution

Added worker_threads.isMainThread detection to HTML pipe:

  • Workers save tests to temp JSONL files
  • Main thread loads and aggregates all worker files
  • Generates single HTML report with all test data
  • Uses official Node.js API, no adapter changes needed

@codeant-ai
Copy link
Copy Markdown

codeant-ai bot commented Feb 16, 2026

User den.kuchma@ukr.net does not have a PR Review subscription.

Go to Team management and add this email to the PR Review subscription.

Copy link
Copy Markdown
Contributor

@DavertMik DavertMik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Use worker subscriber-dispatcher Workers API

Comment thread src/pipe/html.js
return;
}

try {
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this code is not clear
please divide it into separate function
I would like to undestand the logic by observing the code, but now I should dive deep

@DenysKuchma
Copy link
Copy Markdown
Contributor Author

Better way to fix here: codeceptjs/CodeceptJS#5464

@DenysKuchma DenysKuchma deleted the feature/codeceptjs-workers-html-isMainThread branch February 28, 2026 14:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants