[camera/still] window that opens to display still image doesn't close

Hello Jeff, or anyone else affected,

Accepted checkbox into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/checkbox/0.13.8 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please change the bug tag from verification-needed to verification-done. If it does not, change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

I tested this on a Precise system with xawtv installed. Checkbox 0.13.7 is present first. I launched checkbox-qt, deselected all tests, and selected only the camera category (3 tests). Then I started the run:

- First test to come up is camera/display. I click "test". A live video window opens, then closes after about 10 seconds. I clicked on "Yes" to mark the test as passed.
- Next comes camera/still. A capture window with a picture of me (heh) comes up. No indication I have to do anything, and the window remains there. I counted 30 seconds, then closed the window. Marked the test as passed.
- The test run ends and I get to the report screen, but it's not relevant to this fix so I just closed checkbox.

I then upgraded checkbox and checkbox-qt to 0.13.8 from -proposed. I used the same procedure as above:
- First test to come up is camera/display. I click "test"Behaves as before, opening a video window and closing automatically after 10 seconds. I clicked on "Yes" to mark the test as passed.
- Next comes camera/still. A capture window with a picture of me (heh) comes up. No indication I have to do anything, and the window remains there. I counted 30 seconds, then closed the window. This constitutes a FAILURE because the window was supposed to close after 10 seconds.

Upon closer inspection of the code, I see that we call eog in two different places. This is because we try calling v4lctl first (first call to eog; I didn't apply the fix here). If v4lctl (which comes with xawtv) is not installed, we fall back to using gstreamer to capture the image (this is where I applied the fix).

A default Ubuntu system will not have xawtv installed, so I removed xawtv and tried to run the test again. I realized that the entire fallback code has two problems and doesn't work: first, the exception raised when v4lctl is not present is not CalledProcessError, but OSError (which we're not catching). Then, the call() where we invoke eog uses filename, which is not defined (should be file.name).

So all in all the fix was applied in the wrong place and does not work, I have unfortunately to mark this as verification-failed.

Thanks!

Hello Jeff, or anyone else affected,

Accepted checkbox into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/checkbox/0.13.9 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please change the bug tag from verification-needed to verification-done. If it does not, change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

This bug was fixed in the package checkbox - 0.13.9

---------------
checkbox (0.13.9) precise-proposed; urgency=low

  * Updated fix for bug #1026614 which was missing some code.
  * Removed bad fix for bug 990133.

checkbox (0.13.8) precise-proposed; urgency=low

  * New upstream release (LP: #1061198)

  [Marc Tardif]
  * checkbox/dispatcher.py: Updated license to GPLv3 to be consistent with
    rest of Checkbox files (LP: #1049881)

  [Sylvain Pineau]
  * qt/frontend/qtfront.ui: Fixed the test purpose widget size to allow two
    lines of description (LP: #1032255)
  * scripts/camera_test, jobs/camera.txt.in: Added a 10s timeout to the camera
    still test (LP: #990133)

  [Daniel Manrique]
  * jobs/audio.txt.in: replaced gconfaudiosink by autoaudiosink (LP: #978895)
  * Instruct Chromium browser to accept file:// URLs so it can correctly
    open the checkbox submission.xml report (LP: #1026614)
  * scripts/network_check: InvalidURL exception is caught and handled more
    cleanly (LP: #751701)
  * Set the correct user (root) for fwts-wakealarm test (LP: #1004102)
  * Set correct user (root) for usb/storage-preinserted, so it works correctly
    on servers (LP: #1004131)

  [Nathan Williams]
  * scripts/network_check: Fixed exception handling in the absence of zenity
    (LP: #988260)
 -- Daniel Manrique <email address hidden> Tue, 20 Nov 2012 16:31:02 -0500