[Fix] jsx-no-target-blank: Improve fixer with option allowReferrer

jsx-eslint · Jan 5, 2022 · 180b468 · 180b468
1 parent 151bb2b
commit 180b468
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/lib/rules/jsx-no-target-blank.js b/lib/rules/jsx-no-target-blank.js
@@ -175,6 +175,7 @@ module.exports = {
             || (enforceDynamicLinks === 'always' && hasDynamicLink(node, linkAttribute));
           if (hasDangerousLink && !hasSecureRel(node, allowReferrer, warnOnSpreadAttributes, spreadAttributeIndex)) {
             const messageId = allowReferrer ? 'noTargetBlankWithoutNoopener' : 'noTargetBlankWithoutNoreferrer';
+            const relValue = allowReferrer ? 'noopener' : 'noreferrer';
             report(context, messages[messageId], messageId, {
               node,
               fix(fixer) {
@@ -188,11 +189,11 @@ module.exports = {
                 }
 
                 if (!relAttribute) {
-                  return fixer.insertTextAfter(nodeWithAttrs.attributes.slice(-1)[0], ' rel="noreferrer"');
+                  return fixer.insertTextAfter(nodeWithAttrs.attributes.slice(-1)[0], ` rel="${relValue}"`);
                 }
 
                 if (!relAttribute.value) {
-                  return fixer.insertTextAfter(relAttribute, '="noreferrer"');
+                  return fixer.insertTextAfter(relAttribute, `="${relValue}"`);
                 }
 
                 if (relAttribute.value.type === 'Literal') {

diff --git a/tests/lib/rules/jsx-no-target-blank.js b/tests/lib/rules/jsx-no-target-blank.js
@@ -249,7 +249,13 @@ ruleTester.run('jsx-no-target-blank', rule, {
     },
     {
       code: '<a href="https://example.com/20" target="_blank"></a>',
-      output: '<a href="https://example.com/20" target="_blank" rel="noreferrer"></a>',
+      output: '<a href="https://example.com/20" target="_blank" rel="noopener"></a>',
+      options: [{ allowReferrer: true }],
+      errors: [{ messageId: 'noTargetBlankWithoutNoopener' }],
+    },
+    {
+      code: '<a href="https://example.com/20" target="_blank" rel></a>',
+      output: '<a href="https://example.com/20" target="_blank" rel="noopener"></a>',
       options: [{ allowReferrer: true }],
       errors: [{ messageId: 'noTargetBlankWithoutNoopener' }],
     },