Closed
Bug 787352
Opened 12 years ago
Closed 11 years ago
cfx test should pass on Fennec
Categories
(Add-on SDK Graveyard :: General, defect, P1)
Add-on SDK Graveyard
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: evold, Assigned: jsantell)
References
Details
At the moment many of our tests don't pass on Fennec. We should either write/fix tests we have to work on Fennec, or bypass the ones that won't work on Fennec.
Reporter | ||
Updated•12 years ago
|
Reporter | ||
Updated•12 years ago
|
Blocks: sdk-on-fennec
Comment 3•12 years ago
|
||
Sounds easier to test only *erik-certified* modules.
Matteo: what is the status of application support metadata, it could be an handy way to say which test (and not modules from lib/ foldes) need to be run on fennec.
Comment 4•12 years ago
|
||
Alex, there is a meeting scheduled for Monday where we will discuss about it.
Personally, I think we will easily find an agreement about "regular" supports, so we could implement them straight forward. I believe it will take a little more to decide how to support "aggregation module" (modules that exposes different implementation depends by the application), and how to specify the compatibility.
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → evold
Depends on: 802580
Summary: Need all package tests to pass on fennec → Need all tests to pass on fennec
Reporter | ||
Updated•11 years ago
|
Reporter | ||
Updated•11 years ago
|
Summary: Need all tests to pass on fennec → cfx testall should pass on Fennec
Reporter | ||
Updated•11 years ago
|
Summary: cfx testall should pass on Fennec → cfx test should pass on Fennec
Reporter | ||
Updated•11 years ago
|
Assignee: evold → nobody
Assignee | ||
Updated•11 years ago
|
Assignee: nobody → jsantell
Assignee | ||
Comment 5•11 years ago
|
||
`cfx test` is now passing. We can open individual bugs for future issues.
4411 of 4411 tests passed.
2275.051570 seconds
cfx test -a fennec-on-device -b adb --mobile-app fennec --force-mobile -o -v
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•