snap-ally 1.0.2 → 1.0.3

package/dist/SnapAllyReporter.js

@@ -140,22 +140,31 @@ class SnapAllyReporter {
         const videoPath = this.assetsManager.copyVideos(result, testFolder);
         const screenshotPaths = this.assetsManager.copyScreenshots(result, testFolder);
         const attachments = this.assetsManager.copyAllOtherAttachments(result, testFolder);
-        const a11yAttachment = result.attachments.find((a) => a.name === 'A11y');
-        if (!a11yAttachment && this.options.verbose) {
+        // A single test may call scanA11y/checkAccessibility multiple times (e.g.
+        // before and after login), producing one 'A11y' attachment per scan. Read
+        // every 'A11y' attachment and merge their violations so findings from all
+        // scans are reported — not just the first.
+        const a11yAttachments = result.attachments.filter((a) => a.name === 'A11y' && a.body);
+        if (a11yAttachments.length === 0 && this.options.verbose) {
             console.warn(`[SnapAlly] A11y attachment missing for test: ${test.title}. Available: ${result.attachments.map(a => a.name).join(', ')}`);
         }
-        let a11yData = null;
-        if (a11yAttachment && a11yAttachment.body) {
+        const parsedData = [];
+        for (const att of a11yAttachments) {
             try {
-                a11yData = JSON.parse(a11yAttachment.body.toString());
+                const raw = JSON.parse(att.body.toString());
+                // Handle cases where the payload is the direct ReportData or wrapped in a data property
+                const data = (raw && typeof raw === 'object' && 'data' in raw ? raw.data : raw);
+                if (data)
+                    parsedData.push(data);
             }
             catch (err) {
-                console.error(`[SnapAlly] Failed to parse A11y attachment: ${err}. Body was: ${a11yAttachment.body.toString().substring(0, 100)}`);
+                console.error(`[SnapAlly] Failed to parse A11y attachment: ${err}. Body was: ${att.body.toString().substring(0, 100)}`);
             }
         }
-        // Handle cases where a11yData might be the direct ReportData or wrapped in a data property
-        const actualData = (a11yData && typeof a11yData === 'object' && 'data' in a11yData ? a11yData.data : a11yData);
-        const violations = (actualData === null || actualData === void 0 ? void 0 : actualData.a11yErrors) || (actualData === null || actualData === void 0 ? void 0 : actualData.violations) || [];
+        // Use the first scan for page-level metadata (pageUrl, ADO settings, ...),
+        // but aggregate violations across every scan in the test.
+        const actualData = parsedData[0] || null;
+        const violations = parsedData.flatMap((d) => d.a11yErrors || d.violations || []);
         const a11yErrorCount = violations.reduce((acc, curr) => { var _a, _b; return acc + (curr.total || ((_a = curr.target) === null || _a === void 0 ? void 0 : _a.length) || ((_b = curr.nodes) === null || _b === void 0 ? void 0 : _b.length) || 0); }, 0);
         const filteredSteps = (() => {
             const blocklist = ['Evaluate', 'Create page', 'Close page', 'Before Hooks', 'After Hooks', 'Worker Teardown', 'Worker Cleanup', 'Attach', 'Wait for timeout', 'Capture A11y screenshot', 'Scroll into view', 'Bounding box'];

package/dist/templates/report-app.js

@@ -109,14 +109,22 @@ function renderTestExecutionReport(data) {
     }
 
     // Helper method for array elements
-    const renderList = (array, listId, cardId) => {
+    const renderList = (array, listId, cardId, asHtml = false) => {
         if (array && array.length > 0) {
             document.getElementById(cardId).classList.remove('hidden');
             const list = document.getElementById(listId);
             const tpl = document.getElementById('string-item-template');
             array.forEach((item) => {
                 const clone = tpl.content.cloneNode(true);
-                clone.querySelector('li').textContent = item;
+                const li = clone.querySelector('li');
+                // Exceptions arrive as pre-sanitized HTML from ansiToHtml (entities
+                // already escaped, only known <span> color tags injected), so render
+                // them as HTML to show the colors. Everything else is plain text.
+                if (asHtml) {
+                    li.innerHTML = item;
+                } else {
+                    li.textContent = item;
+                }
                 list.appendChild(clone);
             });
         }
@@ -130,7 +138,7 @@ function renderTestExecutionReport(data) {
     const filteredErrs = (data.errors || []).filter(
         (err) => !err.includes('Accessibility audit failed')
     );
-    renderList(filteredErrs, 'list-exceptions', 'card-exceptions');
+    renderList(filteredErrs, 'list-exceptions', 'card-exceptions', true);
 
     // Normalize to an array of videos
     const videos = Array.isArray(data.videoPath)

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "snap-ally",
-    "version": "1.0.2",
+    "version": "1.0.3",
     "description": "A custom Playwright reporter for Accessibility testing using Axe, with HTML reporting and Azure DevOps integration.",
     "main": "dist/index.js",
     "types": "dist/index.d.ts",