Filip/playwright
1
0
Fork 0
Code Issues Pull requests Actions Packages Projects Releases Wiki Activity

added NONE_ISOLATED env for getting round test isiolation if needed

Browse source
This commit is contained in:
Siarhei_Kliushnikau 2024-09-22 09:33:19 +02:00
parent b3a82bef46
commit 6a8b308cb8
1 changed files with 6 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
packages/playwright/src/worker/workerMain.ts
View file

@ -266,7 +266,7 @@ export class WorkerMain extends ProcessRunner {
} }
}; };
if (!this._isStopped) if (!(this._isStopped && !process.env.NONE_ISOLATED))
this._fixtureRunner.setPool(test._pool!); this._fixtureRunner.setPool(test._pool!);
const suites = getSuites(test); const suites = getSuites(test);
@ -320,7 +320,7 @@ export class WorkerMain extends ProcessRunner {
await testInfo._tracing.startIfNeeded(traceFixtureRegistration.fn); await testInfo._tracing.startIfNeeded(traceFixtureRegistration.fn);
}); });
if (this._isStopped || isSkipped) { if ((this._isStopped && !process.env.NONE_ISOLATED) || isSkipped) {
// Two reasons to get here: // Two reasons to get here:
// - Last test is skipped, so we should not run the test, but run the cleanup. // - Last test is skipped, so we should not run the test, but run the cleanup.
// - Worker is requested to stop, but was not able to run full cleanup yet. // - Worker is requested to stop, but was not able to run full cleanup yet.
@ -412,7 +412,7 @@ export class WorkerMain extends ProcessRunner {
if (testInfo._isFailure()) if (testInfo._isFailure())
this._isStopped = true; this._isStopped = true;
if (this._isStopped) { if (this._isStopped && !process.env.NONE_ISOLATED) {
// Run all remaining "afterAll" hooks and teardown all fixtures when worker is shutting down. // Run all remaining "afterAll" hooks and teardown all fixtures when worker is shutting down.
// Mark as "cleaned up" early to avoid running cleanup twice. // Mark as "cleaned up" early to avoid running cleanup twice.
this._didRunFullCleanup = true; this._didRunFullCleanup = true;
@ -541,7 +541,9 @@ export class WorkerMain extends ProcessRunner {
private async _runAfterAllHooksForSuite(suite: Suite, testInfo: TestInfoImpl) { private async _runAfterAllHooksForSuite(suite: Suite, testInfo: TestInfoImpl) {
if (!this._activeSuites.has(suite)) if (!this._activeSuites.has(suite))
return; return;
this._activeSuites.delete(suite); if (!process.env.NONE_ISOLATED) {
this._activeSuites.delete(suite);
}
await this._runAllHooksForSuite(suite, testInfo, 'afterAll'); await this._runAllHooksForSuite(suite, testInfo, 'afterAll');
} }