Closed
Bug 1517298
Opened 6 years ago
Closed 6 years ago
Move Thunderbird mozharness unittest configs out of m-c
Categories
(Thunderbird :: Testing Infrastructure, defect)
Thunderbird
Testing Infrastructure
Tracking
(Not tracked)
RESOLVED
FIXED
Thunderbird 67.0
People
(Reporter: rjl, Assigned: rjl)
References
Details
Attachments
(5 files)
In bug 1506715 comment 8, Geoff Brown [:gbrown] stated:
> The mochitest harness changes are fine, but I think you need a clearer
> separation of thunderbird from the mozharness configuration. Ideally, there
> shouldn't be thunderbird mozharness configuration in mozilla-central.
The end result of this bug is that thunderbird_extra.py is moved to comm/mozharness/unittests and thunderbird_buildbot.py is completely removed as it's obsolete.
Assignee | ||
Comment 1•6 years ago
|
||
First step in moving Thunderbird mozharness unittest configurations out of
M-C. This adds the comm/mozharness directory to the mozharness test archive
under mozharness/configs/comm so that they can be access later by
desktop_unittest.py.
Assignee | ||
Comment 2•6 years ago
|
||
Moving TB specific mozharness unittest configurations out of mozilla repository.
Assignee | ||
Comment 3•6 years ago
|
||
Adds the necessary taskcluster and mozharness configuration for Thunderbird
Mochitests.
Note that the addition of "all_mochitest_suites" in thunderbird_extra.py will
replace the inherited list of tests from mozilla.
Depends on D16375
Assignee | ||
Comment 4•6 years ago
|
||
The try runs for this series of patches look promising.
try:
- https://treeherder.mozilla.org/#/jobs?repo=try&revision=79fa8608ba3d43acae2a58de932e6aedba7ff47c
- Running a simple Firefox build to double check that it still works right.
try-comm-central:
- https://treeherder.mozilla.org/#/jobs?repo=try-comm-central&revision=651607f081a70b1017d10d85c90d14c7677923c7
- The mozharness.zip file created by the build steps contains the mozharness configuration files from comm- as expected. The unittests were able to access the files they needed to run.
- The Mochitests failed, but I don't think it was a CI issue.
Flags: needinfo?(geoff)
Comment 5•6 years ago
|
||
The Mochitests failed, but I don't think it was a CI issue.
I had a better test run with some prefs changed, so you're right.
Flags: needinfo?(geoff)
Updated•6 years ago
|
Attachment #9036080 -
Attachment description: Bug 1517298 - Add comm mozharness configs to mozharness test archive. r?gbrown → Bug 1517298 - Add comm mozharness configs to mozharness test archive.
Assignee | ||
Comment 7•6 years ago
|
||
thunderbird_buildbot.py is no longer necessary now that Thunderbird builds
on Taskcluster.
thunderbird_extra.py is moved into comm-*.
Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/integration/autoland/rev/8431bedf934c
Add comm mozharness configs to mozharness test archive. r=firefox-build-system-reviewers,mshal
Comment 9•6 years ago
|
||
bugherder |
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 66.0
Assignee | ||
Comment 10•6 years ago
|
||
reopening since there is more to land on this bug.
Comment 11•6 years ago
|
||
Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/comm-central/rev/655c34c266f6
Move TB unittest configs into comm repository. r=darktrojan
https://hg.mozilla.org/comm-central/rev/3c26a13818d0
TB Mochhitest configuration for taskcluster. r=darktrojan
Assignee | ||
Comment 12•6 years ago
|
||
Developer requested that mochitests only run on optimized builds for right
now.
Comment 13•6 years ago
|
||
Pushed by thunderbird@calypsoblue.org:
https://hg.mozilla.org/integration/autoland/rev/870a3cdb1a99
Remove Thunderbird unittest configs from mozilla-*. r=jmaher
Assignee | ||
Updated•6 years ago
|
Assignee | ||
Updated•6 years ago
|
Keywords: leave-open
Comment 14•6 years ago
|
||
bugherder |
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Assignee | ||
Comment 15•6 years ago
|
||
D17987, attachment 9039938 [details] needs to be committed.
Comment 16•6 years ago
|
||
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/2320b1ccb435
Set mochitests to only run for optimized builds. r=darktrojan
Status: REOPENED → RESOLVED
Closed: 6 years ago → 6 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•