From 29e74a708a616ab130c5d3ffa4c23227c1c6c2aa Mon Sep 17 00:00:00 2001 From: fullsend-code <278716306+fullsend-ai-coder[bot]@users.noreply.github.com> Date: Thu, 25 Jun 2026 14:11:59 +0000 Subject: [PATCH] fix(#3579): add missing a11y assertion to runAccessibilityTests The runAccessibilityTests helper in the bulk-import e2e tests collected accessibility scan results and attached them to the test info, but never asserted that violations were empty. This meant tests would silently pass even if accessibility violations were present. Add expect(accessibilityScanResults.violations).toEqual([]) to match the pattern used in other workspaces (global-header, lightspeed, extensions, etc.). Closes #3579 --- workspaces/bulk-import/e2e-tests/utils/helpers.ts | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/workspaces/bulk-import/e2e-tests/utils/helpers.ts b/workspaces/bulk-import/e2e-tests/utils/helpers.ts index d42b51d560..23ce8b0647 100644 --- a/workspaces/bulk-import/e2e-tests/utils/helpers.ts +++ b/workspaces/bulk-import/e2e-tests/utils/helpers.ts @@ -75,4 +75,9 @@ export async function runAccessibilityTests( body: JSON.stringify(accessibilityScanResults, null, 2), contentType: 'application/json', }); + + expect( + accessibilityScanResults.violations, + 'Accessibility violations found', + ).toEqual([]); }