Skip to content

Commit

Permalink
Merge pull request #16781 from viorelaioia/bug_978099
Browse files Browse the repository at this point in the history
Bug 978099 - Better wait in test_video_empty.TestVideoEmpty
  • Loading branch information
Zac committed Mar 3, 2014
2 parents 06a8c1d + dc7a128 commit fbb6faa
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 7 deletions.
4 changes: 0 additions & 4 deletions tests/python/gaia-ui-tests/gaiatest/apps/videoplayer/app.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ class VideoPlayer(Base):

name = 'Video'

_progress_bar_locator = (By.ID, 'throbber')
_thumbnails_locator = (By.ID, 'thumbnails')

# Video list/summary view
Expand All @@ -27,9 +26,6 @@ def wait_for_thumbnails_to_load(self, files_number):
self.wait_for_condition(lambda m: len(m.find_elements(*self._video_items_locator)) == files_number,
timeout=files_number * 5)

def wait_for_progress_bar_not_visible(self):
self.wait_for_element_not_displayed(*self._progress_bar_locator)

@property
def total_video_count(self):
return len(self.marionette.find_elements(*self._video_items_locator))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -18,10 +18,9 @@ def test_empty_video(self):

video_player = VideoPlayer(self.marionette)
video_player.launch()
video_player.wait_for_progress_bar_not_visible()

# Verify title when no videos
self.assertEqual(video_player.empty_video_title, 'Add videos to get started')
# Wait for title when no videos
self.wait_for_condition(lambda m: video_player.empty_video_title == 'Add videos to get started')

# Verify text when no videos
# Note: Text will need to be updated if/when Bug 834477 is fixed
Expand Down

0 comments on commit fbb6faa

Please sign in to comment.