Merge pull request #2496 from johanrd/fix/2408

Post-merge review of #2408 (`template-no-autofocus-attribute`)

Author: NullVoxPopuli

lib/rules/template-no-autofocus-attribute.js

@@ -27,7 +27,6 @@ module.exports = {
             node: autofocusAttr,
             messageId: 'noAutofocus',
             fix(fixer) {
-              // Remove the attribute including preceding whitespace
               const sourceCode = context.sourceCode;
               const text = sourceCode.getText();
               const attrStart = autofocusAttr.range[0];
@@ -43,6 +42,19 @@ module.exports = {
           });
         }
       },
+
+      GlimmerMustacheStatement(node) {
+        if (!node.hash || !node.hash.pairs) {
+          return;
+        }
+        const autofocusPair = node.hash.pairs.find((pair) => pair.key === 'autofocus');
+        if (autofocusPair) {
+          context.report({
+            node: autofocusPair,
+            messageId: 'noAutofocus',
+          });
+        }
+      },
     };
   },
 };

tests/lib/rules/template-no-autofocus-attribute.js

@@ -53,5 +53,75 @@ ruleTester.run('template-no-autofocus-attribute', rule, {
         },
       ],
     },
+    {
+      code: `<template>
+        {{input type="text" autofocus=true}}
+      </template>`,
+      output: null,
+      errors: [
+        {
+          messageId: 'noAutofocus',
+          type: 'GlimmerHashPair',
+        },
+      ],
+    },
+    {
+      code: `<template>
+        {{component "input" type="text" autofocus=true}}
+      </template>`,
+      output: null,
+      errors: [
+        {
+          messageId: 'noAutofocus',
+          type: 'GlimmerHashPair',
+        },
+      ],
+    },
+    {
+      code: `<template>
+        <div autofocus>
+        </div>
+      </template>`,
+      output: `<template>
+        <div>
+        </div>
+      </template>`,
+      errors: [
+        {
+          messageId: 'noAutofocus',
+          type: 'GlimmerAttrNode',
+        },
+      ],
+    },
+    {
+      code: `<template>
+        <h1 autofocus>
+        </h1>
+      </template>`,
+      output: `<template>
+        <h1>
+        </h1>
+      </template>`,
+      errors: [
+        {
+          messageId: 'noAutofocus',
+          type: 'GlimmerAttrNode',
+        },
+      ],
+    },
+    {
+      code: `<template>
+        <input autofocus="autofocus" />
+      </template>`,
+      output: `<template>
+        <input />
+      </template>`,
+      errors: [
+        {
+          messageId: 'noAutofocus',
+          type: 'GlimmerAttrNode',
+        },
+      ],
+    },
   ],
 });