davazp
diff --git a/‎packages/eslint-plugin-typescript/lib/rules/class-name-casing.js
Lines changed: 2 additions & 7 deletions b/‎packages/eslint-plugin-typescript/lib/rules/class-name-casing.js
Lines changed: 2 additions & 7 deletions
diff --git a/‎packages/eslint-plugin-typescript/lib/rules/indent.js
Lines changed: 0 additions & 1 deletion b/‎packages/eslint-plugin-typescript/lib/rules/indent.js
Lines changed: 0 additions & 1 deletion
diff --git a/‎packages/eslint-plugin-typescript/tests/lib/eslint-rules/no-shadow.js
Lines changed: 5 additions & 0 deletions b/‎packages/eslint-plugin-typescript/tests/lib/eslint-rules/no-shadow.js
Lines changed: 5 additions & 0 deletions
diff --git a/‎packages/eslint-plugin-typescript/tests/lib/eslint-rules/no-undef.js
Lines changed: 5 additions & 0 deletions b/‎packages/eslint-plugin-typescript/tests/lib/eslint-rules/no-undef.js
Lines changed: 5 additions & 0 deletions
diff --git a/‎packages/eslint-plugin-typescript/tests/lib/rules/indent.js
Lines changed: 2 additions & 2 deletions b/‎packages/eslint-plugin-typescript/tests/lib/rules/indent.js
Lines changed: 2 additions & 2 deletions
diff --git a/‎packages/typescript-eslint-parser/src/analyze-scope.ts
Lines changed: 1 addition & 4 deletions b/‎packages/typescript-eslint-parser/src/analyze-scope.ts
Lines changed: 1 addition & 4 deletions
diff --git a/‎packages/typescript-eslint-parser/src/visitor-keys.ts
Lines changed: 0 additions & 9 deletions b/‎packages/typescript-eslint-parser/src/visitor-keys.ts
Lines changed: 0 additions & 9 deletions
@@ -54,10 +54,7 @@ module.exports = {
       switch (decl.type) {
         case 'ClassDeclaration':
         case 'ClassExpression':
-          friendlyName = 'Class';
-          break;
-        case 'TSAbstractClassDeclaration':
-          friendlyName = 'Abstract class';
+          friendlyName = decl.abstract ? 'Abstract class' : 'Class';
           break;
         case 'TSInterfaceDeclaration':
           friendlyName = 'Interface';
@@ -81,9 +78,7 @@ module.exports = {
     //----------------------------------------------------------------------
 
     return {
-      'ClassDeclaration, TSInterfaceDeclaration, TSAbstractClassDeclaration, ClassExpression'(
-        node
-      ) {
+      'ClassDeclaration, TSInterfaceDeclaration, ClassExpression'(node) {
         // class expressions (i.e. export default class {}) are OK
         if (node.id && !isPascalCase(node.id.name)) {
           report(node);
 
@@ -29,7 +29,6 @@ const KNOWN_NODES = new Set([
   'TSNullKeyword',
 
   // ts specific nodes we want to support
-  'TSAbstractClassDeclaration',
   'TSAbstractClassProperty',
   'TSAbstractMethodDefinition',
   'TSArrayType',
 
@@ -22,6 +22,11 @@ ruleTester.run('no-shadow', rule, {
     `
 type foo = any;
 function bar(foo: any) {}
+    `,
+    // https://github.com/typescript-eslint/typescript-eslint/issues/20
+    `
+export abstract class Foo {}
+export class FooBar extends Foo {}
     `
   ],
   invalid: []
 
@@ -61,6 +61,11 @@ interface Runnable {
 export type SomeThing = {
     id: string;
 }
+    `,
+    // https://github.com/typescript-eslint/typescript-eslint/issues/20
+    `
+export abstract class Foo {}
+export class FooBar extends Foo {}
     `
   ],
   invalid: []
 
@@ -22,7 +22,7 @@ function nonTsTestCase(example) {
 
 const individualNodeTests = [
   {
-    node: 'TSAbstractClassDeclaration',
+    node: 'ClassDeclaration',
     code: [
       `
 abstract class Foo {
@@ -157,7 +157,7 @@ interface Foo {
     ]
   },
   {
-    node: 'TSEmptyBodyDeclareFunction',
+    node: 'TSDeclareFunction',
     code: [
       `
 declare function foo() : {
 
@@ -202,7 +202,7 @@ class Referencer extends OriginalReferencer {
   /**
    * Override.
    * Visit decorators.
-   * @param {ClassDeclaration|ClassExpression|TSAbstractClassDeclaration} node The class node to visit.
+   * @param {ClassDeclaration|ClassExpression} node The class node to visit.
    * @returns {void}
    */
   visitClass(node: any) {
@@ -709,9 +709,6 @@ class Referencer extends OriginalReferencer {
     this.close(node);
   }
 
-  TSAbstractClassDeclaration(node: any) {
-    this.ClassDeclaration(node);
-  }
   TSAbstractClassProperty(node: any) {
     this.ClassProperty(node);
   }
 
@@ -36,15 +36,6 @@ export const visitorKeys = eslintVisitorKeys.unionWith({
   ClassProperty: ['decorators', 'key', 'typeAnnotation', 'value'],
   Decorator: ['expression'],
   TSAbstractClassProperty: ['decorators', 'key', 'typeAnnotation', 'value'],
-  TSAbstractClassDeclaration: [
-    'decorators',
-    'id',
-    'typeParameters',
-    'superClass',
-    'superTypeParameters',
-    'implements',
-    'body'
-  ],
   TSAbstractKeyword: [],
   TSAbstractMethodDefinition: ['key', 'value'],
   TSAnyKeyword: [],
Original file line number	Diff line number	Diff line change
`@@ -22,6 +22,11 @@ ruleTester.run('no-shadow', rule, {`
`22`	`22`	`
`23`	`23`	`type foo = any;`
`24`	`24`	`function bar(foo: any) {}`
	`25`	+ `,
	`26`	`+ // https://github.com/typescript-eslint/typescript-eslint/issues/20`
	`27`	+ `
	`28`	`+export abstract class Foo {}`
	`29`	`+export class FooBar extends Foo {}`
`25`	`30`	`
`26`	`31`	`],`
`27`	`32`	`invalid: []`
Original file line number	Diff line number	Diff line change
`@@ -61,6 +61,11 @@ interface Runnable {`
`61`	`61`	`export type SomeThing = {`
`62`	`62`	`id: string;`
`63`	`63`	`}`
	`64`	+ `,
	`65`	`+ // https://github.com/typescript-eslint/typescript-eslint/issues/20`
	`66`	+ `
	`67`	`+export abstract class Foo {}`
	`68`	`+export class FooBar extends Foo {}`
`64`	`69`	`
`65`	`70`	`],`
`66`	`71`	`invalid: []`
Original file line number	Diff line number	Diff line change
`@@ -22,7 +22,7 @@ function nonTsTestCase(example) {`
`22`	`22`
`23`	`23`	`const individualNodeTests = [`
`24`	`24`	`{`
`25`		`- node: 'TSAbstractClassDeclaration',`
	`25`	`+ node: 'ClassDeclaration',`
`26`	`26`	`code: [`
`27`	`27`	`
`28`	`28`	`abstract class Foo {`
`@@ -157,7 +157,7 @@ interface Foo {`
`157`	`157`	`]`
`158`	`158`	`},`
`159`	`159`	`{`
`160`		`- node: 'TSEmptyBodyDeclareFunction',`
	`160`	`+ node: 'TSDeclareFunction',`
`161`	`161`	`code: [`
`162`	`162`	`
`163`	`163`	`declare function foo() : {`