Skip to content

Commit 105bff9

Browse files
committedSep 1, 2021
test: adapt test-v8-flags to V8 9.3
%_IsSmi was removed. %IsSmi can be used instead. Refs: v8/v8@863a2d6 PR-URL: #39469 Backport-PR-URL: #39947 Reviewed-By: Jiawen Geng <technicalcute@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
1 parent beca890 commit 105bff9

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed
 

‎test/parallel/test-v8-flags.js

+4-4
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@ const vm = require('vm');
88
// Specifically here, V8 may cache compiled scripts between the flip of the
99
// flag. We use a different script each time to work around this problem.
1010
v8.setFlagsFromString('--allow_natives_syntax');
11-
assert(eval('%_IsSmi(42)'));
12-
assert(vm.runInThisContext('%_IsSmi(43)'));
11+
assert(eval('%IsSmi(42)'));
12+
assert(vm.runInThisContext('%IsSmi(43)'));
1313

1414
v8.setFlagsFromString('--noallow_natives_syntax');
15-
assert.throws(function() { eval('%_IsSmi(44)'); },
15+
assert.throws(function() { eval('%IsSmi(44)'); },
1616
/^SyntaxError: Unexpected token '%'$/);
17-
assert.throws(function() { vm.runInThisContext('%_IsSmi(45)'); },
17+
assert.throws(function() { vm.runInThisContext('%IsSmi(45)'); },
1818
/^SyntaxError: Unexpected token '%'$/);

0 commit comments

Comments
 (0)