firefox-ios
firefox-ios copied to clipboard
[Looking for Feedback] Add FXIOS-10164 [Homepage] More Pocket Section Tests
:scroll: Tickets
:bulb: Description
Added some more tests for the homepage rebuild - pocket section. These tests are currently passing, but looking for some feedback if this should be the standard of how we do tests. Feel free to leave feedback and can respond when I return!
:pencil: Checklist
You have to check all boxes before merging
- [x] Filled in the above information (tickets numbers and description of your work)
- [x] Updated the PR name to follow our PR naming guidelines
- [x] Wrote unit tests and/or ensured the tests suite is passing
- [ ] When working on UI, I checked and implemented accessibility (minimum Dynamic Text and VoiceOver)
- [ ] If needed, I updated documentation / comments for complex code and public methods
- [ ] If needed, added a backport comment (example
@Mergifyio backport release/v120)
| Messages | |
|---|---|
| :book: | Project coverage: 32.65% |
| :book: | Edited 6 files |
| :book: | Created 3 files |
Client.app: Coverage: 30.48
| File | Coverage | |
|---|---|---|
| PocketMiddleware.swift | 100.0% | ✅ |
| DependencyHelper.swift | 0.0% | ⚠️ |
| PocketManager.swift | 100.0% | ✅ |
Generated by :no_entry_sign: Danger Swift against 58b48cebb3cc0c2d1889b1ca09820202d6110d8b
This PR has been automatically marked as stale. Please leave any comment to keep this PR opened. It will be closed automatically if no further update occurs in the next 7 days. Thank you for your contributions!
Moving from draft to ready for review and will move back to draft if further work is needed to update these tests. Wanted to mainly get feedback if we should write our future tests this way.