debugger: call this.resume() after this.run()

lance · MylesBorins · commit d8b902f787cb · 2017-01-31T20:04:29.000-05:00
When the debugger has started we need to call `this.resume` otherwise, the prompt won't appear. Fixes: #9854 PR-URL: #10099 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl> Reviewed-By: Rich Trott <rtrott@gmail.com>
diff --git a/lib/_debugger.js b/lib/_debugger.js
@@ -835,7 +835,7 @@ function Interface(stdin, stdout, args) {
   // Run script automatically
   this.pause();
 
-  setImmediate(() => { this.run(); });
+  setImmediate(() => { this.run(() => this.resume()); });
 }
 
 
diff --git a/test/parallel/test-debug-prompt.js b/test/parallel/test-debug-prompt.js
@@ -0,0 +1,13 @@
+'use strict';
+
+const assert = require('assert');
+const common = require('../common');
+const spawn = require('child_process').spawn;
+
+const proc = spawn(process.execPath, ['debug', 'foo']);
+proc.stdout.setEncoding('utf8');
+
+proc.stdout.once('data', common.mustCall((data) => {
+  assert.strictEqual(data, 'debug> ');
+  proc.kill();
+}));

Original file line number	Diff line number	Diff line change
`@@ -835,7 +835,7 @@ function Interface(stdin, stdout, args) {`
`835`	`835`	`// Run script automatically`
`836`	`836`	`this.pause();`
`837`	`837`
`838`		`- setImmediate(() => { this.run(); });`
	`838`	`+ setImmediate(() => { this.run(() => this.resume()); });`
`839`	`839`	`}`
`840`	`840`
`841`	`841`