Skip to content

bug(settings): Fix broken integration tests - #21017

Merged
dschom merged 1 commit into
mainfrom
fix-broken-integration-tests
Aug 12, 2026
Merged

bug(settings): Fix broken integration tests#21017
dschom merged 1 commit into
mainfrom
fix-broken-integration-tests

Conversation

@dschom

@dschom dschom commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Because

  • TimeoutAndCancel page had broken tests
  • A previous PR comment about decorating firefox.com/pair with <b> broke some tests.

This pull request

  • Fixes tests
  • Handles tags around firefox.com/pair

Issue that this pull request solves

Closes: (issue number)

Checklist

Put an x in the boxes that apply

  • My commit is GPG signed.
  • If applicable, I have modified or added tests which pass locally.
  • I have added necessary documentation (if appropriate).
  • I have verified that my changes render correctly in RTL (if appropriate).
  • I have manually reviewed all AI generated code.

How to review (Optional)

  • Key files/areas to focus on:
  • Suggested review order:
  • Risky or complex parts:

Screenshots (Optional)

Please attach the screenshots of the changes made in case of change in user interface.

Other information (Optional)

Any other information that is important to this pull request.

Because:
- TimeoutAndCancel page had broken tests

This Commit:
- Fixes tests
@dschom
dschom requested a review from a team as a code owner August 12, 2026 20:06

@vbudhram vbudhram left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@dschom You got more context on the fix 😅

@dschom
dschom merged commit aeb243a into main Aug 12, 2026
18 of 20 checks passed
@dschom
dschom deleted the fix-broken-integration-tests branch August 12, 2026 21:49
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.

2 participants