test: fixed test case for v8

nodejs · Aug 2, 2017 · d17a3dc · d17a3dc
1 parent f9a5056
commit d17a3dc
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 19 deletions.
diff --git a/test/message/nexttick_throw.v8.out b/test/message/nexttick_throw.v8.out
@@ -2,11 +2,10 @@
 *test*message*nexttick_throw.js:*
 *undefined_reference_error_maker;
 *^
-ReferenceError: *undefined_reference_error_maker*
+ReferenceError: *undefined_reference_error_maker is not defined*
 *at *test*message*nexttick_throw.js:*:*
 *at _combinedTickCallback (internal/process/next_tick.js:*:*)
 *at *_tickCallback (internal/process/next_tick.js:*:*)
 *at Module.runMain (module.js:*:*)
-*at run (bootstrap_node.js:*:*)
 *at startup (bootstrap_node.js:*:*)
 *at *bootstrap_node.js:*:*
diff --git a/test/parallel/test-vm-new-script-new-context.js b/test/parallel/test-vm-new-script-new-context.js
@@ -42,14 +42,16 @@ const Script = require('vm').Script;
 }
 
 {
-  assert.throws(
-    function() {
-      script.runInNewContext();
-    },
-    common.engineSpecificMessage({
-      v8: /^ReferenceError: foo is not defined$/,
-      chakracore: /ReferenceError: Use before declaration/
-    }));
+  const script = new Script('foo.bar = 5;');
+  assert.throws(function() {
+    script.runInNewContext();
+  }, common.engineSpecificMessage({
+    v8: /^ReferenceError: foo is not defined$/,
+    chakracore: /^TypeError: Unable to set property 'bar' of undefined or null reference$/
+  }));
+}
+
+{
   global.hello = 5;
   const script = new Script('hello = 2');
   script.runInNewContext();
@@ -95,14 +97,12 @@ const Script = require('vm').Script;
   script.runInNewContext({ f: f });
   assert.strictEqual(f.a, 2);
 
-  assert.throws(
-    function() {
-      script.runInNewContext();
-    },
-    common.engineSpecificMessage({
-      v8: /^ReferenceError: f is not defined$/,
-      chakracore: /'a' of undefined or null/
-    }));
+  assert.throws(function() {
+    script.runInNewContext();
+  }, common.engineSpecificMessage({
+    v8: /^ReferenceError: f is not defined$/,
+    chakracore: /^TypeError: Unable to set property 'a' of undefined or null reference$/
+  }));
 }
 
 {
@@ -111,6 +111,6 @@ const Script = require('vm').Script;
     script.runInNewContext.call('\'hello\';');
   }, common.engineSpecificMessage({
     v8: /^TypeError: this\.runInContext is not a function$/,
-    chakracore: /TypeError: Object doesn't support property or method 'runInContext'/
+    chakracore: /^TypeError: Object doesn't support property or method 'runInContext'$/
   }));
 }