tests: fix error message

Tiriel · Tiriel · commit 398246f76590 · 2017-12-27T09:52:03.000+01:00
Fix following review. Reinstated assertion on error message, and corrected comment. Leaving continu to break loop and avoid unneeded assertions. Fixes: #17148 Refs: #17148 (comment)
diff --git a/test/parallel/test-require-deps-deprecation.js b/test/parallel/test-require-deps-deprecation.js
@@ -39,14 +39,16 @@ for (const m of deprecatedModules) {
   } catch (err) {}
 }
 
-// Instead of checking require, check that resolve isn't pointing toward
-// /node/deps, as user might already have node installed with acorn in
-// require.resolve range
+// Instead of checking require, check that resolve isn't pointing toward a
+// built-in module, as user might already have node installed with acorn in
+// require.resolve range.
+// Ref: https://github.com/nodejs/node/issues/17148
 for (const m of deps) {
   let path;
   try {
     path = require.resolve(m);
   } catch (err) {
+    assert.ok(err.toString().startsWith('Error: Cannot find module '));
     continue;
   }
   assert.notStrictEqual(path, m);
