fix #342 set child process debug port to available — replace process.…

…execArgv in tests instead of local option
avajs · Jun 20, 2016 · 00ea657 · 00ea657
1 parent 81c8e0b
commit 00ea657
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/api.js b/api.js
@@ -135,7 +135,7 @@ Api.prototype._run = function (files, _options) {
 };
 
 Api.prototype.computeForkExecArgs = function (files) {
-	var execArgv = this.options.testOnlyExecArgv || process.execArgv;
+	var execArgv = process.execArgv;
 	var debugArgIndex = -1;
 
 	// --debug-brk is used in addition to --inspect to break on first line and wait

diff --git a/test/api.js b/test/api.js
@@ -1027,7 +1027,9 @@ function generatePassDebugTests(execArgv, expectedInspectIndex) {
 	test('pass ' + execArgv.join(' ') + ' to fork', function (t) {
 		t.plan(expectedInspectIndex === -1 ? 3 : 2);
 
-		var api = new Api({testOnlyExecArgv: execArgv});
+		var oldExecArgv = process.execArgv;
+		process.execArgv = execArgv;
+		var api = new Api();
 		return api.computeForkExecArgs(['foo.js'])
 			.then(function (result) {
 				t.true(result.length === 1);
@@ -1037,6 +1039,9 @@ function generatePassDebugTests(execArgv, expectedInspectIndex) {
 				} else {
 					t.true(/--inspect=\d+/.test(result[0][expectedInspectIndex]));
 				}
+			})
+			.finally(function () {
+				process.execArgv = oldExecArgv;
 			});
 	});
 }