Merge lp:~noskcaj/ubuntu-manual-tests/parole into lp:ubuntu-manual-tests

Proposed by Jackson Doak on 2013-06-29
Status: Merged
Merged at revision: 163
Proposed branch: lp:~noskcaj/ubuntu-manual-tests/parole
Merge into: lp:ubuntu-manual-tests
Diff against target: 22 lines (+17/-0)
1 file modified
testcases/packages/Parole (+17/-0)
To merge this branch: bzr merge lp:~noskcaj/ubuntu-manual-tests/parole
Reviewer Review Type Date Requested Status
Nicholas Skaggs (community) Approve on 2013-07-10
Elfy 2013-06-29 Pending
Review via email: mp+172212@code.launchpad.net

This proposal supersedes a proposal from 2013-06-28.

Description of the change

Added the basic version of the parole test.

To post a comment you must log in.
Elfy (elfy) wrote : Posted in a previous version of this proposal

Can we add more detail please.

There are no tests for menu items.

Not sure what you mean by

    <dt>Localization strings for embedded subtitles</dt>

Thanks

review: Needs Fixing
Jackson Doak (noskcaj) wrote : Posted in a previous version of this proposal

I was just copying stuff from the totem test, i'll fix it soon.

On Sun, Jun 30, 2013 at 12:59 AM, Elfy <email address hidden> wrote:

> Review: Needs Fixing
>
> Can we add more detail please.
>
> There are no tests for menu items.
>
> Not sure what you mean by
>
> <dt>Localization strings for embedded subtitles</dt>
>
> Thanks
>
>
>
> --
>
> https://code.launchpad.net/~noskcaj/ubuntu-manual-tests/parole/+merge/172154
> You are the owner of lp:~noskcaj/ubuntu-manual-tests/parole.
>

Nicholas Skaggs (nskaggs) wrote :

I'm ok with merging this.. can we deprecate the old one on the isotracker?

review: Approve
Jackson Doak (noskcaj) wrote :

That was the goal.

On Thu, Jul 11, 2013 at 6:18 AM, Nicholas Skaggs
<email address hidden> wrote:
> Review: Approve
>
> I'm ok with merging this.. can we deprecate the old one on the isotracker?
> --
> https://code.launchpad.net/~noskcaj/ubuntu-manual-tests/parole/+merge/172212
> You are the owner of lp:~noskcaj/ubuntu-manual-tests/parole.

Nicholas Skaggs (nskaggs) wrote :

Merging -- you can remove the old one yourself from the testsuite :-)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== added file 'testcases/packages/Parole'
2--- testcases/packages/Parole 1970-01-01 00:00:00 +0000
3+++ testcases/packages/Parole 2013-06-29 20:39:24 +0000
4@@ -0,0 +1,17 @@
5+Test-case name: parole/parole-001
6+This tests that parole can open many different files.
7+<dl>
8+ <dt>Launch Parole</dt>
9+ <dd>Parole opens</dd>
10+ <dt>Download some different video and audio formats from https://samples.mplayerhq.hu/</dt>
11+ <dt>Open then Play each file with parole</dt>
12+ <dd>All files play (including those that needed new codecs)</dd>
13+ <dd>If a required codec is missing, an installation prompt appears and installs the missing codec. Afterwards, media playback begins</dd>
14+ <dt>Insert then Play some CDs and DVDs</dt>
15+ <dd>CDs and DVDs play (note: to play encrypted DVDs, you will need to have libdvdcss2 installed)</dd>
16+ <dd>Chapters and tracks for CDs/DVDs are shown in the playlist</dd>
17+ <dt>Use (external) subtitles for a video file</dt>
18+ <dd>Subtitles work and show up correctly</dd>
19+</dl>
20+
21+<strong>If all actions produce the expected results listed, please <a href="results#add_result">submit</a> a 'passed' result. If an action fails, or produces an unexpected result, please <a href="results#add_result">submit</a> a 'failed' result and <a href="../../buginstructions">file a bug</a>. Please be sure to include the bug number when you <a href="results#add_result">submit</a> your result</strong>
22\ No newline at end of file

Subscribers

People subscribed via source and target branches