refactor(checks/navigation): improve internal-link-present-evaluate

Make `internal-link-present-evaluate` work with virtualNode rather than actualNode. Closes issue #2466
dequelabs · Aug 20, 2021 · 2467b3a · 2467b3a
1 parent f18c2e0
commit 2467b3a
Show file tree

Hide file tree

Showing 2 changed files with 88 additions and 87 deletions.
diff --git a/lib/checks/navigation/internal-link-present-evaluate.js b/lib/checks/navigation/internal-link-present-evaluate.js
@@ -1,10 +1,10 @@
-import { querySelectorAll } from '../../core/utils';
-
-function internalLinkPresentEvaluate(node, options, virtualNode) {
-  const links = querySelectorAll(virtualNode, 'a[href]');
-  return links.some(vLink => {
-    return /^#[^/!]/.test(vLink.actualNode.getAttribute('href'));
-  });
-}
-
-export default internalLinkPresentEvaluate;
+import { querySelectorAll } from '../../core/utils';
+
+function internalLinkPresentEvaluate(node, options, virtualNode) {
+  const links = querySelectorAll(virtualNode, 'a[href]');
+  return links.some(vLink => {
+    return /^#[^/!]/.test(vLink.attr('href'));
+  });
+}
+
+export default internalLinkPresentEvaluate;
diff --git a/test/checks/navigation/internal-link-present.js b/test/checks/navigation/internal-link-present.js
@@ -1,77 +1,78 @@
-describe('internal-link-present', function() {
-  'use strict';
-
-  var fixture = document.getElementById('fixture');
-  var shadowSupported = axe.testUtils.shadowSupport.v1;
-  var checkContext = axe.testUtils.MockCheckContext();
-  var checkSetup = axe.testUtils.checkSetup;
-  var shadowCheckSetup = axe.testUtils.shadowCheckSetup;
-
-  afterEach(function() {
-    fixture.innerHTML = '';
-    axe._tree = undefined;
-    checkContext.reset();
-  });
-
-  it('should return true when an internal link is found', function() {
-    var params = checkSetup('<div id="target"><a href="#haha">hi</a></div>');
-    assert.isTrue(
-      axe.testUtils
-        .getCheckEvaluate('internal-link-present')
-        .apply(checkContext, params)
-    );
-  });
-
-  it('should return false when a hashbang URL was used', function() {
-    var params = checkSetup('<div id="target"><a href="#!foo">hi</a></div>');
-    assert.isFalse(
-      axe.testUtils
-        .getCheckEvaluate('internal-link-present')
-        .apply(checkContext, params)
-    );
-  });
-
-  it('should return false when a hash route URL was used', function() {
-    var params = checkSetup('<div id="target"><a href="#/home">hi</a></div>');
-    assert.isFalse(
-      axe.testUtils
-        .getCheckEvaluate('internal-link-present')
-        .apply(checkContext, params)
-    );
-  });
-
-  it('should return false when a hashbang + slash route URL was used', function() {
-    var params = checkSetup('<div id="target"><a href="#!/home">hi</a></div>');
-    assert.isFalse(
-      axe.testUtils
-        .getCheckEvaluate('internal-link-present')
-        .apply(checkContext, params)
-    );
-  });
-
-  it('should otherwise return false', function() {
-    var params = checkSetup(
-      '<div id="target"><a href="http://www.deque.com/#haha">hi</a></div>'
-    );
-    assert.isFalse(
-      axe.testUtils
-        .getCheckEvaluate('internal-link-present')
-        .apply(checkContext, params)
-    );
-  });
-
-  (shadowSupported ? it : xit)(
-    'should return true when internal link is found in shadow dom',
-    function() {
-      var params = shadowCheckSetup(
-        '<div id="target"></div>',
-        '<a href="#haha">hi</a>'
-      );
-      assert.isTrue(
-        axe.testUtils
-          .getCheckEvaluate('internal-link-present')
-          .apply(checkContext, params)
-      );
-    }
-  );
-});
+describe('internal-link-present', function() {
+  'use strict';
+
+  var fixture = document.getElementById('fixture');
+  var shadowSupported = axe.testUtils.shadowSupport.v1;
+  var checkContext = axe.testUtils.MockCheckContext();
+  var shadowCheckSetup = axe.testUtils.shadowCheckSetup;
+  var queryFixture = axe.testUtils.queryFixture;
+
+  afterEach(function() {
+    fixture.innerHTML = '';
+    axe._tree = undefined;
+    checkContext.reset();
+  });
+
+  it('should return true when an internal link is found', function() {
+    var vNode = queryFixture('<div id="target"><a href="#haha">hi</a></div>');
+    assert.isTrue(
+      axe.testUtils
+        .getCheckEvaluate('internal-link-present')
+        .call(checkContext, null, {}, vNode)
+    );
+  });
+
+  it('should return false when a hashbang URL was used', function() {
+    var vNode = queryFixture('<div id="target"><a href="#!foo">hi</a></div>');
+    assert.isFalse(
+      axe.testUtils
+        .getCheckEvaluate('internal-link-present')
+        .call(checkContext, null, {}, vNode)
+    );
+  });
+
+  it('should return false when a hash route URL was used', function() {
+    var vNode = queryFixture('<div id="target"><a href="#/home">hi</a></div>');
+    assert.isFalse(
+      axe.testUtils
+        .getCheckEvaluate('internal-link-present')
+        .call(checkContext, null, {}, vNode)
+    );
+  });
+
+  it('should return false when a hashbang + slash route URL was used', function() {
+    var vNode = queryFixture('<div id="target"><a href="#!/home">hi</a></div>');
+    assert.isFalse(
+      axe.testUtils
+        .getCheckEvaluate('internal-link-present')
+        .call(checkContext, null, {}, vNode)
+    );
+  });
+
+  it('should otherwise return false', function() {
+    var vNode = queryFixture(
+      '<div id="target"><a href="http://www.deque.com/#haha">hi</a></div>'
+    );
+    assert.isFalse(
+      axe.testUtils
+        .getCheckEvaluate('internal-link-present')
+        .call(checkContext, null, {}, vNode)
+    );
+  });
+
+  (shadowSupported ? it : xit)(
+    'should return true when internal link is found in shadow dom',
+    function() {
+      var params = shadowCheckSetup(
+        '<div id="target"></div>', 
+        '<a href="#haha">hi</a>'
+      );
+	  var vNode = params[2];
+      assert.isTrue(
+        axe.testUtils
+          .getCheckEvaluate('internal-link-present')
+          .call(checkContext, null, {}, vNode)
+      );
+    }
+  );
+});