Closed
Bug 673756
Opened 13 years ago
Closed 4 years ago
Add jscoverage reports to Lightning unit tests
Categories
(Calendar :: Build Config, defect, P3)
Calendar
Build Config
Tracking
(Not tracked)
RESOLVED
INVALID
People
(Reporter: Fallen, Unassigned)
References
Details
Attachments
(2 files)
5.22 KB,
patch
|
Details | Diff | Splinter Review | |
173.98 KB,
image/png
|
Details |
No description provided.
Reporter | ||
Comment 1•13 years ago
|
||
Unfortunately, jscoverage doesn't support e4x <http://siliconforks.com/jscoverage/bugs/37>. I've patched jscoverage to at least not choke on Thunderbird+Lightning (if you want to test this, be sure to use --no-highlight).
Afterwards it works pretty well, but of course our unit test coverage isn't that great yet. I'll attach a screenshot in a moment.
Reporter | ||
Comment 2•13 years ago
|
||
Reporter | ||
Comment 3•12 years ago
|
||
Now that we got rid of e4x, this should be much easier to put together. I guess we'd need to revamp our buildconfig though, or hack it in.
Reporter | ||
Updated•11 years ago
|
Priority: -- → P3
Comment 4•4 years ago
|
||
Philipp, do you think this bug reports needs to be kept for Lightning as the code is now integrated with Thunderbird?
Flags: needinfo?(philipp)
Reporter | ||
Comment 5•4 years ago
|
||
I'm fine closing this one. Almost 10 years later I'd still like to see code coverage from unit tests, but that is up to Thunderbird to provide.
Status: NEW → RESOLVED
Closed: 4 years ago
Flags: needinfo?(philipp)
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•