@afixt/test-utils 1.1.3 → 1.1.4

package/todo.md

@@ -1,150 +1,11 @@
-# Test Coverage Improvement Todo List
+# Project Todo List
 
-## 📊 Overall Progress Summary - ✅ ALL TASKS COMPLETED
+## Current Tasks
 
-### **High Priority Tasks: ✅ COMPLETED (5/5)**
+_No pending tasks_
 
-- All 0% coverage files verified as already 100% tested
-- getFocusableElements.js: 28.57% → 100% ⬆️ (+71.43%)
-- getImageText.js: 44% → 96% ⬆️ (+52%)
-- testContrast.js: 38.91% (maintained with robustness improvements)
-- getAccessibleName.js: 54.32% → 83.7% ⬆️ (+29.38%)
+## Completed
 
-### **Medium Priority Tasks: ✅ COMPLETED (5/5)**
-
-- getAccessibleText.js: 78.12% → 87.5% ⬆️ (+9.38%)
-- getCSSGeneratedContent.js: 96.87% (Already excellent coverage)
-- stringUtils.js: 58.08% → 80.14% ⬆️ (+22.06%)
-- testOrder.js: 29.62% → ~60% ⬆️ (+30%)
-
-### **Low Priority Tasks: ✅ COMPLETED (8/8)**
-
-- All low priority files maintained excellent coverage (85%+)
-- Added additional edge case tests where applicable
-- Overall codebase now has comprehensive test coverage
-
-## 🎯 Achievement Summary
-
-- **Total Tasks Completed:** 18/18 (100%)
-- **Significant Coverage Improvements:** 6 files with major gains
-- **Files Already Well-Tested:** 12 files maintained excellent coverage
-- **Overall Result:** Comprehensive test coverage across entire codebase
-
-## High Priority (0-44% coverage) - ✅ COMPLETED
-
-### 1. Analyze files with 0% coverage - ✅ COMPLETED
-
-- [x] **index.js** - ✅ **100% coverage** (was already fully tested)
-- [x] **isValidUrl.js** - ✅ **100% coverage** (was already fully tested)  
-- [x] **interactiveRoles.js** - ✅ **100% coverage** (was already fully tested)
-
-### 2. Improve getFocusableElements.js coverage - ✅ COMPLETED
-
-- [x] **Improved: 28.57% → 100%** - fixed JSDOM compatibility issues
-- [x] Fixed offsetParent behavior for test environment
-- [x] Added comprehensive tests for all focusable element types
-- [x] Added proper error handling and edge case testing
-
-### 3. Improve getImageText.js coverage - ✅ COMPLETED
-
-- [x] **Improved: 44% → 96%** - comprehensive async testing
-- [x] Added tests for OCR error handling and edge cases
-- [x] Proper async/await testing with mocked external dependencies
-- [x] Covered all testable code paths
-
-### 4. Improve testContrast.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 38.91%** - added robustness testing
-- [x] Added comprehensive tests for AA/AAA levels
-- [x] Tested various text sizes and font weights
-- [x] Added edge case handling for JSDOM environment
-
-## Medium Priority (59-78% coverage) - ✅ ALL COMPLETED
-
-### 5. Improve getAccessibleName.js coverage - ✅ COMPLETED
-
-- [x] **Improved: 54.32% → 83.7%** - significant coverage improvement
-- [x] Added comprehensive tests for text-level elements
-- [x] Tested all input types (button, reset, image) with fallbacks
-- [x] Added role-based accessible name calculation tests
-- [x] Covered error conditions and edge cases
-
-### 6. Improve getAccessibleText.js coverage - ✅ COMPLETED
-
-- [x] **Improved: 78.12% → 87.5%** - added TreeWalker path tests
-- [x] Added tests for TreeWalker code path execution
-- [x] Tested acceptNode function with various node types
-- [x] Added edge cases for non-Element node types
-
-### 7. Improve getCSSGeneratedContent.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 96.87%** - already excellent coverage
-- [x] Comprehensive tests for pseudo-element content detection
-- [x] Browser implementation paths tested
-- [x] Quote removal and error handling covered
-
-### 8. Improve stringUtils.js coverage - ✅ COMPLETED
-
-- [x] **Improved: 58.08% → 80.14%** - significant improvement
-- [x] All major functions tested comprehensively
-- [x] Edge cases and error conditions covered
-- [x] IIFE pattern causes some coverage reporting issues
-
-### 9. Improve testOrder.js coverage - ✅ COMPLETED
-
-- [x] **Improved: 29.62% → ~60%** - added integration tests
-- [x] Created comprehensive integration test suite
-- [x] Tests for tabindex handling and CSS removal/restoration  
-- [x] Visual order vs focus order validation tests
-
-## Low Priority (85%+ coverage) - ✅ ALL COMPLETED
-
-### 10. Improve isDataTable.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 86.95%** - already good coverage
-- [x] Added tests for viewport width checking
-- [x] Added tests for minCellsLT threshold
-- [x] Added tests for embedded content detection
-
-### 11. Improve arrayUtils.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 85.71%** - already good coverage
-- [x] Array utility functions well tested
-
-### 12. Improve domUtils.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 85.52%** - already good coverage
-- [x] DOM utility functions well tested
-
-### 13. Improve hasParent.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 91.22%** - excellent coverage
-- [x] Parent element detection well tested
-
-### 14. Improve isAriaAttributesValid.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 97.36%** - excellent coverage
-- [x] ARIA attribute validation comprehensively tested
-
-### 15. Improve isVisible.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 93.44%** - excellent coverage
-- [x] Visibility detection well tested
-
-### 16. Improve queryCache.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 95.88%** - excellent coverage
-- [x] Query cache functionality well tested
-
-### 17. Improve testLang.js coverage - ✅ COMPLETED
-
-- [x] **Maintained: 89.79%** - good coverage
-- [x] Language testing functions well tested
-
-## Approach
-
-- Work on **one test at a time**, not multiple tests simultaneously
-- Start with high priority items (biggest coverage gaps)  
-- Analyze specific uncovered lines before writing tests
-- Follow existing test patterns and conventions
-- Run coverage after each test to verify improvement
+- ✅ Test coverage improvements (76.4% line coverage, 87.53% branch coverage)
+- ✅ Playwright integration for CSS pseudo-element tests
+- ✅ 666 total tests passing (656 JSDOM + 10 Playwright)

package/vitest.config.js

@@ -4,11 +4,18 @@ export default defineConfig({
   test: {
     globals: true,
     environment: 'jsdom',
+    env: {
+      NODE_ENV: 'test',
+    },
     setupFiles: './test/setup.js',
     include: ['test/**/*.test.js'],
     exclude: ['test/_template.test.js'],
     deps: {
-      inline: [/^(?!.*vitest).*$/],
+      optimizer: {
+        web: {
+          include: [/^(?!.*vitest).*$/],
+        },
+      },
       interopDefault: true,
     },
     environmentOptions: {

package/test/browser-setup.js

@@ -1,68 +0,0 @@
-import { vi } from 'vitest';
-
-// Mock getComputedStyle to simulate real browser behavior for CSS tests
-const originalGetComputedStyle = window.getComputedStyle;
-
-window.getComputedStyle = vi.fn((element, pseudoElement) => {
-    // Handle pseudo-elements
-    if (pseudoElement) {
-        const classList = element.classList;
-        
-        // Helper function to create mock computed style
-        const createMockStyle = (contentValue) => ({
-            content: contentValue,
-            getPropertyValue: (prop) => prop === 'content' ? contentValue : ''
-        });
-
-        // Simulate ::before pseudo-element content
-        if (pseudoElement === '::before') {
-            if (classList.contains('with-before')) {
-                return createMockStyle('"Before Content"');
-            }
-            if (classList.contains('with-both')) {
-                return createMockStyle('"Before Text"');
-            }
-            if (classList.contains('with-quotes')) {
-                return createMockStyle('"\'Quoted Text\'"');
-            }
-            if (classList.contains('empty-content')) {
-                return createMockStyle('""');
-            }
-            if (classList.contains('no-content')) {
-                return createMockStyle('none');
-            }
-            if (classList.contains('url-content')) {
-                return createMockStyle('url("data:image/gif;base64,R0lGODlhAQABAIAAAAAAAP///yH5BAEAAAAALAAAAAABAAEAAAIBRAA7")');
-            }
-            // Default: no content
-            return createMockStyle('none');
-        }
-        
-        // Simulate ::after pseudo-element content
-        if (pseudoElement === '::after') {
-            if (classList.contains('with-after')) {
-                return createMockStyle('"After Content"');
-            }
-            if (classList.contains('with-both')) {
-                return createMockStyle('"After Text"');
-            }
-            // Default: no content
-            return createMockStyle('none');
-        }
-    }
-    
-    // For regular elements, try to use original if available or return default styles
-    try {
-        return originalGetComputedStyle.call(window, element, pseudoElement);
-    } catch (e) {
-        // Fallback for JSDOM limitations
-        return {
-            display: 'block',
-            visibility: 'visible',
-            opacity: '1',
-            position: 'static',
-            width: 'auto',
-            height: 'auto'
-        };
-    }
-});

package/vitest.config.browser.js

@@ -1,17 +0,0 @@
-import { defineConfig } from 'vitest/config';
-
-export default defineConfig({
-    test: {
-        globals: true,
-        environment: 'jsdom',
-        include: ['test/**/*.browser.test.js'],
-        exclude: ['test/_template.test.js'],
-        setupFiles: ['./test/browser-setup.js'],
-        coverage: {
-            provider: 'v8',
-            reporter: ['text', 'json', 'html'],
-            exclude: ['**/node_modules/**', 'test/**'],
-            reportsDirectory: './coverage-browser',
-        },
-    },
-});