refactor(eslint-plugin): [prefer-output-readonly] use computed selector (#569)

Author: rafaelss95

packages/eslint-plugin/src/rules/prefer-output-readonly.ts

@@ -1,5 +1,6 @@
 import type { TSESTree } from '@typescript-eslint/experimental-utils';
 import { createESLintRule } from '../utils/create-eslint-rule';
+import { OUTPUT_DECORATOR } from '../utils/selectors';
 
 type Options = [];
 export type MessageIds = 'preferOutputReadonly' | 'suggestAddReadonlyModifier';
@@ -11,22 +12,22 @@ export default createESLintRule<Options, MessageIds>({
     type: 'suggestion',
     docs: {
       description:
-        'Prefer to declare `@Output` as readonly since they are not supposed to be reassigned',
+        'Prefer to declare `@Output` as `readonly` since they are not supposed to be reassigned',
       category: 'Best Practices',
       recommended: false,
       suggestion: true,
     },
     schema: [],
     messages: {
       preferOutputReadonly:
-        'Prefer to declare `@Output` as readonly since they are not supposed to be reassigned',
-      suggestAddReadonlyModifier: 'Add readonly modifier',
+        'Prefer to declare `@Output` as `readonly` since they are not supposed to be reassigned',
+      suggestAddReadonlyModifier: 'Add `readonly` modifier',
     },
   },
   defaultOptions: [],
   create(context) {
     return {
-      'ClassProperty[readonly=undefined] > Decorator[expression.callee.name="Output"]'({
+      [`ClassProperty:not([readonly]) > ${OUTPUT_DECORATOR}`]({
         parent: { key },
       }: TSESTree.Decorator & { parent: TSESTree.ClassProperty }) {
         context.report({

packages/eslint-plugin/tests/rules/prefer-output-readonly.test.ts

@@ -17,6 +17,11 @@ const suggestAddReadonlyModifier: MessageIds = 'suggestAddReadonlyModifier';
 
 ruleTester.run(RULE_NAME, rule, {
   valid: [
+    `
+    class Test {
+      testEmitter = new EventEmitter<string>();
+    }
+    `,
     `
     class Test {
       @Output() readonly testEmitter = new EventEmitter<string>();