Checkbox

Using the View Report button causes Checkbox-qt to behave and exit abnormally

Bug #956307 reported by Daniel Manrique
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox
Fix Released
Undecided
Tiago Salem Herrmann
Checkbox 0.13.5 "precise_fixes_3"

Bug Description

This tested on Checkbox 0.13.4.

Steps to reproduce:

1- Launch checkbox-qt
2- Deselect all tests
3- Select only "hotkey tests" (to do a quick test run)
4- Press "Start testing" and select "yes" on all tests
5- Click on 'View results'.
6- Close the browser that opens.
7- Exit checkbox-qt by closing the window.

Expected results:
- Browser opens to view the report
- No errors are logged
- Upon exit, a .cache/checkbox/plugins.bpickle with some configuration information remains.

Actual results:
- Browser does open correctly
- Trace 1 as shown below appears in the log file.
- When closing checkbox-qt, the plugins.bpickle file disappears and trace 2 (see further down below) appears in the logfile.

This keeps checkbox-qt from correctly remembering some user preferences, such as whether to resume or restart the run, and the abnormal exiting can cause unpredictable results.

TRACE 1:
2012-03-15 14:54:14,216 ERROR Exception in handler for D-Bus signal:
Traceback (most recent call last):
  File "/usr/lib/python2.7/dist-packages/dbus/connection.py", line 214, in maybe_handle_message
    self._handler(*args, **kwargs)
  File "/home/roadmr/Documents/checkboxes/checkbox/trunk/checkbox_qt/qt_interface.py", line 66, in onReviewTestsClicked
    self.show_url(self.report_url)
  File "/home/roadmr/Documents/checkboxes/checkbox/trunk/checkbox/user_interface.py", line 190, in show_url
    sys.exit(0)
SystemExit: 0

TRACE 2:
OSError: [Errno 2] No such file or directory: '/tmp/checkboxYxs5Pl'
2012-03-15 14:56:49,173 ERROR Error running reactor.
Traceback (most recent call last):
  File "/home/roadmr/Documents/checkboxes/checkbox/trunk/checkbox/application.py", line 52, in run
    self.reactor.run()
  File "/home/roadmr/Documents/checkboxes/checkbox/trunk/checkbox/reactor.py", line 112, in run
    self.fire("stop")
  File "/home/roadmr/Documents/checkboxes/checkbox/trunk/checkbox/reactor.py", line 74, in fire
    results.append(handler(*args, **kwargs))
  File "./plugins/persist_info.py", line 60, in save
    self.persist.save()
  File "/home/roadmr/Documents/checkboxes/checkbox/trunk/checkbox/contrib/persist.py", line 136, in save
    os.rename(filepath, filepath+".old")
  File "./plugins/lock_prompt.py", line 69, in handler
    self._manager.reactor.stop_all()
  File "/home/roadmr/Documents/checkboxes/checkbox/trunk/checkbox/reactor.py", line 118, in stop_all
    raise StopAllException
StopAllException

Related branches

lp:~tiagosh/checkbox/nofork
Daniel Manrique (community): Approve
Diff: 96 lines (+15/-34)
2 files modified
checkbox/user_interface.py (+14/-34)
debian/changelog (+1/-0)
lp:~roadmr/ubuntu/precise/checkbox/0.13.5
Ubuntu Sponsors: Pending requested
Diff: 101688 lines (+21514/-14618)
123 files modified
checkbox/application.py (+1/-1)
checkbox/user_interface.py (+15/-34)
checkbox_qt/qt_interface.py (+25/-12)
debian/changelog (+57/-0)
debian/po/cs.po (+2/-2)
debian/po/de.po (+2/-2)
debian/po/en_AU.po (+2/-2)
debian/po/en_GB.po (+2/-2)
debian/po/es.po (+2/-2)
debian/po/fr.po (+2/-2)
debian/po/he.po (+2/-2)
debian/po/hu.po (+2/-2)
debian/po/it.po (+2/-2)
debian/po/ja.po (+2/-2)
debian/po/nl.po (+2/-2)
debian/po/oc.po (+2/-2)
debian/po/pl.po (+2/-2)
debian/po/pt_BR.po (+2/-2)
debian/po/ro.po (+2/-2)
debian/po/ru.po (+2/-2)
debian/po/uk.po (+2/-2)
debian/po/zh_TW.po (+2/-2)
jobs/keys.txt.in (+1/-1)
jobs/mediacard.txt.in (+16/-16)
jobs/optical.txt.in (+10/-9)
jobs/suspend.txt.in (+1/-1)
plugins/apport_prompt.py (+3/-4)
plugins/user_interface.py (+23/-0)
po/ace.po (+146/-144)
po/af.po (+146/-144)
po/am.po (+148/-146)
po/ar.po (+146/-144)
po/ast.po (+164/-157)
po/az.po (+95/-93)
po/be.po (+146/-144)
po/bg.po (+146/-144)
po/bn.po (+146/-144)
po/bo.po (+146/-144)
po/br.po (+146/-144)
po/bs.po (+146/-144)
po/ca.po (+230/-180)
po/ca@valencia.po (+146/-144)
po/ckb.po (+120/-93)
po/cs.po (+1030/-200)
po/cy.po (+148/-146)
po/da.po (+346/-148)
po/de.po (+742/-667)
po/dv.po (+148/-146)
po/el.po (+68/-66)
po/en_AU.po (+733/-685)
po/en_CA.po (+146/-144)
po/en_GB.po (+719/-717)
po/eo.po (+146/-144)
po/es.po (+505/-189)
po/et.po (+146/-144)
po/eu.po (+146/-144)
po/fa.po (+148/-146)
po/fi.po (+196/-145)
po/fr.po (+254/-144)
po/ga.po (+148/-146)
po/gd.po (+146/-144)
po/gl.po (+147/-145)
po/he.po (+146/-144)
po/hi.po (+146/-144)
po/hr.po (+146/-144)
po/hu.po (+211/-145)
po/hy.po (+148/-146)
po/id.po (+148/-146)
po/is.po (+146/-144)
po/it.po (+1158/-205)
po/ja.po (+146/-144)
po/jbo.po (+148/-146)
po/ka.po (+148/-146)
po/kk.po (+146/-144)
po/km.po (+148/-146)
po/kn.po (+148/-146)
po/ko.po (+195/-165)
po/ku.po (+148/-146)
po/ky.po (+148/-146)
po/lt.po (+146/-144)
po/lv.po (+146/-144)
po/mk.po (+148/-146)
po/ml.po (+148/-146)
po/mr.po (+148/-146)
po/ms.po (+95/-93)
po/my.po (+146/-144)
po/nb.po (+146/-144)
po/nds.po (+148/-146)
po/ne.po (+146/-144)
po/nl.po (+44/-44)
po/nn.po (+146/-144)
po/oc.po (+147/-145)
po/pl.po (+146/-144)
po/ps.po (+148/-146)
po/pt.po (+146/-144)
po/pt_BR.po (+199/-149)
po/ro.po (+1349/-239)
po/ru.po (+44/-44)
po/sd.po (+44/-44)
po/shn.po (+148/-146)
po/si.po (+148/-146)
po/sk.po (+68/-66)
po/sl.po (+799/-173)
po/sq.po (+146/-144)
po/sr.po (+148/-146)
po/sv.po (+146/-144)
po/ta.po (+152/-145)
po/te.po (+152/-150)
po/th.po (+146/-144)
po/tr.po (+1102/-199)
po/ug.po (+1309/-191)
po/uk.po (+45/-45)
po/ur.po (+148/-146)
po/uz.po (+148/-146)
po/vi.po (+146/-144)
po/zh_CN.po (+147/-145)
po/zh_HK.po (+146/-144)
po/zh_TW.po (+146/-144)
qt/frontend/qtfront.cpp (+75/-35)
qt/frontend/qtfront.h (+12/-7)
qt/frontend/qtfront.ui (+102/-35)
scripts/removable_storage_test (+3/-2)
scripts/removable_storage_watcher (+11/-11)
lp:ubuntu/precise/checkbox
Daniel Manrique (roadmr)
Changed in checkbox:
milestone: none → 0.13.5
Tiago Salem Herrmann (tiagosh)
Changed in checkbox:
assignee: nobody → Tiago Salem Herrmann (tiagosh)
status: New → In Progress
Daniel Manrique (roadmr)
Changed in checkbox:
status: In Progress → Fix Committed
Ara Pulido (ara)
Changed in checkbox:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.