From aa218055931bbe78c08a745dff9603e2e1b3741b Mon Sep 17 00:00:00 2001
From: Garrett Murphey <gmurphey@gmurphey.com>
Date: Wed, 1 Aug 2018 14:59:23 -0400
Subject: [PATCH] Fixing route recognizing logic when using hash location
 implementation.

---
 addon/href-to.js           | 21 +++++++++++++++++++--
 tests/unit/href-to-test.js | 30 ++++++++++++++++++++++++++++++
 2 files changed, 49 insertions(+), 2 deletions(-)

diff --git a/addon/href-to.js b/addon/href-to.js
index e4e309b..d230158 100644
--- a/addon/href-to.js
+++ b/addon/href-to.js
@@ -93,9 +93,26 @@ export default class {
     return this.applicationInstance.lookup('service:router');
   }
 
-  _getRootUrl() {
+  _getLocationImplementation() {
+    let router = this._getRouter();
+
+    return router.get('location.implementation');
+  }
+
+  _getRouterRootUrl() {
     let router = this._getRouter();
-    let rootURL = router.get('rootURL');
+
+    return router.get('rootURL');
+  }
+
+  _getRootUrl() {
+    let locationType = this._getLocationImplementation();
+
+    if (locationType === 'hash') {
+      return '#/';
+    }
+
+    let rootURL = this._getRouterRootUrl();
 
     if (rootURL.charAt(rootURL.length - 1) !== '/') {
       rootURL = rootURL + '/';
diff --git a/tests/unit/href-to-test.js b/tests/unit/href-to-test.js
index 2d386fd..416608b 100644
--- a/tests/unit/href-to-test.js
+++ b/tests/unit/href-to-test.js
@@ -107,3 +107,33 @@ test('#getUrlWithoutRoot should remove the rootUrl', function(assert) {
   hrefTo._getRootUrl = () => '/';
   assert.equal(hrefTo.getUrlWithoutRoot(), '/a/inbox', 'the url shouldn\'t include the rootUrl');
 });
+
+module('#_getRootUrl', function() {
+  test('it builds auto location roots correctly', function(assert) {
+    let event = getClickEventOnEl("<a href='/a/inbox'>");
+    let hrefTo = createHrefToForEvent(event);
+
+    hrefTo._getLocationImplementation = () => 'auto';
+    hrefTo._getRouterRootUrl = () => '/';
+
+    assert.equal(hrefTo._getRootUrl(), '/');
+
+    hrefTo._getRouterRootUrl = () => '/app';
+
+    assert.equal(hrefTo._getRootUrl(), '/app/');
+  });
+
+  test('it builds hash location roots correctly', function(assert) {
+    let event = getClickEventOnEl("<a href='/a/inbox'>");
+    let hrefTo = createHrefToForEvent(event);
+
+    hrefTo._getLocationImplementation = () => 'hash';
+    hrefTo._getRouterRootUrl = () => '/';
+
+    assert.equal(hrefTo._getRootUrl(), '#/');
+
+    hrefTo._getRouterRootUrl = () => '/app';
+
+    assert.equal(hrefTo._getRootUrl(), '#/');
+  });
+});