testaro 28.2.0 → 28.2.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testaro",
-  "version": "28.2.0",
+  "version": "28.2.2",
   "description": "Run 960 web accessibility tests from 9 tools and get a standardized report",
   "main": "index.js",
   "scripts": {

package/procs/getLocatorData.js

@@ -33,6 +33,9 @@ exports.getLocatorData = async loc => {
       if (! text) {
         text = element.outerHTML.replace(/\s+/g, ' ').trim();
       }
+      if (/^<[^<>]+>$/.test(text)) {
+        text = element.parentElement.outerHTML.replace(/\s+/g, ' ').trim();
+      }
       // Location.
       let location = {
         doc: 'dom',

package/testaro/hover.js

@@ -65,7 +65,6 @@ exports.reporter = async (page, withItems) => {
       }
     }
     catch(error) {
-      console.log(`Hovering timed out (${error.message.replace(/[\n].+/s, '')})`);
     }
   }
   // Populate and return the result.

package/testaro/pseudoP.js

@@ -15,29 +15,28 @@ const {init, report} = require('../procs/testaro');
 // Runs the test and returns the result.
 exports.reporter = async (page, withItems) => {
   // Initialize the locators and result.
-  const all = await init(page, 'body *', {has: page.locator('br + br')});
+  const all = await init(page, 'body br + br');
   // For each locator:
   for (const loc of all.allLocs) {
-    const isBad = await loc.evaluate(el => {
-      // Get whether it has 2 adjacent br elements with no non-space text between them.
-      const childNodes = Array.from(el.childNodes);
-      const realChildNodes = childNodes.filter(
-        node => node.nodeType !== Node.TEXT_NODE || node.nodeValue.replace(/\s/g, '').length
-      );
-      return realChildNodes.some(
-        (node, index) => node.nodeName === 'BR' && realChildNodes[index + 1].nodeName === 'BR'
-      );
+    // Return whether the second br element violates the rule.
+    const parentTagNameIfBad = await loc.evaluate(el => {
+      el.parentElement.normalize();
+      const previousSib = el.previousSibling;
+      return previousSib.nodeType === Node.ELEMENT_NODE
+      || previousSib.nodeType === Node.TEXT_NODE && /^\s+$/.test(previousSib)
+      ? el.parentElement.tagName
+      : false;
     });
     // If it does:
-    if (isBad) {
+    if (parentTagNameIfBad) {
       // Add the locator to the array of violators.
-      all.locs.push(loc);
+      all.locs.push([loc, parentTagNameIfBad]);
     }
-  }
+  };
   // Populate and return the result.
   const whats = [
-    'Element contains 2 or more adjacent br elements that may be a pseudo-paragraph',
+    'Adjacent br elements within a __param__ element may be pseudo-paragraphs',
     'Elements contain 2 or more adjacent br elements that may be pseudo-paragraphs'
   ];
-  return await report(withItems, all, 'pseudoP', whats, 0);
+  return await report(withItems, all, 'pseudoP', whats, 0, 'br');
 };

package/tests/testaro.js

@@ -56,7 +56,7 @@ const evalRules = {
   miniText: 'text smaller than 11 pixels',
   motion: 'motion without user request',
   nonTable: 'table elements used for layout',
-  opFoc: 'Operable elements that are not Tab-focusable',
+  opFoc: 'operable elements that are not Tab-focusable',
   pseudoP: 'adjacent br elements suspected of nonsemantically simulating p elements',
   radioSet: 'radio buttons not grouped into standard field sets',
   role: 'invalid and native-replacing explicit roles',