Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Task/Issue URL: https://app.asana.com/1/137249556945/project/1148564399326804/task/1211575849635044?focus=true
CC:
Description
Adding tests for new NTP state management
Note: I time-boxed adding UI tests, but was blocked on reliably locating the search box. Since there are no tests for NTP for now, I decided to skip the test. There is a new initiative to deliver e2e tests for each feature, so I expect NTP will be raised and discussed there.
Testing Steps
Impact and Risks
None: Internal tooling, documentation
Internal references:
Definition of Done | Engineering Expectations | Tech Design Template
Note
Adds focused unit tests for NewTabPageStateProvider covering feature-flag gating and Combine publisher/state behavior.
macOS/UnitTests/NewTabPage/NewTabPageStateProviderTests.swift
.newTabPageTabIDs
vsnewTabPagePerTab
gating forgetState()
and publisher emissions.getState()
, maintaining consistency after enable/disable transitions.Written by Cursor Bugbot for commit 55c30b8. This will update automatically on new commits. Configure here.