Investigate test_persona_cookie.py

RESOLVED FIXED

Status

Firefox OS
Gaia::UI Tests
P1
normal
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: zac, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

46 bytes, text/x-github-pull-request
Details | Review | Splinter Review
(Reporter)

Description

5 years ago
This suddenly started failing before for an unknown reason.
(Reporter)

Comment 1

5 years ago
Seems like it just needs a larger timeout.
(Reporter)

Comment 2

5 years ago
Created attachment 8371025 [details] [review]
github pr
Attachment #8371025 - Flags: review?(florin.strugariu)
Attachment #8371025 - Flags: review?(andrei.hutusoru)
Comment on attachment 8371025 [details] [review]
github pr

R+ But we are blocked because of the keyboard issue.
Attachment #8371025 - Flags: review?(florin.strugariu) → review+
merge this after the keyboard is fixed

Comment 5

5 years ago
The timeout looks ok there, but the test currently fails from the keyboard issue not being displayed, so I think it's better to hold off a little bit.

Comment 7

5 years ago
I started an adhoc job because I wanted to be sure. I got the same fail locally as in the adhoc job.
The "throbber" is still loading, this is not disappearing. I guess this is from the bidpom site.

the test fails here:
https://github.com/mozilla-b2g/gaia/blob/master/tests/python/gaia-ui-tests/gaiatest/apps/browser/app.py#L74
(Reporter)

Comment 8

5 years ago
That's very odd! 
I'll investigate more.
Richard: would you be able to help us investigate this with your Hamachi?  Thx!
Flags: needinfo?(rpappalardo)
(Reporter)

Comment 10

5 years ago
This is fine on device but I've decided we'll keep it disabled on Travis as it has two external dependencies and occasionally one may fail.
We still have test_persona_app.py on Travis which smoketests the Persona on desktopb2g.

I'll re-adjust my pull request to suit this.
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Flags: needinfo?(rpappalardo)
Resolution: --- → FIXED
(Reporter)

Comment 11

5 years ago
Comment on attachment 8371025 [details] [review]
github pr

re-r?
Attachment #8371025 - Flags: review+ → review?(florin.strugariu)
(Reporter)

Updated

5 years ago
Attachment #8371025 - Flags: review?(florin.strugariu)
Attachment #8371025 - Flags: review?(andrei.hutusoru)
You need to log in before you can comment on or make changes to this bug.