Closed
Bug 663197
Opened 11 years ago
Closed 11 years ago
remove a couple of directories from mochitest-1 for android tests
Categories
(Release Engineering :: General, defect, P3)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: jmaher, Assigned: bear)
References
Details
(Whiteboard: [android][tegra][mobile_unittests])
Attachments
(1 file)
1.02 KB,
patch
|
aki
:
review+
bear
:
checked-in+
|
Details | Diff | Splinter Review |
currently we have a list of directories that we test against for mochitest-1 on android. With our recent push to remove .xul and non DOM tests from mochitest and put them in mochitest-chrome, we have removed a couple of these directories from mochitest: content/xul/document/test content/xul/templates/tests currently these tests live in mochitest-chrome. please remove the above two directories from the list of directories we run for mochitest-1 on android.
Assignee | ||
Comment 1•11 years ago
|
||
joel - where would I remove these dirs from? I don't think we (releng) control these resources or am I being confused again.
Priority: -- → P3
Whiteboard: [android][tegra][mobile_unittests]
Comment 2•11 years ago
|
||
http://hg.mozilla.org/build/buildbot-configs/file/tip/mozilla-tests/config.py#l339
Comment 4•11 years ago
|
||
Do we have a bug already filed on moving those hard-coded lists of directories into the tree? You can fix it, but only at the cost of burning TraceMonkey and anyone else who is running Android mochitests until they merge.
Reporter | ||
Comment 5•11 years ago
|
||
this list of directories is a stepping stone until we can get all the mochitests running in android. Currently removing these directories (a few test files each) from the list of M1 will remove us testing on all branches (:bear, correct me if I am wrong here). Yes, we won't be getting the same coverage, but currently we have so many failing tests on mobile that anything we have continue to keep running which is green is 100% more valuable than not running tests at all.
Assignee | ||
Comment 6•11 years ago
|
||
Do I need to add a project/branch exception to the list? Or just shorten the list for the short-term
Reporter | ||
Comment 7•11 years ago
|
||
philor, can you start a bug thread for making this data driven. My ultimate goal is to have this chunked (as we do for desktop) once we fix/manifest out all tests which cannot run on mobile.
Assignee | ||
Comment 8•11 years ago
|
||
Attachment #538361 -
Flags: review?(aki)
Updated•11 years ago
|
Attachment #538361 -
Flags: review?(aki) → review+
Assignee | ||
Comment 9•11 years ago
|
||
Comment on attachment 538361 [details] [diff] [review] remove two mochitest's from the current list committed changeset 4259:0ca5275e446e
Attachment #538361 -
Flags: checked-in+
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•9 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•