X Tutup
Skip to content

Regenerate the test fixtures with Node 24.3.0#6932

Merged
alcuadrado merged 1 commit intov-nextfrom
node-24.3.0-test-reporter-fixes
Jun 26, 2025
Merged

Regenerate the test fixtures with Node 24.3.0#6932
alcuadrado merged 1 commit intov-nextfrom
node-24.3.0-test-reporter-fixes

Conversation

@alcuadrado
Copy link
Member

Node 24.3.0 reverted 3 changes they had introduced in Node 24.0.0, so we need to regenerate a fixture.

These are:

[dce1995c55] - Revert "test_runner: remove promises returned by t.test()" (Romain Menke) #58282
[8b0c5edbb6] - Revert "test_runner: remove promises returned by test()" (Romain Menke) #58282
[6ef7329c8c] - Revert "test_runner: automatically wait for subtests to finish" (Romain Menke) #58282

@alcuadrado alcuadrado requested review from Copilot and galargh June 26, 2025 14:43
@alcuadrado alcuadrado added the no changeset needed This PR doesn't require a changeset label Jun 26, 2025
@changeset-bot
Copy link

changeset-bot bot commented Jun 26, 2025

⚠️ No Changeset found

Latest commit: 6585a03

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

Regenerate the expected output for the nested-test fixture to align with Node v24.3.0’s reverted test_runner behaviors.

  • Update test numbering and counts to reflect restored promise handling and cancellation.
  • Add newly surfaced stack frames for async subtests in the output.

@alcuadrado alcuadrado enabled auto-merge June 26, 2025 14:50
@alcuadrado alcuadrado added this pull request to the merge queue Jun 26, 2025
Merged via the queue into v-next with commit 8195d18 Jun 26, 2025
238 checks passed
@alcuadrado alcuadrado deleted the node-24.3.0-test-reporter-fixes branch June 26, 2025 14:58
@github-project-automation github-project-automation bot moved this from Backlog to Done in Hardhat Jun 26, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 25, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

no changeset needed This PR doesn't require a changeset

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup