From 8cc3b9947f6ecf5dd89bb90ece00952c8dcf789e Mon Sep 17 00:00:00 2001
From: Binayak Ghosh <ghoshbinayak@gmail.com>
Date: Wed, 28 Jun 2023 13:16:23 +0530
Subject: [PATCH 1/2] disable programattic api for process listners

---
 lib/index.js          |  2 +-
 lib/runner/cli/cli.js | 11 ++++++++---
 2 files changed, 9 insertions(+), 4 deletions(-)

diff --git a/lib/index.js b/lib/index.js
index 59d9bd3d35..b2ec3dc692 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -279,7 +279,7 @@ Nightwatch.runTests = async function(testSource, settings) {
 Nightwatch.CliRunner = function(argv = {}) {
   const CliRunner = require('./runner/cli/cli.js');
 
-  return new CliRunner(argv);
+  return new CliRunner(argv, true);
 };
 
 /**
diff --git a/lib/runner/cli/cli.js b/lib/runner/cli/cli.js
index 0048f4f279..7eb3f24301 100644
--- a/lib/runner/cli/cli.js
+++ b/lib/runner/cli/cli.js
@@ -83,7 +83,7 @@ class CliRunner {
     }
   }
 
-  constructor(argv = {}) {
+  constructor(argv = {}, isProgrammatic = false) {
     if (argv.source && !argv._source) {
       argv._source = argv.source;
       delete argv.source;
@@ -109,7 +109,10 @@ class CliRunner {
     this.globals = null;
     this.testEnv = null;
     this.testEnvArray = [];
-    this.processListener = new ProcessListener();
+    this.isProgrammatic = isProgrammatic;
+    if (!isProgrammatic) {
+      this.processListener = new ProcessListener();
+    }
   }
 
   initTestSettings(userSettings = {}, baseSettings = null, argv = null, testEnv = '', asyncLoading) {
@@ -452,7 +455,9 @@ class CliRunner {
       globalsInstance: this.globals
     });
 
-    this.processListener.setTestRunner(this.testRunner);
+    if (!this.isProgrammatic) {
+      this.processListener.setTestRunner(this.testRunner);
+    }
 
     return this;
   }

From 9753fd06a96ee337800a52d6322bac2a1c9352de Mon Sep 17 00:00:00 2001
From: Binayak Ghosh <ghoshbinayak@gmail.com>
Date: Tue, 11 Jul 2023 12:08:43 +0530
Subject: [PATCH 2/2] refactor and add tests

---
 lib/index.js                           |  6 ++-
 lib/runner/cli/cli.js                  |  8 +--
 test/src/index/testProgrammaticApis.js | 70 ++++++++++++++++++++++++++
 3 files changed, 78 insertions(+), 6 deletions(-)

diff --git a/lib/index.js b/lib/index.js
index b2ec3dc692..07c330d71d 100644
--- a/lib/index.js
+++ b/lib/index.js
@@ -43,6 +43,7 @@ module.exports.createClient = function({
   debug = false,
   enable_global_apis = false,
   config = './nightwatch.json',
+  disable_process_listener = false,
   test_settings
 } = {}) {
   if (browserName && !env) {
@@ -73,7 +74,8 @@ module.exports.createClient = function({
     timeout,
     devtools,
     debug,
-    parallel
+    parallel,
+    disable_process_listener
   });
 
   const settings = arguments[0] || {};
@@ -279,7 +281,7 @@ Nightwatch.runTests = async function(testSource, settings) {
 Nightwatch.CliRunner = function(argv = {}) {
   const CliRunner = require('./runner/cli/cli.js');
 
-  return new CliRunner(argv, true);
+  return new CliRunner(argv);
 };
 
 /**
diff --git a/lib/runner/cli/cli.js b/lib/runner/cli/cli.js
index 7eb3f24301..bc42fffc46 100644
--- a/lib/runner/cli/cli.js
+++ b/lib/runner/cli/cli.js
@@ -83,7 +83,7 @@ class CliRunner {
     }
   }
 
-  constructor(argv = {}, isProgrammatic = false) {
+  constructor(argv = {}) {
     if (argv.source && !argv._source) {
       argv._source = argv.source;
       delete argv.source;
@@ -109,8 +109,8 @@ class CliRunner {
     this.globals = null;
     this.testEnv = null;
     this.testEnvArray = [];
-    this.isProgrammatic = isProgrammatic;
-    if (!isProgrammatic) {
+
+    if (!argv.disable_process_listener) {
       this.processListener = new ProcessListener();
     }
   }
@@ -455,7 +455,7 @@ class CliRunner {
       globalsInstance: this.globals
     });
 
-    if (!this.isProgrammatic) {
+    if (this.processListener) {
       this.processListener.setTestRunner(this.testRunner);
     }
 
diff --git a/test/src/index/testProgrammaticApis.js b/test/src/index/testProgrammaticApis.js
index 27b2307e38..f281e54be9 100644
--- a/test/src/index/testProgrammaticApis.js
+++ b/test/src/index/testProgrammaticApis.js
@@ -570,4 +570,74 @@ describe('test programmatic apis', function () {
     CliRunner.createDefaultConfig = createDefaultConfig;
     CliRunner.prototype.loadConfig = loadConfig;
   });
+
+  it('test if process listener get disabled', async function() {
+    let processListenerCalled = false;
+    mockery.enable({useCleanCache: true, warnOnUnregistered: false});
+    mockery.registerMock('../process-listener.js', class {
+      constructor() {
+        processListenerCalled = true;
+      }
+
+      setTestRunner() {}
+    });
+
+    const CliRunner = common.require('runner/cli/cli.js');
+    const Nightwatch = common.require('index.js');
+
+    const createDefaultConfig = CliRunner.createDefaultConfig;
+    const loadConfig = CliRunner.prototype.loadConfig;
+    const defaultConfig = {
+      test_settings: {
+        default: {
+          launchUrl: 'http://localhost'
+        }
+      },
+      selenium: {
+        port: 10195,
+        start_process: false
+      },
+      selenium_host: 'localhost'
+    };
+
+    CliRunner.createDefaultConfig = function(destFileName) {
+      return defaultConfig;
+    };
+
+    CliRunner.prototype.loadConfig = function () {
+      return defaultConfig;
+    };
+
+    const clientWithoutListner = Nightwatch.createClient({
+      timeout: 500,
+      useAsync: false,
+      output: false,
+      silent: false,
+      headless: true,
+      output_folder: 'output',
+      globals: {
+        testGlobal: 'one'
+      },
+      disable_process_listener: true
+    });
+
+    assert.ok(!processListenerCalled);
+
+    const client = Nightwatch.createClient({
+      timeout: 500,
+      useAsync: false,
+      output: false,
+      silent: false,
+      headless: true,
+      output_folder: 'output',
+      globals: {
+        testGlobal: 'one'
+      }
+    });
+
+    assert.ok(processListenerCalled);
+
+    CliRunner.createDefaultConfig = createDefaultConfig;
+    CliRunner.prototype.loadConfig = loadConfig;
+  });
 });