Closed
Bug 1445352
Opened 7 years ago
Closed 6 years ago
no code-coverage for (at-least?) JSX files
Categories
(Firefox :: New Tab Page, enhancement, P3)
Firefox
New Tab Page
Tracking
()
People
(Reporter: dmosedale, Assigned: andreio)
References
Details
(Keywords: github-merged, Whiteboard: [chore])
Attachments
(1 file)
(deleted),
text/x-github-pull-request
|
Details |
STR:
`npm run testmc`
`npm run debugcoverage`
View the HTML coverage report, and notice that there are no JSX coverage reports generated.
Comment 1•7 years ago
|
||
https://github.com/mozilla/activity-stream/commit/fd1b60691 added code coverage and it reported only common/{Actions,Reducers}.jsm and lib/{ActivityStream,Store}.jsm being covered.
Although back then there were also no content-src tests. The first content-src test was added in https://github.com/mozilla/activity-stream/commit/d661e2df and coverage did include content-src directory but only content-src/lib/{detect-user-session-start,init-store}.js so still no jsx.
Updated•7 years ago
|
Severity: normal → enhancement
Iteration: --- → 61.2 - Apr 9
status-firefox60:
--- → wontfix
status-firefox61:
--- → affected
Priority: -- → P2
Updated•7 years ago
|
Iteration: 61.2 - Apr 9 → 61.3 - Apr 23
Updated•7 years ago
|
Iteration: 61.3 - Apr 23 → 61.4 - May 7
Updated•7 years ago
|
Iteration: 61.4 - May 7 → 62.1 - May 21
Updated•7 years ago
|
Priority: P2 → P3
Updated•7 years ago
|
Iteration: 62.1 - May 21 → 62.2 - Jun 4
Updated•7 years ago
|
Iteration: 62.2 - Jun 4 → 63.1 - July 9
Updated•6 years ago
|
Priority: P3 → P2
Updated•6 years ago
|
Whiteboard: [chore]
Updated•6 years ago
|
Iteration: 63.1 - July 9 → ---
Priority: P2 → P3
Assignee | ||
Updated•6 years ago
|
Assignee: nobody → andrei.br92
Comment 2•6 years ago
|
||
Comment 3•6 years ago
|
||
Status: NEW → RESOLVED
Iteration: --- → 68.2 - Apr 1 - 14
Closed: 6 years ago
status-firefox68:
--- → fixed
Keywords: github-merged
Resolution: --- → FIXED
Target Milestone: --- → Firefox 68
Updated•5 years ago
|
Updated•5 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
•