NodeJS and Deno set codes to deprecated errors

zloirock · Nov 17, 2021 · 3c83f54 · 3c83f54
1 parent 00b448d
commit 3c83f54
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/tests/pure/web.dom-exception.constructor.js b/tests/pure/web.dom-exception.constructor.js
@@ -1,4 +1,4 @@
-import { DESCRIPTORS } from '../helpers/constants';
+import { DESCRIPTORS, NODE } from '../helpers/constants';
 import DOMException from 'core-js-pure/stable/dom-exception';
 import Symbol from 'core-js-pure/es/symbol';
 
@@ -58,7 +58,8 @@ QUnit.test('DOMException', assert => {
     assert.same(error.message, '42', `new DOMException({}, "${ name }").message`);
     assert.same(error.name, name, `new DOMException({}, "${ name }").name`);
     if (errors[name].m) assert.same(error.code, errors[name].c, `new DOMException({}, "${ name }").code`);
-    else assert.same(error.code, 0, `new DOMException({}, "${ name }").code`);
+    // NodeJS and Deno set codes to deprecated errors
+    else if (!NODE) assert.same(error.code, 0, `new DOMException({}, "${ name }").code`);
     assert.same(String(error), `${ name }: 42`, `String(new DOMException({}, "${ name }"))`); // Safari 10.1 bug
     if (HAS_STACK) assert.ok('stack' in error, `'stack' in new DOMException({}, "${ name }")`);
 

diff --git a/tests/tests/web.dom-exception.constructor.js b/tests/tests/web.dom-exception.constructor.js
@@ -1,4 +1,4 @@
-import { DESCRIPTORS } from '../helpers/constants';
+import { DESCRIPTORS, NODE } from '../helpers/constants';
 
 const errors = {
   IndexSizeError: { s: 'INDEX_SIZE_ERR', c: 1, m: 1 },
@@ -57,7 +57,8 @@ QUnit.test('DOMException', assert => {
     assert.same(error.message, '42', `new DOMException({}, "${ name }").message`);
     assert.same(error.name, name, `new DOMException({}, "${ name }").name`);
     if (errors[name].m) assert.same(error.code, errors[name].c, `new DOMException({}, "${ name }").code`);
-    else assert.same(error.code, 0, `new DOMException({}, "${ name }").code`);
+    // NodeJS and Deno set codes to deprecated errors
+    else if (!NODE) assert.same(error.code, 0, `new DOMException({}, "${ name }").code`);
     assert.same(String(error), `${ name }: 42`, `String(new DOMException({}, "${ name }"))`); // Safari 10.1 bug
     if (HAS_STACK) assert.ok('stack' in error, `'stack' in new DOMException({}, "${ name }")`);