Ubuntu Autopilot Tests

Merge lp://staging/~noskcaj/ubuntu-autopilot-tests/xfce4-screenshooter into lp://staging/ubuntu-autopilot-tests

xfce4-screenshooter
Merge into trunk

Proposed by Jackson Doak on 2013-12-19

Status:	Approved
Approved by:	Dan Chapman  on 2013-12-27
Approved revision:	64
Proposed branch:	lp://staging/~noskcaj/ubuntu-autopilot-tests/xfce4-screenshooter
Merge into:	lp://staging/ubuntu-autopilot-tests
Diff against target:	31 lines (+25/-0) 1 file modified ubuntu_autopilot_tests/xfce4_screenshooter/test_xfce4_screenshooter.py (+25/-0)
To merge this branch:	bzr merge lp://staging/~noskcaj/ubuntu-autopilot-tests/xfce4-screenshooter
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Dan Chapman  (community)		2013-12-19	Approve on 2013-12-27
Review via email: mp+199723@code.staging.launchpad.net

Description of the change

Adds a basic test for xfce4-screenshooter.

Revision history for this message

Dan Chapman  (dpniel) wrote on 2013-12-27:

This looks good to me as a simple testcase. I will hold off on merging until I have spoken with balloons about best place for xubuntu tests to live. :-)

review: Approve

Revision history for this message

Nicholas Skaggs (nskaggs) wrote on 2014-01-23:

Dan, I believe we can merge this under the xubuntu_tests subdirectory. I'm honestly not opposed to leaving them all in one directory but the precedence to split into subdirectories is set in ubuntu-manual-tests, so I think we can follow it. I'll await your feedback, but +1 for me.

If at all possible I would like to see another assert added, but it might not be possible. For instance, is there a hotkey to take a screenshot we can press, and see if the app responds as it should?

Unmerged revisions

64. By Jackson Doak on 2013-12-19: Remove unneeded comment
63. By Jackson Doak on 2013-12-19: Make a working screenshooter test
62. By Jackson Doak on 2013-12-19: Merge from the main branch
61. By Jackson Doak on 2013-11-23: Add xfce4_screenshooter test, not yet working