1127762 - Fix intermittent failure in test_music_change_rating.py

Status: RESOLVED FIXED

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

Description

[Viorela Ioia [:viorela]]

Attachment: screenshot.png

Test test_music_change_rating.py is failing on mozilla-central: http://jenkins1.qa.scl3.mozilla.com/view/UI/job/flame-kk-319.mozilla-central.ui.functional.non-smoke.1/160/HTML_Report/
I was able to reproduce the failure locally, by running the automated test several times. It is failing intermittently, repro rate: 2/5
I watched the phone while running the test and I noticed that rating star view was not displayed when trying to give rating of 4.
Wait for the star to be visible before tapping on it might be a good solution to fix this failure. 

Traceback (most recent call last):
File "/var/jenkins/1/workspace/flame-kk-319.mozilla-central.ui.functional.non-smoke.1/.env/local/lib/python2.7/site-packages/marionette_client-0.8.7-py2.7.egg/marionette/marionette_test.py", line 283, in run
testMethod()
File "/var/jenkins/1/workspace/flame-kk-319.mozilla-central.ui.functional.non-smoke.1/tests/python/gaia-ui-tests/gaiatest/tests/functional/music/test_music_change_rating.py", line 46, in test_music_change_rating
self.assertEquals(player_view.star_rating, 4)
AssertionError: 0 != 4

Gaia-Rev        8238eeacc7030b2cdbf7ab4eba2f36779b702599
Gecko-Rev       https://hg.mozilla.org/mozilla-central/rev/29b05d283b00
Build-ID        20150129160230
Version         38.0a1
Device-Name     flame
FW-Release      4.4.2
FW-Incremental  eng.cltbld.20150129.194056
FW-Date         Thu Jan 29 19:41:07 EST 2015
Bootloader      L1TC100118D0

Comment 1

[Viorela Ioia [:viorela]]

Attachment: Github PR: https://github.com/mozilla-b2g/gaia/pull/27814

Comment 2

[Florin Strugariu [:Bebe]]

Comment on attachment 8557052 [details] [review]
Github PR: https://github.com/mozilla-b2g/gaia/pull/27814

Let's update that wait
:P

Comment 3

[Viorela Ioia [:viorela]]

Comment on attachment 8557052 [details] [review]
Github PR: https://github.com/mozilla-b2g/gaia/pull/27814

updated

Comment 4

[Florin Strugariu [:Bebe]]

https://github.com/mozilla-b2g/gaia/commit/edeb624cd5b5829d14a52a0dbc8bd3b7203b8331