aboutsummaryrefslogtreecommitdiffstats
path: root/test/ng/locationSpec.js
diff options
context:
space:
mode:
authorIgor Minar2014-01-09 23:16:06 -0800
committerIgor Minar2014-01-09 23:47:35 -0800
commit5a4145fe16bb61720843ae938b789e32dbbfdb7d (patch)
tree735c964f23bf1d56a5c7dfd96a661ba31ab3ad5b /test/ng/locationSpec.js
parent039b990d8dc73e59418114b49c36797e44127c76 (diff)
downloadangular.js-5a4145fe16bb61720843ae938b789e32dbbfdb7d.tar.bz2
revert: fix($location): return '/' for root path in hashbang mode
This reverts commit 63cd873fef3207deef30c7a7ed66f4b8f647dc12. The change breaks existing tests of Google apps. The problem is that while we tried to avoid adding #/ to window.location.href unnecessarily we failed doing so. Likely because by setting $path, at some point (during a digest) we try to check if $location changed and we mistake the default '/' with an explicit settign of the path via the `path()` method. This results in us writing the url with '#/' into $browser.url() which updates the window.location by adding "#/" to the url - something we tried to avoid in the first place. I'll reopen PR #5712.
Diffstat (limited to 'test/ng/locationSpec.js')
-rw-r--r--test/ng/locationSpec.js10
1 files changed, 0 insertions, 10 deletions
diff --git a/test/ng/locationSpec.js b/test/ng/locationSpec.js
index b66f689b..ff823d30 100644
--- a/test/ng/locationSpec.js
+++ b/test/ng/locationSpec.js
@@ -1487,16 +1487,6 @@ describe('$location', function() {
expect(location.url()).toBe('/not-starting-with-slash');
expect(location.absUrl()).toBe('http://server/pre/index.html#/not-starting-with-slash');
});
-
-
- it("should return / for path for the application root path", function() {
- location = new LocationHashbangUrl('http://server/pre/index.html', '#');
- location.$$parse('http://server/pre/index.html');
- expect(location.path()).toBe('/');
-
- location.$$parse('http://server/pre/');
- expect(location.path()).toBe('/');
- });
});