Closed Bug 937068 Opened 12 years ago Closed 12 years ago

Enhance wait in test_contact_add_photo

Categories

(Firefox OS Graveyard :: Gaia::UI Tests, defect, P1)

Product:
Firefox OS Graveyard
Component:
Gaia::UI Tests
Platform:
Other
Gonk (Firefox OS)
Type:
defect

Tracking

(b2g-v1.2 fixed)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
b2g-v1.2 --- fixed

People

(Reporter: zcampbell, Assigned: moz.teodosia)

Details

Attachments

(3 files)

test_add_photo_contact_hamachi.png
44.18 KB, image/png
Details
Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/13576
358 bytes, text/html
zcampbell
: review+
Bebe
: review+
Details
[v1.2]Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/13624
358 bytes, text/html
bsilverberg
: review+
Details
As you can see from the screenshot the Contacts app is still in 'loading' mode while the test is completed. The waits in this test need to be strengthened. Then the test should pass.
Assignee: nobody → moz.teodosia
The test is failing for me with issue in bug 934537 after I run the test more than once. I updated the xfail bug and added a wait.
Pointer to Github pull-request
Attachment #830158 - Flags: review?(zcampbell)
Attachment #830158 - Flags: review?(trifandreialin)
Attachment #830158 - Flags: review?(florin.strugariu)
Attachment #830158 - Flags: review?(andrei.hutusoru)
Comment on attachment 830158 [details] Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/13576 looks ok but we should rename the bug and get out the un-xfail
Attachment #830158 - Flags: review?(florin.strugariu) → review+
Comment on attachment 830158 [details] Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/13576 r+ too, I will merge
Attachment #830158 - Flags: review?(zcampbell) → review+
Summary: Unxfail and enhance wait in test_contact_add_photo → Enhance wait in test_contact_add_photo
Merged: https://github.com/mozilla-b2g/gaia/commit/9748748c3303b270274b880052395245ab1b6c82 Let's cherry-pick the enhanced wait snippet for v1.2 but leave the xfail code behind.
status-b2g-v1.2: --- → affected
Pointer to Github pull-request
Attachment #830845 - Attachment description: Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/13624 → [v1.2]Pointer to Github pull request: https://github.com/mozilla-b2g/gaia/pull/13624
Attachment #830845 - Flags: review?(zcampbell)
Attachment #830845 - Flags: review?(florin.strugariu)
Attachment #830845 - Flags: review?(andrei.hutusoru)
Attachment #830158 - Flags: review?(trifandreialin)
Attachment #830158 - Flags: review?(andrei.hutusoru)
The v1.2 branch is not affected by bug 934537. I ran the test several times and it passes.
Attachment #830845 - Flags: review?(zcampbell)
Attachment #830845 - Flags: review?(trifandreialin)
Attachment #830845 - Flags: review?(bob.silverberg)
Landed in v1.2: https://github.com/mozilla-b2g/gaia/commit/b29292e92f32b5a0ab14ea3346eae02d94cd2cd4
Status: NEW → RESOLVED
Closed: 12 years ago
status-b2g-v1.2: affected → fixed
Resolution: --- → FIXED
Attachment #830845 - Flags: review?(trifandreialin)
Attachment #830845 - Flags: review?(florin.strugariu)
Attachment #830845 - Flags: review?(bob.silverberg)
Attachment #830845 - Flags: review?(andrei.hutusoru)
Attachment #830845 - Flags: review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: