1242985 - Fix jetpack-package-harness to not mark as skipped test suites which fail to load

Status: RESOLVED INCOMPLETE

(Add-on SDK Graveyard :: General, defect)

Description

[Luca Greco [:rpl] [:luca] [:lgreco]]

In the process of inspecting the differences between the addon-sdk tests running on try and on travis, I noticed that on try the "test-event-dom.js" test file is skipped by mistake, and it actually fails to load (because of a bug in "test/util.js" it raises a "Cu is not defined" exception).

The test is marked as skipped because of the following line in the jetpack-package-harness: 

https://dxr.mozilla.org/mozilla-central/rev/aa90f482e16db77cdb7dea84564ea1cbd8f7f6b3/testing/mochitest/jetpack-package-harness.js#55

Comment 1

[Luca Greco [:rpl] [:luca] [:lgreco]]

Attachment: MozReview Request: Bug 1242985 - Fix jetpack-package-harness to not mark as skipped test suites which fail to load. r?mossop

Review commit: https://reviewboard.mozilla.org/r/32469/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/32469/

Comment 2

[Dave Townsend [:mossop]]

Comment on attachment 8712183 [details]
MozReview Request: Bug 1242985 - Fix jetpack-package-harness to not mark as skipped test suites which fail to load. r?mossop

https://reviewboard.mozilla.org/r/32469/#review29195

Comment 3

[Firefox Bug Husbandry Bot]

https://bugzilla.mozilla.org/show_bug.cgi?id=1399562