|Submitter||Diff||Changes||Open Issues||Last Updated|
|Error loading review requests:|
now that web extensions will be default in firefox 57, I would like to ensure we are disabling tests that are not needed- specifically the jetpack job.
:andym- does this bug sound correct?
Sounds correct thanks, I think :kmag wanted to turn off the jetpack tests in the 57 cycle to save some test cycles for everyone.
Yes, I was planning to write a patch to disable all SDK tests other than the loader tests today. The loader tests can run in a regular mochitest job, though.
Created attachment 8893016 [details] [diff] [review] remove jetpack job from taskcluster configs
Attachment #8893016 - Flags: review?(kmaglione+bmo)
Comment on attachment 8893016 [details] [diff] [review] remove jetpack job from taskcluster configs Review of attachment 8893016 [details] [diff] [review]: ----------------------------------------------------------------- ::: taskcluster/ci/test/tests.yml @@ -906,5 @@ > - linux.*: > - - unittests/linux_unittest.py > - - remove_executables.py > - extra-options: > - - --mochitest-suite=jetpack-package We still need to run the jetpack-package suite, for now, since the devtools still use the SDK loader. After my other patch, the loader tests will be the only ones in that suite, though, so they can be moved to another mochitest job without too much disruption.
Attachment #8893016 - Flags: review?(kmaglione+bmo) → review-
Comment on attachment 8893010 [details] Bug 1386694: Part 1 - Disable most SDK tests. https://reviewboard.mozilla.org/r/164012/#review169352
Attachment #8893010 - Flags: review?(dtownsend) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/f6cdb28c2f2a7788d5909cd0cb3422a580475a4b Bug 1386694: Part 1 - Disable most SDK tests. r=Mossop
Status: NEW → RESOLVED
Last Resolved: 11 months ago
status-firefox57: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla57
I believe there is a bit more work to do here to shift the remaining jetpack-package suite into regular mochitests.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
You need to log in before you can comment on or make changes to this bug.