tools: fix bug in prefer-primordials lint rule

PR-URL: #46659 Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
nodejs · Feb 20, 2023 · 1cd22e7 · 1cd22e7
1 parent b4deb2f
commit 1cd22e7
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/test/parallel/test-eslint-prefer-primordials.js b/test/parallel/test-eslint-prefer-primordials.js
@@ -57,7 +57,9 @@ new RuleTester({
       {
         code: `
           const { ObjectDefineProperty, Symbol } = primordials;
-          ObjectDefineProperty(o, Symbol.toStringTag, { value: "o" })
+          ObjectDefineProperty(o, Symbol.toStringTag, { value: "o" });
+          const val = Symbol.toStringTag;
+          const { toStringTag } = Symbol;
         `,
         options: [{ name: 'Symbol', ignore: ['toStringTag'] }]
       },

diff --git a/tools/eslint-rules/prefer-primordials.js b/tools/eslint-rules/prefer-primordials.js
@@ -114,7 +114,7 @@ module.exports = {
           node,
         );
         const parentName = parent?.name;
-        if (!isTarget(nameMap, name) && !isTarget(nameMap, parentName)) {
+        if (!isTarget(nameMap, name) && (!isTarget(nameMap, parentName) || isIgnored(nameMap, parentName, name))) {
           return;
         }