fixing unit tests

Author: Parth Shah

app/services/helpers.service.js

@@ -166,15 +166,14 @@
     function redirectPostLogin(nextParam) {
       // make sure domain is topcoder | dev | qa
       nextParam = decodeURIComponent(nextParam);
-      var re1 = /^(\w+\.)*topcoder(-\w+)*\.com/;
+      var re1 = /^(https?:\/\/)*(\w+\.)*topcoder(-\w+)*\.com/;
       var re2 = /^\/\w+/;
 
       if (re1.test(nextParam)) {
-        $window.location.href = decodeURIComponent(nextParam);
+        $window.location.href = nextParam;
       } else if (re2.test(nextParam)) {
         $location.url(nextParam);
       } else {
-        console.log($state);
         $state.go('dashboard');
       }
     }

app/services/helpers.service.spec.js

@@ -6,6 +6,7 @@ describe('Helper Service', function() {
       href: "/"
     }
   };
+  // sinon.spy(fakeWindow.location, "href");
   var fakeLocation = {
     url: function(param) {
       return;
@@ -39,8 +40,8 @@ describe('Helper Service', function() {
 
   describe("redirectPostLogin()", function() {
     it("should redirect to the next param", function() {
-      Helpers.redirectPostLogin(encodeURIComponent("www.topcoder-dev.com"));
-      expect(fakeWindow.location.href).to.equal("www.topcoder-dev.com");
+      Helpers.redirectPostLogin(encodeURIComponent("http://www.topcoder-dev.com"));
+      expect(fakeWindow.location.href).to.equal("http://www.topcoder-dev.com");
     });
 
     it("should redirect to the next param", function() {