Closed
Bug 1541359
Opened 7 years ago
Closed 7 years ago
Chore: Increase our frontend code coverage
Categories
(Firefox :: New Tab Page, task, P1)
Firefox
New Tab Page
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox68 | --- | fixed |
People
(Reporter: andreio, Assigned: andreio)
References
Details
(Keywords: github-merged)
Attachments
(1 file)
We now have code coverage for our jsx code.
Similar to how we periodically update our dependencies we should improve our coverage.
npm run debugcoverage will open the coverage report page
The settings/baseline is specified in karma.mc.config.js and is broken down into three different larger sections:
// All DS components
"content-src/components/DiscoveryStreamComponents/**/*.jsx"
// All asrouter components
"content-src/asrouter/**/*.jsx"
// Everything else
"content-src/components/**/*.jsx"
Add more tests for one or more of these sections and bump the coverage numbers in karma.mc.config.js.
A first step should be to create test files for the components that don't have one yet with a basic it("should render") test.
| Assignee | ||
Updated•7 years ago
|
Type: defect → task
Comment 1•7 years ago
|
||
| Assignee | ||
Updated•7 years ago
|
Assignee: nobody → andrei.br92
Updated•7 years ago
|
Iteration: --- → 68.2 - Apr 1 - 14
Priority: -- → P1
| Assignee | ||
Updated•7 years ago
|
Keywords: github-merged
Comment 2•7 years ago
|
||
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox68:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
Updated•7 years ago
|
Component: Activity Streams: Newtab → New Tab Page
You need to log in
before you can comment on or make changes to this bug.
Description
•