-
Notifications
You must be signed in to change notification settings - Fork 34
Implements New PinnedTabs UI Tests #2695
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
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 implements new UI tests for pinned and unpinned tabs functionality in the macOS browser. It adds test coverage for drag-and-drop behavior, pin/unpin state management, and window creation scenarios related to tab manipulation.
- Adds 4 new test cases covering pinned/unpinned tab behaviors
- Introduces utility methods for tab dragging and assertion helpers
- Validates that pinned tabs cannot be dragged into new windows while unpinned tabs can
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 7 comments.
| File | Description |
|---|---|
| macOS/UITests/TabBarTests.swift | Adds test for preventing new window creation when dragging the only tab, plus shared drag/assertion utilities |
| macOS/UITests/PinnedTabsTests.swift | Adds three new tests for pin/unpin state toggling and drag behavior, plus supporting utility methods |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Task/Issue URL: https://app.asana.com/1/137249556945/project/1211150618152277/task/1211969382845380
Tech Design URL:
CC:
Description
In this PR we're adding several new UI Tests for Pinned / Unpinned Tabs.
Testing Steps
TabBarTestsSuite is green, locallyPinnedTabsTestsSuite is green as wellImpact and Risks
None: Internal tooling, documentation
What could go wrong?
Nothing really. This PR implements new UI tests, and does not alter any code that actually gets shipped.
Quality Considerations
Nothing in particular
Notes to Reviewer
Hope you don't mind @diegoreymendez, sending your way a brief UI Tests PR.
Thanks in advance!!
Internal references:
Definition of Done | Engineering Expectations | Tech Design Template
Note
Adds UI tests covering pin/unpin and cross-window drag behaviors for tabs, and sets the pinned tabs AX subrole to nil to ensure proper tab group accessibility.
macOS/UITests/PinnedTabsTests.swift):waitForSecondWindow,assertSingleWindowScenario,drag*TabAboveWindow,bringForemostWindowToForeground).macOS/DuckDuckGo/TabBar/View/TabBarViewController.swift):pinnedTabsCollectionViewaccessibility subrole tonilto align with.tabGrouprole and improve AX structure.Written by Cursor Bugbot for commit 5fe25dc. This will update automatically on new commits. Configure here.