Skip to content

Commit

Permalink
Updated from comments
Browse files Browse the repository at this point in the history
  • Loading branch information
AutomatedTester committed Mar 14, 2024
1 parent 31c51e3 commit f591113
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 5 deletions.
8 changes: 4 additions & 4 deletions lib/runner/cli/cli.js
Original file line number Diff line number Diff line change
Expand Up @@ -293,7 +293,7 @@ class CliRunner {

if (plugins.length > 0) {
plugins.forEach((plugin) => {
if (plugin.globals && Utils.isFunction(plugin.globals.registerEventHandlers)){
if (plugin.globals && Utils.isFunction(plugin.globals.registerEventHandlers)) {
plugin.globals.registerEventHandlers(NightwatchEventHub);
}
});
Expand Down Expand Up @@ -450,11 +450,11 @@ class CliRunner {
}

isTestWorkersEnabled() {
if (this.testRunner.supportsTestWorkers === false) {
if (this.testRunner.supportsParallelTestSuiteRun === false) {
return false;
}

const testWorkers = this.test_settings.testWorkersEnabled && !singleSourceFile(this.argv);
const testWorkers = this.test_settings.testWorkersEnabled && !singleSourceFile(this.argv);
if (!testWorkers) {

if (this.argv.debug) {
Expand All @@ -470,7 +470,7 @@ class CliRunner {

if (isMobile(desiredCapabilities) && !this.usingServer(this.testEnvSettings[env])) {

if (Concurrency.isWorker()) {
if (Concurrency.isWorker()) {
Logger.info('Disabling parallelism while running tests on mobile platform');
}

Expand Down
2 changes: 1 addition & 1 deletion lib/runner/test-runners/cucumber.js
Original file line number Diff line number Diff line change
Expand Up @@ -249,7 +249,7 @@ class CucumberRunnner extends Runner {
return true;
}

get supportsTestWorkers() {
get supportsParallelTestSuiteRun() {
return false;
}

Expand Down

0 comments on commit f591113

Please sign in to comment.