aria-ease 6.5.0 → 6.6.0

package/dist/src/utils/test/index.cjs

@@ -234,11 +234,6 @@ ${"\u2500".repeat(60)}`);
 ${"\u2550".repeat(60)}`);
         this.log(`\u{1F4CA} Summary
 `);
-        const staticIcon = this.staticFailures === 0 ? "\u2705" : "\u274C";
-        const staticStatus = this.staticFailures === 0 ? "PASS" : "FAIL";
-        this.log(`${staticIcon} Static ARIA Tests: ${staticStatus}`);
-        this.log(`   ${this.staticPasses}/${this.staticPasses + this.staticFailures} required attributes present`);
-        this.log("");
         if (totalFailures === 0 && this.skipped === 0 && this.optionalSuggestions === 0) {
           this.log(`\u2705 All ${totalRun} tests passed!`);
           this.log(`   ${this.componentName.charAt(0).toUpperCase()}${this.componentName.slice(1)} component meets WAI-ARIA expectations for Roles, States, Properties, and Keyboard Interactions \u2713`);
@@ -495,29 +490,20 @@ This indicates a problem with the menu component's close functionality.`
         }
       }
       async shouldSkipTest(test, page) {
-        for (const act of test.action) {
-          if (act.type === "keypress" && (act.target === "submenuTrigger" || act.target === "submenu")) {
-            const submenuSelector = this.selectors[act.target];
-            if (submenuSelector) {
-              const submenuCount = await page.locator(submenuSelector).count();
-              if (submenuCount === 0) {
-                return true;
-              }
-            }
-          }
+        const requiresSubmenu = test.action.some(
+          (act) => act.target === "submenu" || act.target === "submenuTrigger" || act.target === "submenuItems"
+        ) || test.assertions.some(
+          (assertion) => assertion.target === "submenu" || assertion.target === "submenuTrigger" || assertion.target === "submenuItems"
+        );
+        if (!requiresSubmenu) {
+          return false;
         }
-        for (const assertion of test.assertions) {
-          if (assertion.target === "submenu" || assertion.target === "submenuTrigger") {
-            const submenuSelector = this.selectors[assertion.target];
-            if (submenuSelector) {
-              const submenuCount = await page.locator(submenuSelector).count();
-              if (submenuCount === 0) {
-                return true;
-              }
-            }
-          }
+        const submenuTriggerSelector = this.selectors.submenuTrigger;
+        if (!submenuTriggerSelector) {
+          return true;
         }
-        return false;
+        const submenuTriggerCount = await page.locator(submenuTriggerSelector).count();
+        return submenuTriggerCount === 0;
       }
       getMainSelector() {
         return this.mainSelector;
@@ -652,6 +638,9 @@ var init_ActionExecutor = __esm({
         this.selectors = selectors;
         this.timeoutMs = timeoutMs;
       }
+      isOptionalMenuTarget(target) {
+        return ["submenu", "submenuTrigger", "submenuItems"].includes(target);
+      }
       /**
        * Check if error is due to browser/page being closed
        */
@@ -772,7 +761,7 @@ var init_ActionExecutor = __esm({
           } else if (keyValue.includes(" ")) {
             keyValue = keyValue.replace(/ /g, "");
           }
-          if (target === "focusable" && ["ArrowUp", "ArrowDown", "ArrowLeft", "ArrowRight", "Escape"].includes(keyValue)) {
+          if (target === "focusable" && ["ArrowUp", "ArrowDown", "ArrowLeft", "ArrowRight", "Escape", "Home", "End", "Tab", "Shift+Tab"].includes(keyValue)) {
             await this.page.keyboard.press(keyValue);
             return { success: true };
           }
@@ -783,9 +772,10 @@ var init_ActionExecutor = __esm({
           const locator = this.page.locator(selector).first();
           const elementCount = await locator.count();
           if (elementCount === 0) {
+            const optionalMenuTarget = this.isOptionalMenuTarget(target);
             return {
               success: false,
-              error: `${target} element not found (optional submenu test)`,
+              error: optionalMenuTarget ? `${target} element not found (optional submenu test)` : `${target} element not found.`,
               shouldBreak: true
               // Signal to skip this test
             };
@@ -1154,23 +1144,35 @@ This usually means:
       }).catch(() => {
       });
     }
-    const failuresBeforeStatic = failures.length;
+    const hasSubmenuCapability = componentName === "menu" && !!componentContract.selectors.submenuTrigger ? await page.locator(componentContract.selectors.submenuTrigger).count() > 0 : false;
+    let staticFailed = 0;
     const staticAssertionRunner = new AssertionRunner(page, componentContract.selectors, assertionTimeoutMs);
     for (const test of componentContract.static[0]?.assertions || []) {
       if (test.target === "relative") continue;
       const staticDescription = `${test.target}${test.attribute ? ` (${test.attribute})` : ""}`;
+      if (componentName === "menu" && test.target === "submenuTrigger" && !hasSubmenuCapability) {
+        passes.push(`Skipping submenu static assertion for ${test.target}: no submenu capability detected in rendered component.`);
+        reporter.reportStaticTest(staticDescription, true);
+        continue;
+      }
       const targetSelector = componentContract.selectors[test.target];
       if (!targetSelector) {
         const failure = `Selector for target ${test.target} not found.`;
         failures.push(failure);
+        staticFailed += 1;
         reporter.reportStaticTest(staticDescription, false, failure);
         continue;
       }
       const target = page.locator(targetSelector).first();
       const exists = await target.count() > 0;
       if (!exists) {
+        if (test.isOptional === true) {
+          reporter.reportStaticTest(staticDescription, true);
+          continue;
+        }
         const failure = `Target ${test.target} not found.`;
         failures.push(failure);
+        staticFailed += 1;
         reporter.reportStaticTest(staticDescription, false, failure);
         continue;
       }
@@ -1207,6 +1209,7 @@ This usually means:
         if (!hasAny && !allRedundant) {
           const failure = test.failureMessage + ` None of the attributes "${test.attribute}" are present.`;
           failures.push(failure);
+          staticFailed += 1;
           reporter.reportStaticTest(staticDescription, false, failure);
         } else if (!allRedundant && hasAny) {
           passes.push(`At least one of the attributes "${test.attribute}" exists on the element.`);
@@ -1232,6 +1235,7 @@ This usually means:
             reporter.reportStaticTest(staticDescription, true);
           } else if (!result.success && result.failMessage) {
             failures.push(result.failMessage);
+            staticFailed += 1;
             reporter.reportStaticTest(staticDescription, false, result.failMessage);
           }
         }
@@ -1261,9 +1265,12 @@ This usually means:
       }
       const actionExecutor = new ActionExecutor(page, componentContract.selectors, actionTimeoutMs);
       const assertionRunner = new AssertionRunner(page, componentContract.selectors, assertionTimeoutMs);
+      let shouldSkipCurrentTest = false;
+      let shouldAbortCurrentTest = false;
       for (const act of action) {
         if (!page || page.isClosed()) {
           failures.push(`CRITICAL: Browser/page closed during test execution. Remaining actions skipped.`);
+          shouldAbortCurrentTest = true;
           break;
         }
         let result;
@@ -1281,18 +1288,29 @@ This usually means:
           continue;
         }
         if (!result.success) {
-          if (result.error) {
-            failures.push(result.error);
-          }
           if (result.shouldBreak) {
             if (result.error?.includes("optional submenu test")) {
               reporter.reportTest(dynamicTest, "skip", result.error);
+              shouldSkipCurrentTest = true;
+            } else if (result.error) {
+              failures.push(result.error);
+              shouldAbortCurrentTest = true;
             }
             break;
           }
+          if (result.error) {
+            failures.push(result.error);
+          }
           continue;
         }
       }
+      if (shouldSkipCurrentTest) {
+        continue;
+      }
+      if (shouldAbortCurrentTest) {
+        reporter.reportTest(dynamicTest, "fail", failures[failures.length - 1]);
+        continue;
+      }
       for (const assertion of assertions) {
         const result = await assertionRunner.validate(assertion, dynamicTest.description);
         if (result.success && result.passMessage) {
@@ -1312,7 +1330,6 @@ This usually means:
       }
     }
     const staticTotal = componentContract.static[0].assertions.length;
-    const staticFailed = failures.length - failuresBeforeStatic;
     const staticPassed = Math.max(0, staticTotal - staticFailed);
     reporter.reportStatic(staticPassed, staticFailed);
     reporter.summary(failures);

package/dist/src/utils/test/index.js

@@ -1,4 +1,4 @@
-import { closeSharedBrowser, ContractReporter, contract_default } from './chunk-AUJAN4RK.js';
+import { closeSharedBrowser, ContractReporter, contract_default } from './chunk-LKN5PRYD.js';
 import { axe } from 'jest-axe';
 import fs from 'fs/promises';
 
@@ -108,7 +108,7 @@ Error: ${error instanceof Error ? error.message : String(error)}`
       const devServerUrl = await checkDevServer(url);
       if (devServerUrl) {
         console.log(`\u{1F3AD} Running Playwright tests on ${devServerUrl}`);
-        const { runContractTestsPlaywright } = await import('./contractTestRunnerPlaywright-HL73FADJ.js');
+        const { runContractTestsPlaywright } = await import('./contractTestRunnerPlaywright-RGKMGXND.js');
         contract = await runContractTestsPlaywright(componentName, devServerUrl);
       } else {
         throw new Error(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "aria-ease",
-  "version": "6.5.0",
+  "version": "6.6.0",
   "description": "Accessibility infrastructure for the entire frontend engineering lifecycle. Build accessible patterns, run automated audits, verify component interactions, and gate deployments — all in one system.",
   "main": "dist/index.cjs",
   "type": "module",

package/dist/src/chunk-ZJXZZDUR.js

@@ -1,127 +0,0 @@
-// src/utils/handleKeyPress/handleKeyPress.ts
-function isTextInput(el) {
-  if (el.tagName !== "INPUT") return false;
-  const type = el.type;
-  return ["text", "email", "password", "tel", "number"].includes(type);
-}
-function isTextArea(el) {
-  return el.tagName === "TEXTAREA";
-}
-function isNativeButton(el) {
-  return el.tagName === "BUTTON" || el.tagName === "INPUT" && ["button", "submit", "reset"].includes(el.type);
-}
-function isLink(el) {
-  return el.tagName === "A";
-}
-function moveFocus(elementItems, currentIndex, direction) {
-  const len = elementItems.length;
-  const nextIndex = (currentIndex + direction + len) % len;
-  elementItems.item(nextIndex).focus();
-}
-function isClickableButNotSemantic(el) {
-  return el.getAttribute("data-custom-click") !== null && el.getAttribute("data-custom-click") !== void 0;
-}
-function handleMenuClose(menuElement, menuTriggerButton) {
-  menuElement.style.display = "none";
-  const menuTriggerButtonId = menuTriggerButton.getAttribute("id");
-  if (!menuTriggerButtonId) {
-    console.error("[aria-ease] Menu trigger button must have an id attribute to properly set aria attributes.");
-    return;
-  }
-  menuTriggerButton.setAttribute("aria-expanded", "false");
-}
-function hasSubmenu(menuItem) {
-  return menuItem.getAttribute("aria-haspopup") === "true" || menuItem.getAttribute("aria-haspopup") === "menu";
-}
-function getSubmenuId(menuItem) {
-  return menuItem.getAttribute("aria-controls");
-}
-function handleKeyPress(event, elementItems, elementItemIndex, menuElementDiv, triggerButton, openSubmenu, closeSubmenu, onOpenChange) {
-  const currentEl = elementItems.item(elementItemIndex);
-  switch (event.key) {
-    case "ArrowUp":
-    case "ArrowLeft": {
-      if (event.key === "ArrowLeft" && menuElementDiv && closeSubmenu) {
-        const labelledBy = menuElementDiv.getAttribute("aria-labelledby");
-        if (labelledBy) {
-          const parentTrigger = document.getElementById(labelledBy);
-          if (parentTrigger && parentTrigger.getAttribute("role") === "menuitem") {
-            event.preventDefault();
-            closeSubmenu();
-            parentTrigger.focus();
-            return;
-          }
-        }
-      }
-      if (!isTextInput(currentEl) && !isTextArea(currentEl)) {
-        event.preventDefault();
-        moveFocus(elementItems, elementItemIndex, -1);
-      } else if (isTextInput(currentEl) || isTextArea(currentEl)) {
-        const cursorStart = currentEl.selectionStart;
-        if (cursorStart === 0) {
-          event.preventDefault();
-          moveFocus(elementItems, elementItemIndex, -1);
-        }
-      }
-      break;
-    }
-    case "ArrowDown":
-    case "ArrowRight": {
-      if (event.key === "ArrowRight" && hasSubmenu(currentEl) && openSubmenu) {
-        event.preventDefault();
-        const submenuId = getSubmenuId(currentEl);
-        if (submenuId) {
-          openSubmenu(submenuId);
-          return;
-        }
-      }
-      if (!isTextInput(currentEl) && !isTextArea(currentEl)) {
-        event.preventDefault();
-        moveFocus(elementItems, elementItemIndex, 1);
-      } else if (isTextInput(currentEl) || isTextArea(currentEl)) {
-        const value = currentEl.value;
-        const cursorEnd = currentEl.selectionStart;
-        if (cursorEnd === value.length) {
-          event.preventDefault();
-          moveFocus(elementItems, elementItemIndex, 1);
-        }
-      }
-      break;
-    }
-    case "Escape": {
-      event.preventDefault();
-      if (menuElementDiv && triggerButton) {
-        if (getComputedStyle(menuElementDiv).display === "block") {
-          handleMenuClose(menuElementDiv, triggerButton);
-          if (onOpenChange) {
-            onOpenChange(false);
-          }
-        }
-        triggerButton.focus();
-      }
-      break;
-    }
-    case "Enter":
-    case " ": {
-      if (!isNativeButton(currentEl) && !isLink(currentEl) && isClickableButNotSemantic(currentEl)) {
-        event.preventDefault();
-        currentEl.click();
-      } else if (isNativeButton(currentEl)) {
-        event.preventDefault();
-        currentEl.click();
-      }
-      break;
-    }
-    case "Tab": {
-      if (menuElementDiv && triggerButton && (!event.shiftKey || event.shiftKey)) {
-        handleMenuClose(menuElementDiv, triggerButton);
-        if (onOpenChange) {
-          onOpenChange(false);
-        }
-      }
-      break;
-    }
-  }
-}
-
-export { handleKeyPress };