Skip to content

Commit

Permalink
Do not wait for reporter is executed when concurrency is enabled (closes
Browse files Browse the repository at this point in the history
  • Loading branch information
AlexKamaev committed Apr 7, 2020
1 parent b4b2b67 commit 264a5d2
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion src/runner/browser-job.js
Expand Up @@ -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);
Expand Down
1 change: 0 additions & 1 deletion test/functional/fixtures/regression/gh-4787/test.js
Expand Up @@ -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(() => {
Expand Down

0 comments on commit 264a5d2

Please sign in to comment.