media card storage tests only look at sdio devices
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Checkbox |
Fix Released
|
High
|
Daniel Manrique |
Bug Description
The fix for bug 953160 in rev 1322 of checkbox updated job definitions so that the insertion and removal tests look at the sdio, usb and scsi buses for card insertions (as some systems present cards as those kinds of devices).
Unfortunately this change was not done for the actual storage transfer tests (the ones using removable_
This can be ascertained in jobs/mediacards
plugin: manual
name: mediacard/
command: removable_
the matching storage test:
plugin: shell
name: mediacard/
depends: mediacard/
command: removable_
notice the second one only looks at sdio, while it should probably consider usb and scsi as well.
Adding them to the test is the "easy" way to fix this. Not perfect, since the removable_
Related branches
- Jeff Lane : Approve
-
Diff: 105 lines (+15/-9)3 files modifieddebian/changelog (+6/-0)
jobs/local.txt.in (+1/-1)
jobs/mediacard.txt.in (+8/-8)
- Mathieu Trudel-Lapierre: Approve
-
Diff: 171775 lines (+80184/-40843)101 files modifiedcheckbox_qt/qt_interface.py (+18/-2)
debian/changelog (+38/-0)
debian/po/es.po (+9/-4)
jobs/local.txt.in (+8/-0)
jobs/mediacard.txt.in (+8/-8)
jobs/networking.txt.in (+2/-2)
jobs/optical.txt.in (+0/-3)
po/ace.po (+788/-406)
po/af.po (+788/-406)
po/am.po (+788/-406)
po/ar.po (+799/-410)
po/ast.po (+1158/-665)
po/az.po (+788/-406)
po/be.po (+840/-415)
po/bg.po (+827/-421)
po/bn.po (+889/-411)
po/bo.po (+820/-420)
po/br.po (+825/-419)
po/bs.po (+840/-415)
po/ca.po (+846/-422)
po/ca@valencia.po (+821/-415)
po/ckb.po (+788/-406)
po/cs.po (+1166/-440)
po/cy.po (+788/-406)
po/da.po (+852/-427)
po/de.po (+1260/-686)
po/dv.po (+788/-406)
po/el.po (+1215/-670)
po/en_AU.po (+1390/-877)
po/en_CA.po (+824/-418)
po/en_GB.po (+1377/-871)
po/eo.po (+819/-416)
po/es.po (+1261/-618)
po/et.po (+788/-406)
po/eu.po (+796/-407)
po/fa.po (+788/-406)
po/fi.po (+905/-681)
po/fr.po (+974/-501)
po/ga.po (+788/-406)
po/gd.po (+792/-410)
po/gl.po (+842/-420)
po/he.po (+868/-424)
po/hi.po (+818/-415)
po/hr.po (+823/-417)
po/hu.po (+848/-418)
po/hy.po (+788/-406)
po/id.po (+819/-419)
po/is.po (+792/-409)
po/it.po (+1197/-423)
po/ja.po (+863/-413)
po/jbo.po (+788/-406)
po/ka.po (+788/-406)
po/kk.po (+822/-416)
po/km.po (+788/-406)
po/kn.po (+788/-406)
po/ko.po (+843/-438)
po/ku.po (+788/-406)
po/ky.po (+788/-406)
po/lt.po (+823/-420)
po/lv.po (+830/-413)
po/mk.po (+788/-409)
po/ml.po (+788/-406)
po/mr.po (+788/-406)
po/ms.po (+1147/-649)
po/my.po (+788/-406)
po/nb.po (+831/-420)
po/nds.po (+788/-406)
po/ne.po (+792/-406)
po/nl.po (+1726/-629)
po/nn.po (+788/-406)
po/oc.po (+826/-414)
po/pl.po (+827/-444)
po/ps.po (+788/-406)
po/pt.po (+825/-419)
po/pt_BR.po (+1180/-620)
po/ro.po (+1199/-463)
po/ru.po (+1018/-682)
po/sd.po (+789/-407)
po/shn.po (+788/-406)
po/si.po (+788/-406)
po/sk.po (+820/-445)
po/sl.po (+1174/-485)
po/sq.po (+841/-416)
po/sr.po (+826/-411)
po/sv.po (+840/-418)
po/ta.po (+788/-406)
po/te.po (+788/-406)
po/th.po (+821/-418)
po/tr.po (+1220/-443)
po/ug.po (+1123/-652)
po/uk.po (+844/-443)
po/ur.po (+788/-406)
po/uz.po (+788/-406)
po/vi.po (+821/-415)
po/zh_CN.po (+820/-414)
po/zh_HK.po (+816/-413)
po/zh_TW.po (+834/-425)
qt/checkbox-qt.ui (+2/-2)
qt/frontend/qtfront.cpp (+8/-3)
qt/frontend/qtfront.h (+4/-2)
qt/frontend/qtfront.ui (+3/-3)
Changed in checkbox: | |
milestone: | none → 0.13.7 |
status: | New → In Progress |
assignee: | nobody → Daniel Manrique (roadmr) |
importance: | Undecided → High |
Changed in checkbox: | |
status: | In Progress → Fix Committed |
Changed in checkbox: | |
status: | Fix Committed → Fix Released |