Skip to content

TUL/Breadcrumbs cypress fix - #1155

Merged
milanmajchrak merged 6 commits into
customer/TULfrom
tul/breadcrumbs-cypress-fix
Dec 12, 2025
Merged

TUL/Breadcrumbs cypress fix#1155
milanmajchrak merged 6 commits into
customer/TULfrom
tul/breadcrumbs-cypress-fix

Conversation

@Kasinhou

@Kasinhou Kasinhou commented Dec 12, 2025

Copy link
Copy Markdown

Problem description

Failing breadcrumbs e2e test

Sync verification

If en.json5 or cs.json5 translation files were updated:

  • Run yarn run sync-i18n -t src/assets/i18n/cs.json5 -i to synchronize messages, and changes are included in this PR.

Manual Testing (if applicable)

Copilot review

  • Requested review from Copilot

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses a failing breadcrumbs e2e test by adding retry logic and timeout configuration to improve test stability. The changes align with patterns used throughout the Cypress test suite for handling flaky accessibility tests.

Key changes:

  • Adds retry configuration (8 retries in both run and open modes) consistent with other test files
  • Increases command timeout to 30 seconds (notably higher than the 10-second standard used elsewhere)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread cypress/integration/breadcrumbs.spec.ts Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@milanmajchrak
milanmajchrak merged commit 2c0fd79 into customer/TUL Dec 12, 2025
3 checks passed
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