Closed
Bug 1186233
Opened 11 years ago
Closed 11 years ago
Set test_packages_url in web platform tests script for those running outside of buildbot
Categories
(Release Engineering :: Applications: MozharnessCore, defect)
Release Engineering
Applications: MozharnessCore
Tracking
(firefox42 fixed)
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| firefox42 | --- | fixed |
People
(Reporter: chmanchester, Assigned: chmanchester)
Details
Attachments
(1 file)
No description provided.
|
Assignee | |
Comment 1•11 years ago
|
||
Bug 1186233 - Set test_packages_url correctly when running wpt from mozharness but outside of buildbot. r=jgraham
Attachment #8636829 -
Flags: review?(james)
|
||
Comment 2•11 years ago
|
||
https://reviewboard.mozilla.org/r/13719/#review12351
::: testing/mozharness/scripts/web_platform_tests.py:60
(Diff revision 1)
> + self.test_packages_url = c.get('test_packages_url')
This is fine in that it fixes the issue, but this is exactly the sort of thing that the TestBase superclass should handle so that we don't have to update every single test script each time a new property is added.
|
|
||
Comment 3•11 years ago
|
||
Comment on attachment 8636829 [details]
MozReview Request: Bug 1186233 - Set test_packages_url correctly when running wpt from mozharness but outside of buildbot. r=jgraham
https://reviewboard.mozilla.org/r/13721/#review12519
Ship It!
Attachment #8636829 -
Flags: review?(james) → review+
|
|
Assignee | |
Comment 4•11 years ago
|
||
This is not part of the build.
Assignee: nobody → cmanchester
Keywords: checkin-needed
Keywords: checkin-needed
|
||
Comment 6•11 years ago
|
||
You need to log in
before you can comment on or make changes to this bug.
Description
•