Skip to content

Log URL for failed accessibility tests#2895

Merged
lukey-luke merged 5 commits intomainfrom
ls/task-log-url-accessibility-tests
Mar 19, 2026
Merged

Log URL for failed accessibility tests#2895
lukey-luke merged 5 commits intomainfrom
ls/task-log-url-accessibility-tests

Conversation

@lukey-luke
Copy link
Copy Markdown
Contributor

@lukey-luke lukey-luke commented Jan 29, 2026

🎫 Ticket

No ticket

🛠 Changes

  • add after hook to log URLs of failed pages for accessibility tests rspec

ℹ️ Context

  • when working with design, there was some confusion about what pages failed accessibility and we determined URL would make it easier to track down issues
  • related conversation on slack HERE

🧪 Validation

Tested locally with make ci-portal-accessibility
Same known 10 errors show up as main, but URLs are included

Screenshot 2026-01-29 at 6 20 26 AM

@lukey-luke lukey-luke marked this pull request as ready for review February 27, 2026 16:25
@lukey-luke lukey-luke requested a review from a team as a code owner February 27, 2026 16:25
Copy link
Copy Markdown
Contributor

@ashley-weaver ashley-weaver left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@lukey-luke lukey-luke merged commit 5afa543 into main Mar 19, 2026
11 checks passed
@lukey-luke lukey-luke deleted the ls/task-log-url-accessibility-tests branch March 19, 2026 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants