Merge pull request #7 from universal-ember/ignore-hash-urls

Ignore hash URLs

test-app/app/templates/application.hbs

@@ -5,4 +5,6 @@
 {{outlet}}
 
 <a href="/foo">here</a>
-<a href="/does-not-exist">here</a>
+<a href="/does-not-exist">here</a>
+<a href="#title">here</a>
+<a href="/#title">/ here</a>

test-support/src/routing/visit-all.ts

@@ -47,7 +47,21 @@ export async function visitAllLinks() {
       continue;
     }
 
-    const key = `${currentURL()}::${toVisit}`;
+    // In-page links are on the page we're already on.
+    // As long as we haven't already encountered an error,
+    // this is silly to check.
+    if (toVisit.startsWith('#')) {
+      continue;
+    }
+
+    const [nonHashPart] = toVisit.split('#');
+
+    // This was our first page, we've already been here
+    if (nonHashPart === '/') {
+      continue;
+    }
+
+    const key = `${currentURL()}::${nonHashPart}`;
 
     if (visited.has(key)) continue;