test: fix helper-debugger-repl.js

Trott · addaleax · commit e97c610850e6 · 2016-12-08T02:33:36.000+01:00
The test `debugger/test-debugger-repl-break-in-module` (and probably others) was failing because the handshake message for debugging is no longer `listening on port <port>` but is instead `listening on <address>:<port>`. This change makes the check less strict so as to hopefully future-proof it at least a little bit against subsequent changes. This test failure is not caught in CI because currently debugger tests are not run in CI. PR-URL: #9486 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Santiago Gimeno <santiago.gimeno@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Minwoo Jung <jmwsoft@gmail.com> Reviewed-By: Prince John Wesley <princejohnwesley@gmail.com>
diff --git a/test/debugger/helper-debugger-repl.js b/test/debugger/helper-debugger-repl.js
@@ -108,7 +108,7 @@ function addTest(input, output) {
 }
 
 var handshakeLines = [
-  /listening on port \d+/,
+  /listening on /,
   /connecting.* ok/
 ];
 

Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ function addTest(input, output) {`
`108`	`108`	`}`
`109`	`109`
`110`	`110`	`var handshakeLines = [`
`111`		`- /listening on port \d+/,`
	`111`	`+ /listening on /,`
`112`	`112`	`/connecting.* ok/`
`113`	`113`	`];`
`114`	`114`