Added unit tests for nav.js

app/js/app.js

@@ -1,7 +1,8 @@
-'use strict';
-
+/*global ProjectCreateCtrl ProjectEditCtrl ProjectListCtrl */
 angular.module('main', ['resProject']).
     config(function ($routeProvider) {
+        'use strict';
+
         $routeProvider.
             when('/', { templateUrl: 'partials/home.html' }).
             when('/about', { templateUrl: 'partials/about.html' }).

app/js/ctrl/nav.js

@@ -1,19 +1,18 @@
-'use strict';
+function NavCtrl($scope, $location) {
+    'use strict';
 
-function NavCtrl($scope, $location) {
     $scope.isUrl = function (path, className) {
-        if ($location.path() == path) {
+        if ($location.path() === path) {
             return className;
-        } else {
-            return "";
         }
-    }
+        return "";
+    };
 
     $scope.hasUrl = function (path, className) {
-        if ($location.path().substring(0, path.length) == path) {
+        if ($location.path().search(path) === 0) {
             return className;
-        } else {
-            return "";
         }
-    }
-}
+        return "";
+    };
+}
+NavCtrl.$inject = ['$scope', '$location'];

app/js/ctrl/project/create.js

@@ -1,7 +1,9 @@
 function ProjectCreateCtrl($scope, $location, Project) {
+    'use strict';
+
     $scope.save = function () {
         Project.save($scope.project, function (project) {
             $location.path('/project');
         });
-    }
+    };
 }

app/js/ctrl/project/edit.js

@@ -1,4 +1,6 @@
 function ProjectEditCtrl($scope, $location, $routeParams, Project) {
+    'use strict';
+
     var self = this;
 
     Project.get({ id: $routeParams.projectId }, function (project) {

app/js/ctrl/project/list.js

@@ -1,3 +1,5 @@
 function ProjectListCtrl($scope, Project) {
+    'use strict';
+
     $scope.projects = Project.query();
 }

app/js/ctrl/signin.js

@@ -1,5 +1,7 @@
 function SignInCtrl($scope) {
-    $scope.signIn = function() {
+    'use strict';
+
+    $scope.signIn = function () {
         alert('Sign in of ' + $scope.name);
     };
 }

app/js/filters.js

@@ -3,8 +3,8 @@
 /* Filters */
 
 angular.module('myApp.filters', []).
-  filter('interpolate', ['version', function(version) {
-    return function(text) {
-      return String(text).replace(/\%VERSION\%/mg, version);
-    }
+  filter('interpolate', ['version', function (version) {
+      return function (text) {
+          return String(text).replace(/\%VERSION\%/mg, version);
+      };
   }]);

test/unit/ctrl/navSpec.js

@@ -1,2 +1,37 @@
-'use strict';
+/*global beforeEach describe expect inject it NavCtrl */
+describe('NavCtrl', function () {
+    'use strict';
 
+    var controller, path, location, scope;
+
+    beforeEach(inject(function ($rootScope, $controller) {
+        path = undefined;
+        location = {
+            path: function () {
+                return path;
+            }
+        };
+        scope = $rootScope.$new();
+        controller = $controller(NavCtrl, { $scope: scope, $location: location });
+    }));
+
+    it('Should return an empty string when url does not match.', function () {
+        path = 'url1';
+        expect(scope.isUrl('url2', 'className1')).toEqual('');
+    });
+
+    it('Should return the className when url does match.', function () {
+        path = 'url1';
+        expect(scope.isUrl('url1', 'className1')).toEqual('className1');
+    });
+
+    it('Should return an empty string when current url does not start with the supplied url.', function () {
+        path = 'url1-url';
+        expect(scope.hasUrl('url2', 'className1')).toEqual('');
+    });
+
+    it('Should return the className when current url starts with the supplied url.', function () {
+        path = 'url1-url';
+        expect(scope.hasUrl('url1', 'className1')).toEqual('className1');
+    });
+});

test/unit/ctrl/project/listSpec.js

@@ -0,0 +1,14 @@
+describe('ProjectListCtrl', function () {
+    'use strict';
+
+    var controller, scope;
+
+    beforeEach(inject(function ($rootScope, $controller, $httpBackend) {
+        scope = $rootScope.$new()
+        controller = $controller(ProjectListCtrl, { $scope: scope, Project: Project });
+    }));
+
+    //it('Should query for all projects and assign it to projects.', function () {
+
+    //});
+});