Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: rework testrunner workers #1296

Merged
merged 1 commit into from
Mar 10, 2020
Merged

test: rework testrunner workers #1296

merged 1 commit into from
Mar 10, 2020

Conversation

dgozman
Copy link
Contributor

@dgozman dgozman commented Mar 9, 2020

This change introduces a TestWorker that can be in a certain state,
meaning it has run some beforeAll hooks of a certain test suite stack.

TestWorker can be created at any time, which allows for a number of features:

  • don't run hooks for suites with no runnable tests;
  • smarter test distribution (and possibility for variuos strategies);
  • recovering from hook failures and test failure by creating a new worker;
  • possible isolation between workers by running them in separate environments.

@dgozman dgozman requested a review from aslushnikov March 9, 2020 21:28
utils/testrunner/TestRunner.js Outdated Show resolved Hide resolved
utils/testrunner/TestRunner.js Outdated Show resolved Hide resolved
utils/testrunner/TestRunner.js Outdated Show resolved Hide resolved
utils/testrunner/TestRunner.js Show resolved Hide resolved
utils/testrunner/TestRunner.js Outdated Show resolved Hide resolved
This change introduces a TestWorker that can be in a certain state,
meaning it has run some beforeAll hooks of a certain test suite stack.

TestWorker can be created at any time, which allows for a number of features:
- don't run hooks for suites with no runnable tests;
- smarter test distribution (and possibility for variuos strategies);
- recovering from hook failures and test failure by creating a new worker;
- possible isolation between workers by running them in separate environments.
@aslushnikov aslushnikov merged commit 0ce8efa into microsoft:master Mar 10, 2020
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.

2 participants