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)

Other
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 934230

People

(Reporter: zcampbell, Assigned: zcampbell)

References

Details

Attachments

(2 files)

      No description provided.
Blocks: 929643
No longer blocks: 929643
Just awaiting a new release to Pypi of marionette client
Attached file github pr
Attachment #821534 - Flags: review?(florin.strugariu)
Comment on attachment 821534 [details] [review]
github pr

Looks great, thanks Zac. I'll leave the r? for Bebe though.
Attachment #821534 - Flags: feedback+
Yep, can't wait to do it on device tests too!
Attachment #821534 - Flags: review?(florin.strugariu) → review+
Thx!
https://github.com/mozilla-b2g/gaia/commit/7a16280b3809457def85ada076f1f0259153917e
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
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.
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
Ok, same tests are failing with the revert, this was a false alert then won't push the revert.
oki thanks Zac, will do this.
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)
Blocks: 934130
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #826435 - Flags: review?(andrei.hutusoru) → review+
Status: REOPENED → RESOLVED
Closed: 11 years ago11 years ago
Resolution: --- → DUPLICATE
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.

Attachment

General

Creator:
Created:
Updated:
Size: