Skip to content

Run tests in a single process#9823

Merged
ematipico merged 2 commits intochore/use-node-testfrom
test-serial
Jan 25, 2024
Merged

Run tests in a single process#9823
ematipico merged 2 commits intochore/use-node-testfrom
test-serial

Conversation

@bluwy
Copy link
Member

@bluwy bluwy commented Jan 25, 2024

Changes

Fixes the perf issue in #9758 by running multiple tests in a single process.

Testing

Ran node test manually.

Docs

n/a. internal change.

@bluwy bluwy requested a review from ematipico January 25, 2024 09:24
@changeset-bot
Copy link

changeset-bot bot commented Jan 25, 2024

⚠️ No Changeset found

Latest commit: 2723670

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.

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

@github-actions github-actions bot added the pkg: integration Related to any renderer integration (scope) label Jan 25, 2024
@ematipico ematipico merged commit 4008e33 into chore/use-node-test Jan 25, 2024
@ematipico ematipico deleted the test-serial branch January 25, 2024 10:27
ematipico pushed a commit that referenced this pull request Jan 25, 2024
* Run tests in a single process

* Make test less flaky
ematipico added a commit that referenced this pull request Jan 25, 2024
* chore(@astrojs/node): use Node.js for testing

* revert file

* address feedback

* feedback

* Run tests in a single process (#9823)

* Run tests in a single process

* Make test less flaky

* chore: remove module

---------

Co-authored-by: Bjorn Lu <[email protected]>
ematipico added a commit that referenced this pull request Feb 5, 2025
* chore(@astrojs/node): use Node.js for testing

* revert file

* address feedback

* feedback

* Run tests in a single process (#9823)

* Run tests in a single process

* Make test less flaky

* chore: remove module

---------

Co-authored-by: Bjorn Lu <[email protected]>
ematipico added a commit that referenced this pull request Feb 5, 2025
* chore(@astrojs/node): use Node.js for testing

* revert file

* address feedback

* feedback

* Run tests in a single process (#9823)

* Run tests in a single process

* Make test less flaky

* chore: remove module

---------

Co-authored-by: Bjorn Lu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: integration Related to any renderer integration (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants