Merge lp:~pharmasolin/ubuntu-manual-tests/VLC_updated into lp:ubuntu-manual-tests
| Status: | Merged | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Merged at revision: | 345 | ||||||||
| Proposed branch: | lp:~pharmasolin/ubuntu-manual-tests/VLC_updated | ||||||||
| Merge into: | lp:ubuntu-manual-tests | ||||||||
| Diff against target: |
136 lines (+39/-14) 2 files modified
testcases/packages/1426_Orca (+5/-14) testcases/packages/1647_vlc (+34/-0) |
||||||||
| To merge this branch: | bzr merge lp:~pharmasolin/ubuntu-manual-tests/VLC_updated | ||||||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Nicholas Skaggs | Approve on 2015-10-29 | ||
| Kev Bowring | 2015-10-26 | Needs Fixing on 2015-10-26 | |
|
Review via email:
|
|||
Description of the Change
https:/
- 344. By Lysenko Denis on 2015-10-26
-
Updated testcase for Orca (bug #1161570)
- 345. By Lysenko Denis on 2015-10-26
-
Fixing for VLC testcase
| Lysenko Denis (pharmasolin) wrote : | # |
Added fixes to VLC testcase, also Orca testcase updated.
| Kev Bowring (flocculant) wrote : | # |
re Orca.
I'm not too happy atm with it being Ubuntu-centric.
Testcases need to be written so they can be run from any of the official flavours as well as main Ubuntu.
No other DE has hud/unity/global menu.
Super in Xubuntu does nothing for example.
| Lysenko Denis (pharmasolin) wrote : | # |
In bug #1161570 it is said "The orca testcase needs to be verified and updated to reflect the accessibility changes that have happened in unity." So how to deal with it? I think we need testcases for different DE in this case, how to test that Orca can "understand" and can "tell" me what i'm focusing with Alt + F1 in Unity DE?
I think that solution can be mention in the test section where Unity is needed as for VLC
<em>Unity DE needed for this test</em>
| Kev Bowring (flocculant) wrote : | # |
changed the bug to be agnostic.
No - we don't need 5 different testcases to suit different DE's.
"how to test that Orca can "understand" and can "tell" me what i'm focusing with Alt + F1 in Unity DE?" run them from a VM maybe?
to be honest you'd in future be better off doing seperate MPs for different testcases - can't approve this until the orca one gets fixed, which means the work you did on vlc is now hanging about :)
| Lysenko Denis (pharmasolin) wrote : | # |
It is not a problem, so solution should be remove Unity DE specific test in this testcase?
| Kev Bowring (flocculant) wrote : | # |
> It is not a problem, so solution should be remove Unity DE specific test in
> this testcase?
Unless a test is very specific then any testcase should be able to run properly on any flavour - so no DE specific.
So rather than just remove anything which is specific - reword it so it works everywhere.
| Nicholas Skaggs (nskaggs) wrote : | # |
Lysenko, first of all, thanks for contributing! As flocculant mentioned, it's best to make each testcase a seperate MP so it can be reviewed independently. That avoids the problem of us wanting to merge your vlc changes, but not the orca ones just yet.
Now, on the Orca changes, thanks for fixing these up. We're concerned about the testcase being out of date again or invalid for other flavors. It would be great if you could try it out on a xubuntu or lubuntu install and make sure these directions still make sense.
All that said, I'm ok with landing this for now. Thanks again for your contribution. I hope to see more MP's in the future.


some inline comments
Also - can you add this to the very beginning of the testcase
<em>Testcase requires use of video and/or audio files</em>
thanks :)