Merge pull request #482 from kingcody/fix/jshint

fix(app-jshint): improve jshint usage

Author: DaftMonk

app/templates/Gruntfile.js

@@ -167,7 +167,16 @@ module.exports = function (grunt) {
         options: {
           jshintrc: 'server/.jshintrc'
         },
-        src: [ 'server/{,*/}*.js']
+        src: [
+          'server/**/*.js',
+          '!server/**/*.spec.js'
+        ]
+      },
+      serverTest: {
+        options: {
+          jshintrc: 'server/.jshintrc-spec'
+        },
+        src: ['server/**/*.spec.js']
       },
       all: [
         '<%%= yeoman.client %>/{app,components}/**/*.js',

app/templates/client/app/account(auth)/settings/settings.controller(coffee).coffee

@@ -8,9 +8,9 @@ angular.module '<%= scriptAppName %>'
   $scope.email = {}
 
   getEmail = (user) ->
-    return [null, null] unless $scope.user.credentials.length
+    return [null, null] unless user.credentials.length
 
-    for c in $scope.user.credentials when c.type is 'email'
+    for c in user.credentials when c.type is 'email'
       return [c.value, c.confirmed]
 
     [null, null]

app/templates/client/app/account(auth)/settings/settings.controller(js).js

@@ -8,13 +8,15 @@ angular.module('<%= scriptAppName %>')
     $scope.email = {};
 
     var getEmail = function(user) {
-      if (!$scope.user.credentials.length) {
+      if (!user.credentials.length) {
         return null;
       }
 
-      for(var i in $scope.user.credentials) {
-        var c = $scope.user.credentials[i];
-        if(c.type==='email') return [c.value, c.confirmed];
+      for(var i in user.credentials) {
+        var c = user.credentials[i];
+        if (c.type === 'email') {
+          return [c.value, c.confirmed];
+        }
       }
     };
 
@@ -36,7 +38,7 @@ angular.module('<%= scriptAppName %>')
           $scope.message = '';
         });
       }
-    }
+    };
 
     $scope.changePassword = function() {
       $scope.pwd.submitted = true;

app/templates/client/app/admin(auth)/admin.controller(js).js

@@ -8,6 +8,6 @@ angular.module('<%= scriptAppName %>')
 
     $scope.delete = <% if(filters.uibootstrap) { %>Modal.confirm.delete(<% } %>function(user) {
       User.remove({ id: user._id });
-      _.remove($scope.users, user)
+      _.remove($scope.users, user);
     }<% if(filters.uibootstrap) { %>)<% } %>;
   });

app/templates/client/app/main/main.controller(coffee).coffee

@@ -1,7 +1,7 @@
 'use strict'
 
 angular.module '<%= scriptAppName %>'
-.controller 'MainCtrl', ($scope, $http<% if(filters.socketio) { %>, socket<% } %><% if(filters.uibootstrap && filters.mongoose) { %>, Modal<% } %>) ->
+.controller 'MainCtrl', ($scope, $http<% if(filters.socketio) { %>, socket<% } %>) ->
   $scope.awesomeThings = []
 
   $http.get('/api/things').success (awesomeThings) ->
@@ -19,4 +19,4 @@ angular.module '<%= scriptAppName %>'
     $http.delete '/api/things/' + thing._id<% } %><% if(filters.socketio) { %>
 
   $scope.$on '$destroy', ->
-    socket.unsyncUpdates 'thing'<% } %>
+    socket.unsyncUpdates 'thing'<% } %>

app/templates/client/app/main/main.controller(js).js

@@ -1,7 +1,7 @@
 'use strict';
 
 angular.module('<%= scriptAppName %>')
-  .controller('MainCtrl', function ($scope, $http<% if(filters.socketio) { %>, socket<% } %><% if(filters.uibootstrap && filters.mongoose) { %>, Modal<% } %>) {
+  .controller('MainCtrl', function ($scope, $http<% if(filters.socketio) { %>, socket<% } %>) {
     $scope.awesomeThings = [];
 
     $http.get('/api/things').success(function(awesomeThings) {
@@ -24,4 +24,4 @@ angular.module('<%= scriptAppName %>')
     $scope.$on('$destroy', function () {
       socket.unsyncUpdates('thing');
     });<% } %>
-  });
+  });

app/templates/server/.jshintrc

@@ -4,12 +4,12 @@
   "bitwise": true,
   "eqeqeq": true,
   "immed": true,
-  "latedef": true,
+  "latedef": "nofunc",
   "newcap": true,
   "noarg": true,
   "regexp": true,
   "undef": true,
   "smarttabs": true,
   "asi": true,
   "debug": true
-}
+}

app/templates/server/.jshintrc-spec

@@ -0,0 +1,11 @@
+{
+  "extends": ".jshintrc",
+  "globals": {
+    "describe": true,
+    "it": true,
+    "before": true,
+    "beforeEach": true,
+    "after": true,
+    "afterEach": true
+  }
+}

app/templates/server/api/thing/thing.spec.js

@@ -17,4 +17,4 @@ describe('GET /api/things', function() {
         done();
       });
   });
-});
+});

app/templates/server/api/user(auth)/user.model.js

@@ -81,7 +81,7 @@ UserSchema
   // returns only first found email
   // TODO: in case of multiple emails, should prioritize confirmed ones
   return this.credentials.filter(function(c) {
-    return c.type==='email';
+    return c.type === 'email';
 
   })[0].value;
 });
@@ -90,21 +90,21 @@ UserSchema
 .virtual('emails')
 .get(function() {
   return this.credentials
-    .filter(function(c) { return c.type==='email'; })
+    .filter(function(c) { return c.type === 'email'; })
     .map(function(c) { return c.value; });
 
 });
 
 UserSchema
 .pre('save', function(next) {<% if (filters.oauth) { %>
   if(!this.localEnabled) {
-    if (Object.keys(this.strategies).length===0) {
+    if (Object.keys(this.strategies).length === 0) {
       return next(new Error('No connected accounts'));
     }
     return next();
   }<% } %>
 
-  mongoose.models['User']<% if (filters.oauth) { %>
+  mongoose.models.User<% if (filters.oauth) { %>
     .find({ localEnabled:true })<% } %>
     .where('credentials.type').equals('email')
     .where('credentials.value').equals(this.email)
@@ -132,15 +132,15 @@ UserSchema.methods = {
   },
   confirm: function(emailOrPhone, cb) {
     this.credentials.forEach(function(c) {
-      if (c.value===emailOrPhone) {
+      if (c.value === emailOrPhone) {
         c.confirmed = true;
       }
     });
     this.save(cb);
   },
   changeEmail: function(oldEmail, newEmail, cb) {
     this.credentials.forEach(function(c) {
-      if (c.value===oldEmail) {
+      if (c.value === oldEmail) {
         c.value = newEmail;
         c.confirmed = false;
       }
@@ -177,14 +177,14 @@ UserSchema.statics = {
     var dataFormatted;
     dataFormatted = [];
 
-    if (data.email != null) {
+    if (data.email !== null) {
       dataFormatted.push({
         'credentials.type': 'email',
         'credentials.value': data.email
       });
     }
 
-    if (data.phone != null) {
+    if (data.phone !== null) {
       dataFormatted.push({
         'credentials.type': 'phone',
         'credentials.value': data.phone
@@ -195,11 +195,11 @@ UserSchema.statics = {
     .or(dataFormatted)
     .exec(function(err, users) {
       if (err) return cb(err);
-      if (users.length===0) return cb(null, null);
+      if (users.length === 0) return cb(null, null);
 
       cb(null, users);
     });
   }<% } %>
 };
 
-module.exports = mongoose.model('User', UserSchema);
+module.exports = mongoose.model('User', UserSchema);