@afixt/test-utils 1.1.1 → 1.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@afixt/test-utils",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "Various utilities for accessibility testing",
   "main": "src/index.js",
   "scripts": {

package/src/domUtils.js

@@ -3,7 +3,7 @@ const domUtils = {
      * Checks if the specified element has the given attribute.
      *
      * @param {HTMLElement} element - The DOM element to check.
-     * @param {string} name - The name of the attribute to look for.
+     * @param {string} attrName - The name of the attribute to look for.
      * @returns {boolean} True if the element has the attribute, false otherwise.
      */
     hasAttr(element, attrName) {

package/src/getAriaAttributesByElement.js

@@ -4,7 +4,7 @@
  * @param {Element} element - The DOM element from which to extract ARIA attributes.
  * @returns {string[]} An array of ARIA attribute names present on the element.
  */
-const getAriaAttributes = (element) => {
+const getAriaAttributesByElement = (element) => {
     let result = [];
     const attrs = element.attributes;
 
@@ -16,4 +16,4 @@ const getAriaAttributes = (element) => {
     return result;
 };
 
-module.exports = getAriaAttributes;
+module.exports = getAriaAttributesByElement;

package/src/getFocusableElements.js

@@ -1,5 +1,5 @@
 /**
- *
+ * Gets all focusable elements within the specified container element.
  * @param  {Element} el - the element to be tested
  * @returns {Array} - Array of focusable elements
  */

package/src/stringUtils.js

@@ -72,7 +72,7 @@ const stringUtils = (function () {
      * @returns {boolean}
      */
     function isAlphaNumeric(str) {
-        const pattern = "/^[a-zA-Z0-9]+$/";
+        const pattern = /^[a-zA-Z0-9]+$/;
         return pattern.test(str);
     }
 

package/todo.md

@@ -1,2 +1 @@
 # Test-Utils Todo
-

package/repairs-needed.md

@@ -1,84 +0,0 @@
-# Required Changes for @afixt/test-utils Library
-
-## Issue
-
-The browser extension is throwing "Invalid input: element must be a DOM Element,
-got: null" errors when viewing dynamic sites like CNN.com. The error originates
-from the @afixt/test-utils library functions.
-
-## Root Cause
-
-The `isVisible()` function (and potentially other functions) in
-@afixt/test-utils throws an error when passed null or non-Element arguments.
-This happens frequently during DOM mutations when:
-
-1. Elements are removed from the DOM during repair operations
-2. Mutation observers fire on disconnected elements
-3. Elements become stale between detection and processing
-
-## Required Changes
-
-### 1. isVisible() Function
-
-The function needs to handle null/invalid inputs gracefully:
-
-```javascript
-function isVisible(element, strict = false) {
-  // Add null check at the beginning
-  if (!element || !(element instanceof Element)) {
-    return false;
-  }
-
-  // Check if element is still connected to the DOM
-  if (!element.isConnected) {
-    return false;
-  }
-
-  // ... rest of the existing function
-}
-```
-
-### 2. getAccessibleText() Function
-
-Should handle null elements:
-
-```javascript
-function getAccessibleText(element) {
-  if (!element || !(element instanceof Element)) {
-    return '';
-  }
-
-  if (!element.isConnected) {
-    return '';
-  }
-
-  // ... rest of the existing function
-}
-```
-
-### 3. getAccessibleName() Function
-
-Should handle null elements:
-
-```javascript
-function getAccessibleName(element) {
-  if (!element || !(element instanceof Element)) {
-    return '';
-  }
-
-  if (!element.isConnected) {
-    return '';
-  }
-
-  // ... rest of the existing function
-}
-```
-
-## Why These Changes Are Needed
-
-1. **Defensive Programming**: DOM elements can become null or disconnected at
-   any time, especially in dynamic web applications
-2. **Mutation Observer Behavior**: When observing DOM changes, elements can be
-   removed between the time a mutation is detected and when it's processed
-3. **Library Robustness**: A utility library should handle invalid inputs
-   gracefully rather than throwing errors