Merge pull request #2667 from johanrd/night_fix/template-no-class-bindings

Post-merge-review: Fix `template-no-class-bindings` false positive in GJS/GTS

Author: NullVoxPopuli

lib/rules/template-no-class-bindings.js

@@ -23,6 +23,11 @@ module.exports = {
   },
 
   create(context) {
+    const isStrictMode = context.filename.endsWith('.gjs') || context.filename.endsWith('.gts');
+    if (isStrictMode) {
+      return {};
+    }
+
     const FORBIDDEN_ATTR_NAMES = new Set([
       'classBinding',
       '@classBinding',

tests/lib/rules/template-no-class-bindings.js

@@ -13,6 +13,15 @@ ruleTester.run('template-no-class-bindings', rule, {
     '<template>{{true}}</template>',
     '<template>{{"hehe"}}</template>',
     '<template><div class="foo"></div></template>',
+    // Rule is HBS-only: @classBinding in GJS/GTS may be a legitimate component argument
+    {
+      filename: 'test.gjs',
+      code: '<template><SomeThing @classBinding="lol:wat" /></template>',
+    },
+    {
+      filename: 'test.gts',
+      code: '<template>{{some-thing classNameBindings="lol:foo:bar"}}</template>',
+    },
   ],
   invalid: [
     {