v8: migrate setFlagsFromString to internal/errors

PR-URL: https://github.com/nodejs/node/pull/16535
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>

1 files changed, 11 insertions, 6 deletions

diff --git a/test/parallel/test-v8-flag-type-check.js b/test/parallel/test-v8-flag-type-check.js
index 144e8f7a2e..67a34b51f2 100644
--- a/test/parallel/test-v8-flag-type-check.js
+++ b/test/parallel/test-v8-flag-type-check.js
@@ -1,9 +1,14 @@
 'use strict';
-require('../common');
-const assert = require('assert');
+const common = require('../common');
 const v8 = require('v8');
 
-assert.throws(function() { v8.setFlagsFromString(1); },
-              /^TypeError: v8 flag must be a string$/);
-assert.throws(function() { v8.setFlagsFromString(); },
-              /^TypeError: v8 flag is required$/);
+[ 1, undefined ].forEach((i) => {
+  common.expectsError(
+    () => v8.setFlagsFromString(i),
+    {
+      code: 'ERR_INVALID_ARG_TYPE',
+      type: TypeError,
+      message: 'The "flags" argument must be of type string'
+    }
+  );
+});