fix(rules): null access in the no-angular-classes rule

Safeguard against null access when using gulp-eslint runner.
alecxe · Jun 17, 2016 · 0052c77 · 0052c77
2 parents 8a54521 + fa7f65f
commit 0052c77
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/lib/rules/no-angular-classes.js b/lib/rules/no-angular-classes.js
@@ -21,7 +21,7 @@ module.exports = function (context) {
 
   return {
     'CallExpression': function (node) {
-      if (node.arguments) {
+      if (node.arguments && node.arguments.length && node.arguments[0].hasOwnProperty('value')) {
         if (isCSSLocator(node)) {
           for (var i = 0; i < prohibitedClasses.length; i++) {
             if (node.arguments[0].value.indexOf(prohibitedClasses[i]) >= 0) {

diff --git a/lib/rules/use-angular-locators.js b/lib/rules/use-angular-locators.js
@@ -16,7 +16,7 @@ module.exports = function (context) {
 
   return {
     'CallExpression': function (node) {
-      if (node.arguments) {
+      if (node.arguments && node.arguments.length && node.arguments[0].hasOwnProperty('value')) {
         if (isCSSLocator(node)) {
           Object.keys(attributeToLocatorMap).forEach(function (key) {
             if (node.arguments[0].value.indexOf(key) >= 0) {