fix: no-dedupe with setters or getters (#435)

gajus · Nov 12, 2019 · da05064 · da05064
1 parent 6f71442
commit da05064
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/src/utilities/getParameterName.js b/src/utilities/getParameterName.js
@@ -30,6 +30,10 @@ export default (identifierNode, context) => {
       tokenIndex++;
     }
 
+    if (identifierNode.kind === 'set' || identifierNode.kind === 'get') {
+      tokenIndex++;
+    }
+
     return context.getSourceCode().getFirstToken(identifierNode, tokenIndex).value;
   }
 

diff --git a/tests/rules/assertions/noDupeKeys.js b/tests/rules/assertions/noDupeKeys.js
@@ -111,5 +111,8 @@ export default {
     {
       code: 'type a = { b: <C>(config: { ...C, key: string}) => C }',
     },
+    {
+      code: 'export interface Foo { get foo(): boolean; get bar(): string; }',
+    },
   ],
 };