Closed
Bug 947957
Opened 12 years ago
Closed 12 years ago
Enable test_quick_settings.py on Travis and TBPL
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(b2g-v1.2 wontfix)
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
b2g-v1.2 | --- | wontfix |
People
(Reporter: bsilverberg, Assigned: AndreiH)
References
Details
Attachments
(1 file, 1 obsolete file)
The test was added to master and v1.2 and can be enabled on Travis and TBPL but that has not been done yet. This bug is to update the manifests to allow it to run on both.
Reporter | ||
Updated•12 years ago
|
status-b2g-v1.2:
--- → affected
Assignee | ||
Comment 1•12 years ago
|
||
Attachment #8348032 -
Flags: review?(florin.strugariu)
Attachment #8348032 -
Flags: review?(bob.silverberg)
Assignee | ||
Comment 2•12 years ago
|
||
Attachment #8348034 -
Flags: review?(florin.strugariu)
Attachment #8348034 -
Flags: review?(bob.silverberg)
Assignee | ||
Comment 3•12 years ago
|
||
And I guess the test is enabled by default in Travis
Updated•12 years ago
|
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Updated•12 years ago
|
Attachment #8348034 -
Attachment is obsolete: true
Attachment #8348034 -
Flags: review?(florin.strugariu)
Attachment #8348034 -
Flags: review?(bob.silverberg)
Comment 5•12 years ago
|
||
Comment on attachment 8348032 [details] [review]
Master PR https://github.com/mozilla-b2g/gaia/pull/14707
r-, fix ordering
Attachment #8348032 -
Flags: review-
Updated•12 years ago
|
Attachment #8348032 -
Flags: review?(florin.strugariu) → review-
Assignee | ||
Comment 6•12 years ago
|
||
Comment on attachment 8348032 [details] [review]
Master PR https://github.com/mozilla-b2g/gaia/pull/14707
Fixed
Attachment #8348032 -
Flags: review?(zcampbell)
Attachment #8348032 -
Flags: review?(florin.strugariu)
Attachment #8348032 -
Flags: review-
Comment 7•12 years ago
|
||
Comment on attachment 8348032 [details] [review]
Master PR https://github.com/mozilla-b2g/gaia/pull/14707
r+, we can merge when the tree is reopened.
but AndreiH you have to watch TBPL after merging to be sure that this does pass on TBPL and be prepared to revert this patch if it fails.
Attachment #8348032 -
Flags: review?(zcampbell) → review+
Updated•12 years ago
|
Attachment #8348032 -
Flags: review?(florin.strugariu) → review+
Comment 8•12 years ago
|
||
Comment 9•12 years ago
|
||
Leaving it open for Andrei to close it after it checks the TBPL
Flags: needinfo?(andrei.hutusoru)
Reporter | ||
Comment 10•12 years ago
|
||
People, please remember to cancel any outstanding reviews for an attachment after it is merged.
Reporter | ||
Updated•12 years ago
|
Attachment #8348032 -
Flags: review?(bob.silverberg)
Assignee | ||
Comment 11•12 years ago
|
||
Closing this as TBPL is ok
Status: REOPENED → RESOLVED
Closed: 12 years ago → 12 years ago
Flags: needinfo?(andrei.hutusoru)
Resolution: --- → FIXED
Updated•12 years ago
|
Assignee: nobody → andrei.hutusoru
You need to log in
before you can comment on or make changes to this bug.
Description
•