Merge pull request angular-ui#129 from thgreasi/master

Linting abstractions, fixes and improvements.

Author: thgreasi

gruntFile.js

@@ -85,27 +85,8 @@ module.exports = function(grunt) {
         options: { jshintrc: '.jshintrc' }
       },
       test: {
-        files:{ src : [ 'test/*.spec.js', 'gruntFile.js'] },
-        options: grunt.util._.extend({}, grunt.file.readJSON('.jshintrc'), {
-          node: true,
-          globals: {
-            angular: false,
-            inject: false,
-            '$': false,
-
-            jasmine: false,
-            afterEach: false,
-            beforeEach: false,
-            ddescribe: false,
-            describe: false,
-            expect: false,
-            iit: false,
-            it: false,
-            spyOn: false,
-            xdescribe: false,
-            xit: false
-          }
-        })
+        files:{ src : [ 'test/*.js', 'gruntFile.js'] },
+        options: grunt.util._.extend({}, grunt.file.readJSON('.jshintrc'), grunt.file.readJSON('test/.jshintrc'))
       }
     },
 

test/.jshintrc

@@ -0,0 +1,20 @@
+{
+  "node": true,
+  "globals": {
+    "angular": false,
+    "inject": false,
+    "$": false,
+
+    "jasmine": false,
+    "afterEach": false,
+    "beforeEach": false,
+    "ddescribe": false,
+    "describe": false,
+    "expect": false,
+    "iit": false,
+    "it": false,
+    "spyOn": false,
+    "xdescribe": false,
+    "xit": false
+  }
+}

test/karma.conf.js

@@ -1,6 +1,8 @@
 // Karma configuration
 // Generated on Sat Dec 28 2013 20:27:08 GMT+0100 (CET)
 
+'use strict';
+
 module.exports = function(config) {
   config.set({
 

test/sortable.test-directives.js

@@ -8,10 +8,10 @@ angular.module('ui.sortable.testDirectives', [])
         scope: true,
         require: '?ngModel',
         template: '<div>Directive: <span class="itemContent" ng-bind="text"></span> !!!</div>',
-        link: function(scope, element, attrs, ngModel) {
+        link: function(scope, element, attrs) {
           scope.$watch(attrs.ngModel, function(value) {
-              scope.text = value;
-            });
+            scope.text = value;
+          });
         }
       };
     }
@@ -23,10 +23,10 @@ angular.module('ui.sortable.testDirectives', [])
         scope: true,
         require: '?ngModel',
         template: '<div>$destroy(able) Directive: <span class="itemContent" ng-bind="text"></span> !!!</div>',
-        link: function(scope, element, attrs, ngModel) {
+        link: function(scope, element, attrs) {
           scope.$watch(attrs.ngModel, function(value) {
-              scope.text = value;
-            });
+            scope.text = value;
+          });
 
           element.bind('$destroy', function() {
             element.html('');