-
Notifications
You must be signed in to change notification settings - Fork 867
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
[Brave News]: Don't use TestingProfile in component unittests #25914
Conversation
Note: This needs to merge AFTER #25911 |
de01697
to
a5b3d9a
Compare
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.
++
optional: imo, test/BUILD.gn
should be in same directory with test files.
a5b3d9a
to
7681bfe
Compare
[puLL-Merge] - brave/brave-core@25914 DescriptionThis PR refactors the test setup for Brave News components, removing dependencies on Chrome-specific testing infrastructure and replacing them with more focused, component-specific test utilities. The changes aim to improve the modularity and independence of the Brave News tests. ChangesChanges
These changes collectively move the test infrastructure away from Chrome-specific components and towards more modular, component-specific testing utilities. This should make the tests more focused and easier to maintain, as well as reducing dependencies on Chrome internals. |
@simonhong agreed, but I'd rather do it as a separate PR if that's okay |
Huh, that is a bit weird. It could go either way, move the tests into the test directory or move the test target out of it, but follow up is fine |
Released in v1.73.14 |
Resolves brave/brave-browser#41534
Waiting on #25911 to merge firstmerged!Submitter Checklist:
QA/Yes
orQA/No
;release-notes/include
orrelease-notes/exclude
;OS/...
) to the associated issuenpm run test -- brave_browser_tests
,npm run test -- brave_unit_tests
wikinpm run presubmit
wiki,npm run gn_check
,npm run tslint
git rebase master
(if needed)Reviewer Checklist:
gn
After-merge Checklist:
changes has landed on
Test Plan:
N/A