Warn on use of deprecated class literal

minichate · Nov 29, 2015 · f4cca48 · f4cca48
1 parent b00015d
commit f4cca48
Show file tree

Hide file tree

Showing 4 changed files with 12 additions and 16 deletions.
diff --git a/lib/helpers/ember-core.js b/lib/helpers/ember-core.js
@@ -38,16 +38,6 @@ module.exports.prototype.findEmberClass = function(className) {
         return;
       }
 
-      var nextToken = _this.file.getTokenByRangeStart(node.range[1]);
-      if (nextToken.type !== 'Punctuator' || nextToken.value !== '.') {
-        return;
-      }
-
-      var following = _this.file.getTokenByRangeStart(nextToken.range[1]);
-      if (following.value !== 'extend' && following.value !== 'create') {
-        return;
-      }
-
       nodes.push(node);
     }
   });

diff --git a/test/lib/rules/disallow-arraycontroller.js b/test/lib/rules/disallow-arraycontroller.js
@@ -32,12 +32,14 @@ describe('lib/rules/disallow-arraycontroller', function () {
             });
           }
         }, {
-          it: 'should not report',
+          it: 'should report',
+          errors: 1,
           code: function() {
             var foo = Ember.ArrayController;
           }
         }, {
-          it: 'should not report',
+          it: 'should report',
+          errors: 1,
           code: function() {
             Ember.ArrayController.foo();
           }

diff --git a/test/lib/rules/disallow-coreview.js b/test/lib/rules/disallow-coreview.js
@@ -32,12 +32,14 @@ describe('lib/rules/disallow-coreview', function () {
             });
           }
         }, {
-          it: 'should not report',
+          it: 'should report',
+          errors: 1,
           code: function() {
             var foo = Ember.CoreView;
           }
         }, {
-          it: 'should not report',
+          it: 'should report',
+          errors: 1,
           code: function() {
             Ember.CoreView.foo();
           }

diff --git a/test/lib/rules/disallow-objectcontroller.js b/test/lib/rules/disallow-objectcontroller.js
@@ -32,12 +32,14 @@ describe('lib/rules/disallow-objectcontroller', function () {
             });
           }
         }, {
-          it: 'should not report',
+          it: 'should report',
+          errors: 1,
           code: function() {
             var foo = Ember.ObjectController;
           }
         }, {
-          it: 'should not report',
+          it: 'should report',
+          errors: 1,
           code: function() {
             Ember.ObjectController.foo();
           }