Skip to content

Conversation

@edleeks87
Copy link
Contributor

Proposed behaviour

Removes superfluous pw tests
Tidies up chromatic snapshots

Current behaviour

PW is testing things already covered elsewhere
Chromatic is snapshotting popover in closed state

Checklist

  • Commits follow our style guide
  • Playwright automation tests added or updated if required
  • Storybook added or updated if required

QA

  • Tested in provided StackBlitz sandbox/Storybook
  • Add new Playwright test coverage if required
  • Carbon implementation matches Design System/designs
  • UI Tests GitHub check reviewed if required

Additional context

Testing instructions

@edleeks87 edleeks87 self-assigned this Feb 6, 2026
@edleeks87 edleeks87 requested review from a team as code owners February 6, 2026 11:31
@edleeks87 edleeks87 force-pushed the pop-container-pw-tests branch from ddc8add to bf9d3c9 Compare February 6, 2026 11:36
@edleeks87 edleeks87 changed the title test(popover-container): remove uneeded playwright tests test(popover-container): remove superfluous playwright tests Feb 6, 2026
@edleeks87 edleeks87 merged commit acb0fb5 into master Feb 9, 2026
28 checks passed
@edleeks87 edleeks87 deleted the pop-container-pw-tests branch February 9, 2026 15:09
@carbonci
Copy link
Collaborator

🎉 This PR is included in version 158.27.3 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants