fix avajs#342 set child process debug port to available — replace process.execArgv in tests instead of local option

Author: develar

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

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;
 			});
 	});
 }