Checkbox

duplicate tests in optical.txt

Bug #868643 reported by Jeff Lane 
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Checkbox
Fix Released
Low
Jeff Lane 

Bug Description

optical.txt contains two essentially identical tests. optical/dvd_movie_playback and optical/dvd_playback. The second one should just be removed as the dvd_movie_playback test is more detailed and they both achieve the same thing.

Related branches

lp:~bladernr/checkbox/868643
Marc Tardif (community): Approve
Diff: 50 lines (+6/-15) (has conflicts)
2 files modified
debian/changelog (+6/-0)
jobs/optical.txt.in (+0/-15)
lp:checkbox
lp:~sylvain-pineau/checkbox/0.15
Daniel Manrique (community): Approve
Diff: 1477 lines (+506/-529) (has conflicts)
1 file modified
debian/changelog (+506/-529)
lp:ubuntu/raring-proposed/checkbox
Revision history for this message
Daniel Manrique (roadmr) wrote :

Does the default Ubuntu installation include the stuff needed to play a DVD out of the box?

Also, one thing I like about the second (optical/dvd_playback) is that it auto-launches totem for you, thus sparing the user of having to look for and launch the movie player manually (which is outside of the scope of this test I think).

In any case, I'm marking this triaged and importance: low, as the test is not included in the default whitelist and thus is only an issue for custom whitelists that may be using both tests.

Changed in checkbox:
status: New → Triaged
importance: Undecided → Low
tags: added: checkbox-testsuite
Revision history for this message
Brendan Donegan (brendan-donegan) wrote :

It doesn't contain the decryption software required to play DVDs out of the box. There is setup you need to do, by installing libdvdread4 and running a script.

Revision history for this message
Daniel Manrique (roadmr) wrote :

OK, see bug https://bugs.launchpad.net/checkbox/+bug/772731 for a more detailed discussion. So I'm setting this to Triaged. The work to be done is simply to remove the optical/dvd_playback test, since apparently it's not being used by us or other teams. Of course, if you think the test should absolutely not be removed, please say so :)

Jeff Lane  (bladernr)
Changed in checkbox:
assignee: nobody → Jeff Lane (bladernr)
status: Triaged → In Progress
Jeff Lane  (bladernr)
Changed in checkbox:
status: In Progress → Fix Committed
Brendan Donegan (brendan-donegan)
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.