Fix a bug when calling evaluateCurrent('/')

When the current route was `/` and you would call `evaluateCurrent('/')`, nothing would happen.

Reported in TehShrike/abstract-state-router#116

Bug added in #08f93795

index.js

@@ -49,7 +49,7 @@ function getPathParts(path) {
 	var chunks = path.split('?')
 	return {
 		path: chunks.shift(),
-		queryString: qs.parse(chunks.join(''))
+		queryString: qs.parse(chunks.join('')),
 	}
 }
 
@@ -91,10 +91,11 @@ function add(routes, routeString, routeFunction) {
 }
 
 function evaluateCurrentPathOrGoToDefault(routes, hashLocation, reverse, onNotFound, defaultPath) {
-	const currentLocation = hashLocation.get()
-	if (currentLocation && currentLocation !== '/') {
-		var routesCopy = routes.slice()
+	var currentLocation = hashLocation.get()
+	var canUseCurrentLocation = currentLocation && (currentLocation !== '/' || defaultPath === '/')
 
+	if (canUseCurrentLocation) {
+		var routesCopy = routes.slice()
 		evaluateCurrentPath(routesCopy, hashLocation, reverse, onNotFound)
 	} else {
 		hashLocation.go(defaultPath)

test/all-tests.js

@@ -244,6 +244,32 @@ module.exports = function tests(locationHash, delayAfterInitialRouteChange) {
 		t.timeoutAfter(4000)
 	})
 
+	test('evaluateCurrent fires the / route when the url is currently /', function(t) {
+		t.plan(1)
+
+		startTest(function(getRoute) {
+			var route = getRoute()
+
+			route.on('not found', function(path, parameters) {
+				t.fail('Should not fire the not found route')
+
+				route.stop()
+				t.end()
+			})
+
+			route.add('/', function(parameters) {
+				t.deepEqual(parameters, {})
+
+				route.stop()
+				t.end()
+			})
+
+			route.evaluateCurrent('/')
+		}, '/')
+
+		t.timeoutAfter(4000)
+	})
+
 	test('replacing a url', function(t) {
 		startTest(function(getRoute) {
 			var route = getRoute()