Remove Math and JSON from no-new-wrappers

eslint · Mar 24, 2020 · 792f8a3 · 792f8a3
1 parent ca3572f
commit 792f8a3
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 21 deletions.
diff --git a/lib/rules/no-new-wrappers.js b/lib/rules/no-new-wrappers.js
@@ -32,7 +32,7 @@ module.exports = {
         return {
 
             NewExpression(node) {
-                const wrapperObjects = ["String", "Number", "Boolean", "Math", "JSON"];
+                const wrapperObjects = ["String", "Number", "Boolean"];
 
                 if (wrapperObjects.indexOf(node.callee.name) > -1) {
                     context.report({

diff --git a/tests/lib/rules/no-new-wrappers.js b/tests/lib/rules/no-new-wrappers.js
@@ -53,26 +53,6 @@ ruleTester.run("no-new-wrappers", rule, {
                 },
                 type: "NewExpression"
             }]
-        },
-        {
-            code: "var a = new Math();",
-            errors: [{
-                messageId: "noConstructor",
-                data: {
-                    fn: "Math"
-                },
-                type: "NewExpression"
-            }]
-        },
-        {
-            code: "var a = new JSON({ myProp: 10 });",
-            errors: [{
-                messageId: "noConstructor",
-                data: {
-                    fn: "JSON"
-                },
-                type: "NewExpression"
-            }]
         }
     ]
 });