Skip to content

test_runner: report entryFile in TestStream events#64309

Open
MoLow wants to merge 1 commit into
nodejs:mainfrom
MoLow:add-entry-fille
Open

test_runner: report entryFile in TestStream events#64309
MoLow wants to merge 1 commit into
nodejs:mainfrom
MoLow:add-entry-fille

Conversation

@MoLow

@MoLow MoLow commented Jul 5, 2026

Copy link
Copy Markdown
Member

No description provided.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/test_runner

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test_runner Issues and PRs related to the test runner subsystem. labels Jul 5, 2026
Signed-off-by: Moshe Atlov <moshe@atlow.co.il>
@MoLow MoLow force-pushed the add-entry-fille branch from b21c234 to 86a4589 Compare July 5, 2026 15:07

@mcollina mcollina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm

@mcollina mcollina added semver-minor PRs that contain new features and should be released in the next minor version. request-ci Add this label to start a Jenkins CI on a PR. labels Jul 5, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jul 5, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@MoLow MoLow added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Jul 6, 2026
MoLow added a commit to MoLow/reporters that referenced this pull request Jul 6, 2026
…is ambiguous (#241)

Under --test process isolation, a helper-file subtest's parentId can match
several concurrently open tests from different processes; the eager resolver
tie-break attached it to an arbitrary one until the declaration-ordered block
self-corrected minutes later (e.g. "restore GCE VM" shown under a DynamoDB
test for 20 minutes of a live run). When more than one open candidate matches,
attach the subtest to its helper-file group instead of guessing; it is adopted
by the first event that sees the collision cleared or by the declaration block.
Replaying the originating 40-file run mid-stream drops wrong-test placements
from 29 to 4 (single-wrong-candidate cases, unresolvable until nodejs/node#64309
provides entryFile — TODO left at findOpenParents).

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>
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. commit-queue Add this label to land a pull request using GitHub Actions. needs-ci PRs that need a full CI run. semver-minor PRs that contain new features and should be released in the next minor version. test_runner Issues and PRs related to the test runner subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants