Merge lp://qastaging/~dpniel/ubuntu-autopilot-tests/evince-autopilot-1.3 into lp://qastaging/ubuntu-autopilot-tests

Proposed by Dan Chapman 
Status: Merged
Merged at revision: 46
Proposed branch: lp://qastaging/~dpniel/ubuntu-autopilot-tests/evince-autopilot-1.3
Merge into: lp://qastaging/ubuntu-autopilot-tests
Diff against target: 352 lines (+166/-146)
1 file modified
ubuntu_autopilot_tests/evince/test_evince.py (+166/-146)
To merge this branch: bzr merge lp://qastaging/~dpniel/ubuntu-autopilot-tests/evince-autopilot-1.3
Reviewer Review Type Date Requested Status
Nicholas Skaggs (community) Approve
Review via email: mp+167316@code.qastaging.launchpad.net

Description of the change

4 working tests
- Display pdf
- Changing pages
- Search document
- Side bar open and close

1 non-working test
- Fullscreen mode

One sleep has snuck its way in due to not being able to introspect the Yelp print dialog. Might be beacuse it introspecting a different application from within a different. Although vis shows Yelp and GtkPrintUnixDialog, the test didnt seem to like it. So have set the sleep to 10 seconds

To post a comment you must log in.
Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Looks good.. Almost got out without needing to fallback because of issues. We'll look into the wmstate for the fullscreen test too. Thanks!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches