From 4677766d210c2dfb19f9c8d930bba99b390fde1c Mon Sep 17 00:00:00 2001
From: Rich Trott <rtrott@gmail.com>
Date: Sun, 30 Apr 2017 15:54:18 -0700
Subject: [PATCH] test: enable test-debugger-pid

Now that `node debug` is an alias for `node inspect`, it's possible that
`node-debug-pid` can run reliably. Modify for current behavior and move
from `disabled` to `parallel`.

PR-URL: https://github.com/nodejs/node/pull/12770
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Refael Ackermann <refack@gmail.com>
---
 .../test-debugger-pid.js                      | 38 ++++++++++---------
 1 file changed, 20 insertions(+), 18 deletions(-)
 rename test/{disabled => parallel}/test-debugger-pid.js (55%)

diff --git a/test/disabled/test-debugger-pid.js b/test/parallel/test-debugger-pid.js
similarity index 55%
rename from test/disabled/test-debugger-pid.js
rename to test/parallel/test-debugger-pid.js
index 7617e84bb409d8..536abf25322e0d 100644
--- a/test/disabled/test-debugger-pid.js
+++ b/test/parallel/test-debugger-pid.js
@@ -8,10 +8,9 @@ let buffer = '';
 // connect to debug agent
 const interfacer = spawn(process.execPath, ['debug', '-p', '655555']);
 
-console.error(process.execPath, 'debug', '-p', '655555');
 interfacer.stdout.setEncoding('utf-8');
 interfacer.stderr.setEncoding('utf-8');
-const onData = function(data) {
+const onData = (data) => {
   data = (buffer + data).split('\n');
   buffer = data.pop();
   data.forEach(function(line) {
@@ -25,26 +24,29 @@ let lineCount = 0;
 interfacer.on('line', function(line) {
   let expected;
   const pid = interfacer.pid;
-  if (common.isWindows) {
-    switch (++lineCount) {
-      case 1:
-        line = line.replace(/^(debug> *)+/, '');
-        const msg = 'There was an internal error in Node\'s debugger. ' +
-                    'Please report this bug.';
-        expected = `(node:${pid}) ${msg}`;
-        break;
+  switch (++lineCount) {
+    case 1:
+      expected =
+        new RegExp(`^\\(node:${pid}\\) \\[DEP0068\\] DeprecationWarning: `);
+      assert.ok(expected.test(line), `expected regexp match for ${line}`);
+      break;
+    case 2:
+      // Doesn't currently work on Windows.
+      if (!common.isWindows) {
+        expected = "Target process: 655555 doesn't exist.";
+        assert.strictEqual(line, expected);
+      }
+      break;
 
-      default:
-        return;
-    }
-  } else {
-    line = line.replace(/^(debug> *)+/, '');
-    expected = `(node:${pid}) Target process: 655555 doesn't exist.`;
+    default:
+      if (!common.isWindows)
+        assert.fail(`unexpected line received: ${line}`);
   }
-
-  assert.strictEqual(expected, line);
 });
 
 interfacer.on('exit', function(code, signal) {
   assert.strictEqual(code, 1, `Got unexpected code: ${code}`);
+  if (!common.isWindows) {
+    assert.strictEqual(lineCount, 2);
+  }
 });