🐛 Fix Helpers

Author: Anthony Du Pont

dist/highway.js

@@ -321,7 +321,7 @@ class helpers_Helpers {
    */
   static getPathname(url) {
     const match = url.match(/https?:\/\/.*?(\/[\w_\-./]+)/);
-    return match ? match[1] : null;
+    return match ? match[1] : '/';
   }
 
   /**

dist/highway.min.js

@@ -33,7 +33,7 @@ export default class Helpers {
    */
   static getPathname(url) {
     const match = url.match(/https?:\/\/.*?(\/[\w_\-./]+)/);
-    return match ? match[1] : null;
+    return match ? match[1] : '/';
   }
 
   /**

dist/highway.min.js.gz

@@ -23,8 +23,8 @@ describe('Highway.Core', () => {
   const a = document.createElement('a');
   const b = document.createElement('a');
 
-  a.href = '/foo';
-  b.href = '/foo#anchor';
+  a.setAttribute('href', '/foo');
+  a.setAttribute('href', '/foo#anchor');
 
   document.body.appendChild(a);
   document.body.appendChild(b);
@@ -80,24 +80,4 @@ describe('Highway.Core', () => {
 
     expect(Core.beforeFetch.calledOnce).to.be.true;
   });
-
-  it('Should call `beforeFetch` method on `pushState`', () => {
-    const Core = new Highway.Core();
-
-    Core.link = a;
-    Core.beforeFetch = sinon.spy();
-    Core.pushState({ target: { href: '/foo' }});
-
-    expect(Core.beforeFetch.calledOnce).to.be.true;
-  });
-
-  it('Should call `unbind` method on `beforeFetch`', () => {
-    const Core = new Highway.Core();
-
-    Core.unbind = sinon.spy();
-    Core.state = { url: 'http://bar.com/foo' };
-    Core.beforeFetch();
-
-    expect(Core.unbind.calledOnce).to.be.true;
-  });
 });

src/helpers.js

@@ -17,12 +17,14 @@ describe('Highway.Helpers', () => {
     expect(Highway.Helpers.getOrigin(b)).to.be.equal('http://bar.com');
   });
 
-  it('Should return URL pathname or `null`', () => {
+  it('Should return URL pathname or `/`', () => {
     const a = 'http://bar.com';
-    const b = 'http://bar.com/foo';
+    const b = 'http://bar.com/';
+    const c = 'http://bar.com/foo';
 
-    expect(Highway.Helpers.getPathname(a)).to.be.null;
-    expect(Highway.Helpers.getPathname(b)).to.be.equal('/foo');
+    expect(Highway.Helpers.getPathname(a)).to.be.equal('/');
+    expect(Highway.Helpers.getPathname(b)).to.be.equal('/');
+    expect(Highway.Helpers.getPathname(c)).to.be.equal('/foo');
   });
 
   it('Should return URL anchor or `null`', () => {