adding unit tests

Author: Parth Shah

app/services/helpers.service.js

@@ -165,6 +165,7 @@
 
     function redirectPostLogin(nextParam) {
       // make sure domain is topcoder | dev | qa
+      nextParam = decodeURIComponent(nextParam);
       var re1 = /^(\w+\.)*topcoder(-\w+)*\.com/;
       var re2 = /^\/\w+/;
 
@@ -173,6 +174,7 @@
       } else if (re2.test(nextParam)) {
         $location.url(nextParam);
       } else {
+        console.log($state);
         $state.go('dashboard');
       }
     }

app/services/helpers.service.spec.js

@@ -6,13 +6,25 @@ describe('Helper Service', function() {
       href: "/"
     }
   };
+  var fakeLocation = {
+    url: function(param) {
+      return;
+    }
+  };
+  sinon.spy(fakeLocation, "url");
+  var fakeState = {
+    go: function(name, params) {return;}
+  };
+  sinon.spy(fakeState, "go");
 
   beforeEach(function() {
-    bard.appModule('tc.services', function($provide) {
+    module('tc.services', function($provide) {
       $provide.value('$window', fakeWindow);
+      $provide.value('$state', fakeState);
+      $provide.value('$location', fakeLocation);
     });
 
-    bard.inject(this, 'Helpers');
+    bard.inject(this, 'Helpers', '$state', '$location');
   });
 
   describe("isEmail()", function() {
@@ -30,5 +42,15 @@ describe('Helper Service', function() {
       Helpers.redirectPostLogin(encodeURIComponent("www.topcoder-dev.com"));
       expect(fakeWindow.location.href).to.equal("www.topcoder-dev.com");
     });
+
+    it("should redirect to the next param", function() {
+      Helpers.redirectPostLogin(encodeURIComponent("www.google.com"));
+      expect($state.go).to.have.been.calledWith("dashboard");
+    });
+
+    it("should redirect to the next param", function() {
+      Helpers.redirectPostLogin(encodeURIComponent("/members/test1/"));
+      expect($location.url).to.have.been.calledWith("/members/test1/");
+    });
   });
 });