From 264a5d217e512892c06b9b7987b341865eb48dc2 Mon Sep 17 00:00:00 2001 From: AlexKamaev Date: Tue, 7 Apr 2020 07:36:16 +0300 Subject: [PATCH] Do not wait for reporter is executed when concurrency is enabled (closes #4894) (#4942) --- src/runner/browser-job.js | 2 +- test/functional/fixtures/regression/gh-4787/test.js | 1 - 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/src/runner/browser-job.js b/src/runner/browser-job.js index 1f59b30366..83b65f66dd 100644 --- a/src/runner/browser-job.js +++ b/src/runner/browser-job.js @@ -125,7 +125,7 @@ export default class BrowserJob extends AsyncEventEmitter { const hasIncompleteTestRuns = this.completionQueue.some(controller => !controller.done); const needWaitLastTestInFixture = this.reportsPending.some(controller => controller.test.fixture !== testRunController.test.fixture); - if (isBlocked || needWaitLastTestInFixture || hasIncompleteTestRuns && !isConcurrency) + if (isBlocked || (hasIncompleteTestRuns || needWaitLastTestInFixture) && !isConcurrency) break; this.reportsPending.push(testRunController); diff --git a/test/functional/fixtures/regression/gh-4787/test.js b/test/functional/fixtures/regression/gh-4787/test.js index 7b54ae8428..5a0404f57c 100644 --- a/test/functional/fixtures/regression/gh-4787/test.js +++ b/test/functional/fixtures/regression/gh-4787/test.js @@ -58,7 +58,6 @@ if (config.useLocalBrowsers && !config.useHeadlessBrowsers) { .src(path.join(__dirname, './testcafe-fixtures/index.js')) .browsers(['chrome', 'firefox']) .reporter(customReporter) - .concurrency(2) .run(); }) .then(() => {