Closed
Bug 929636
Opened 11 years ago
Closed 11 years ago
Change appropriate tests to fail-if for Travis/desktopb2g manifest
Categories
(Firefox OS Graveyard :: Gaia::UI Tests, defect)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 934230
People
(Reporter: zcampbell, Assigned: zcampbell)
References
Details
Attachments
(2 files)
No description provided.
Assignee | ||
Comment 1•11 years ago
|
||
Just awaiting a new release to Pypi of marionette client
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #821534 -
Flags: review?(florin.strugariu)
Comment 3•11 years ago
|
||
Comment on attachment 821534 [details] [review] github pr Looks great, thanks Zac. I'll leave the r? for Bebe though.
Attachment #821534 -
Flags: feedback+
Assignee | ||
Comment 4•11 years ago
|
||
Yep, can't wait to do it on device tests too!
Updated•11 years ago
|
Attachment #821534 -
Flags: review?(florin.strugariu) → review+
Assignee | ||
Comment 5•11 years ago
|
||
Thx! https://github.com/mozilla-b2g/gaia/commit/7a16280b3809457def85ada076f1f0259153917e
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Comment 6•11 years ago
|
||
This bug has another commit in : https://github.com/mozilla-b2g/gaia/commit/afbf45f26a73b7cd5e0a831bea48087331975286 With this commit (not the first one), one ui test is failing on Travis, see https://travis-ci.org/mozilla-b2g/gaia/jobs/13048230 Do you know why ? I'm not very happy that you didn't take the time to file a PR and wait for a green travis before merging this other commit (that isn't even mentioned on this bug). I'm backing out that other commit right now.
Comment 7•11 years ago
|
||
Travis for the revert is https://travis-ci.org/mozilla-b2g/gaia/builds/13077295
Assignee | ||
Comment 8•11 years ago
|
||
Julien, test_browser_bookmark is now failing because Rik backed out the Gecko change that was causing it to fail. It is *NOT* this patch causing it. If there is a commit to back out it is this one: https://github.com/mozilla-b2g/gaia/commit/d0a48341b9f258f9971d79f225ceba58e382eceb
Comment 9•11 years ago
|
||
Ok, same tests are failing with the revert, this was a false alert then won't push the revert.
Comment 10•11 years ago
|
||
oki thanks Zac, will do this.
Comment 11•11 years ago
|
||
Pointer to Github pull-request
Comment 12•11 years ago
|
||
Comment on attachment 826435 [details] Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/13324 Uplift to v1.2
Attachment #826435 -
Flags: review?(zcampbell)
Attachment #826435 -
Flags: review?(trifandreialin)
Attachment #826435 -
Flags: review?(moz.teodosia)
Attachment #826435 -
Flags: review?(florin.strugariu)
Attachment #826435 -
Flags: review?(andrei.hutusoru)
Updated•11 years ago
|
Updated•11 years ago
|
Attachment #826435 -
Flags: review?(andrei.hutusoru) → review+
Updated•11 years ago
|
Status: REOPENED → RESOLVED
Closed: 11 years ago → 11 years ago
Resolution: --- → DUPLICATE
Updated•11 years ago
|
Attachment #826435 -
Flags: review?(zcampbell)
Attachment #826435 -
Flags: review?(trifandreialin)
Attachment #826435 -
Flags: review?(moz.teodosia)
Attachment #826435 -
Flags: review?(florin.strugariu)
You need to log in
before you can comment on or make changes to this bug.
Description
•