Closed
Bug 2018708
Opened 8 months ago
Closed 1 month ago
Migrate TopSites unit tests to Jest/RTL
Categories
(Firefox :: New Tab Page, enhancement)
Firefox
New Tab Page
Tracking
()
RESOLVED
FIXED
158 Branch
| Tracking | Status | |
|---|---|---|
| firefox158 | --- | fixed |
People
(Reporter: maxx, Assigned: thecount)
References
Details
Attachments
(1 external file, 2 obsolete files)
Migrate the large TopSites container test suite covering drag-and-drop interactions, keyboard navigation, Redux state changes, and
sponsored/search shortcut top site behaviors.
Updated tests:
- test/unit/content-src/components/TopSites.test.jsx
- test/unit/content-src/components/TopSites/SearchShortcutsForm.test.jsx
- test/unit/content-src/components/TopSites/TopSiteImpressionWrapper.test.jsx
| Reporter | ||
Comment 1•8 months ago
|
||
Migrates TopSites.test.jsx, SearchShortcutsForm.test.jsx, and
TopSiteImpressionWrapper.test.jsx from Enzyme/Karma to React Testing
Library/Jest.
- Adds test/unit/jest/content-src/components/TopSites.test.jsx with 53 tests
covering TopSites, TopSiteLink, TopSite, TopSiteForm, TopSiteFormInput,
TopSiteList, and TopSiteAddButton - Adds test/unit/jest/content-src/components/TopSites/SearchShortcutsForm.test.jsx
with 3 tests - Adds test/unit/jest/content-src/components/TopSites/TopSiteImpressionWrapper.test.jsx
with 6 tests - Uses Redux Provider with createStore for components with connected children
- Replaces wrapper.instance() calls for internal methods (_allowDrop,
onDragEvent, validateUrl, etc.) with observable DOM assertions; drag-and-drop
and keyboard navigation tests are omitted as implementation details - Marks all three Enzyme files with @enzyme-to-rtl comments
Updated•7 months ago
|
Attachment #9547562 -
Attachment is obsolete: true
| Assignee | ||
Comment 2•2 months ago
|
||
Updated•2 months ago
|
Assignee: nobody → sdowne
Status: NEW → ASSIGNED
Comment 3•2 months ago
|
||
Updated•2 months ago
|
Attachment #9618903 -
Attachment description: WIP: Bug 2018708 - Migrate TopSites unit tests to Jest/RTL → WIP: Bug 2018708 - Migrate TopSites unit tests to Jest/RTL. r?thecount
Updated•2 months ago
|
Attachment #9618903 -
Attachment description: WIP: Bug 2018708 - Migrate TopSites unit tests to Jest/RTL. r?thecount → WIP: Bug 2018708 - Migrate TopSites unit tests to Jest/RTL. r=#home-newtab-reviewers
Updated•2 months ago
|
Attachment #9618903 -
Attachment description: WIP: Bug 2018708 - Migrate TopSites unit tests to Jest/RTL. r=#home-newtab-reviewers → Bug 2018708 - Migrate TopSites unit tests to Jest/RTL. r=#home-newtab-reviewers
Updated•2 months ago
|
Attachment #9618709 -
Attachment is obsolete: true
| Assignee | ||
Updated•2 months ago
|
Updated•1 month ago
|
Attachment #9618903 -
Attachment description: Bug 2018708 - Migrate TopSites unit tests to Jest/RTL. r=#home-newtab-reviewers → Bug 2018708 - Migrate TopSites unit tests to Jest/RTL. r?reemhamz
Pushed by sdowne@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/1a63eb64ee39
https://hg.mozilla.org/integration/autoland/rev/5d28fee1167f
Migrate TopSites unit tests to Jest/RTL. r=reemhamz
Comment 5•1 month ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 1 month ago
status-firefox158:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 158 Branch
Updated•29 days ago
|
QA Whiteboard: [qa-triage-done-c159/b158]
You need to log in
before you can comment on or make changes to this bug.
Description
•