Interrupted runs are recovered even if the user selects "No" when prompted

[Daniel Manrique]

Tested with Checkbox trunk on Oneiric and with 0.11.3 on Natty.

Steps to reproduce:

- run checkbox-gtk
- On the test selection screen, deselect all tests and mark just a few categories.
- Start the test run.
- When on the first test screen, close checkbox.
- Start checkbox-gtk again.
- When prompted to recover the incomplete test run, select "No".

Expected behavior:
- Selection screen reset to its initial state.

Actual behavior:
- Old run data remains, including prior test selection.

The usual workaround is to delete the checkbox data directory, either $HOME/.checkbox or $HOME/.cache/checkbox. However, if checkbox offers to start a new run, it should do so correctly.

[Brendan Donegan (brendan-donegan)]

The only fundamental difference between Yes and No seems to be that Yes brings you directly to the last test run whereas No does show the test selection screen and then returns to the last run test (I can only guess what happens if you deselected the saved test from the list)

[Brendan Donegan (brendan-donegan)]

It seems that deleting just the 'store' directory does the trick, so I think the best path is to have the jobs_prompt plugin receive an event when the user selects 'No' to recover (something like begin-reset) and that can call self.store.delete_all_messages or suchlike.

[Brendan Donegan (brendan-donegan)]

I tried adding an event to the jobs_prompt plugin but it doesn't seem to be receiving it when it's fired in the recover_prompt plugin. Currently investigating why

[Brendan Donegan (brendan-donegan)]

Just had to modify the header of the function I added to handle the reset-jobs event. Now it works.

[Launchpad Janitor (janitor)]

This bug was fixed in the package checkbox - 0.12.7

---------------
checkbox (0.12.7) oneiric; urgency=low

  New upstream release (LP: #850395):

  [Brendan Donegan]
  * Redirecting stderr to pipe to fix the gconf_resource script (LP: #832321)
  * Clear jobs directory when user selects No to recover question (LP: #836623)

  [Daniel Manrique]
  * checkbox/job.py: Guard against bogus timeout values (LP: #827859)
  * More explicit handling of string decoding/encoding, avoids problems with
    non-ascii characters (LP: #833747)
  * Changed architecture from all to any for checkbox base, to build
    architecture-specific binaries (LP: #833696)

  [Jeff Lane]
  * Several corrections necessary due to test name changes or typos found in
    job files

  [Marc Tardif]
  * Connecting hyper text widgets only once (LP: #827904)
  * Detecting MMC readers as OTHER instead of DISK (LP: #822948)
  * Validating the hostname in the SSL certificate (LP: #625076)
  * Validating the submission.xml (LP: #838123)
 -- Daniel Manrique <email address hidden> Fri, 14 Sep 2011 17:15:26 -0400