Closed Bug 1188611 Opened 9 years ago Closed 9 years ago

trychooser syntax is not granular enough for Android tests

Categories

(Infrastructure & Operations Graveyard :: CIDuty, task)

task
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kmoir, Assigned: aselagea)

References

Details

Attachments

(1 file, 1 obsolete file)

Does not differentiate between different Android test platforms, it's all Android or none. Should be able to just run Android API 11 or API 9
Assignee: nobody → kmoir
recent changes to add api15 too in bug 1219094
Assignee: kmoir → nobody
Component: Tools → Buildduty
QA Contact: hwine → bugspam.Callek
Assignee: nobody → alin.selagea
Attached patch bug1188611.patch (obsolete) (deleted) — Splinter Review
Not adding API 11+ since it has been removed from trunk (and implicitly from try) in bug 1073772. I may be missing something here though, so a feedback will be much appreciated :-)
Attachment #8714791 - Flags: feedback?(kmoir)
Comment on attachment 8714791 [details] [diff] [review] bug1188611.patch I don't know that you need the "data project" part which seems to be associated with selecting the builds, not tests. Also, keep the values uppercase to remain consistent with the other test platform names.
Right, data-project both doesn't belong there and is also (fortunately) dead code - bug 1245730.
Attachment #8714791 - Flags: feedback?(kmoir)
Attached patch bug1188611.patch (deleted) — Splinter Review
Attached a new version.
Attachment #8714791 - Attachment is obsolete: true
Attachment #8716344 - Flags: review?(kmoir)
Attachment #8716344 - Flags: review?(kmoir) → review+
Attachment #8716344 - Flags: checked-in+
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Product: Release Engineering → Infrastructure & Operations
Product: Infrastructure & Operations → Infrastructure & Operations Graveyard
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: