Merge lp:~gary-lasker/unity/software-center-launcher-integration-tests-for-5.0 into lp:unity/5.0
| Status: | Merged |
|---|---|
| Approved by: | Łukasz Zemczak on 2012-07-18 |
| Approved revision: | 2386 |
| Merged at revision: | 2391 |
| Proposed branch: | lp:~gary-lasker/unity/software-center-launcher-integration-tests-for-5.0 |
| Merge into: | lp:unity/5.0 |
| Prerequisite: | lp:~bilalakhtar/unity/5.0series-sru-software-center-integration-fixes |
| Diff against target: |
124 lines (+104/-16) 1 file modified
manual-tests/SoftwareCenter.txt (+104/-16) |
| To merge this branch: | bzr merge lp:~gary-lasker/unity/software-center-launcher-integration-tests-for-5.0 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Marco Trevisan (Treviño) | Approve on 2012-07-17 | ||
| Brandon Schaefer (community) | Approve on 2012-07-16 | ||
| Christopher Lee (community) | 2012-07-10 | Approve on 2012-07-16 | |
|
Review via email:
|
|||
Commit Message
manual-
Description of the Change
This branch provides a set of manual tests for the Ubuntu Software Center/Unity launcher integration feature. It actually updates the manual test that is currently found in the branch and adds additional needed detailed test cases.
There are *no* code changes here. All changes are limited to the file manual-
Please note that this kind of manual testing is very useful for this feature as it spans two applications and so all pf the details of the interactions can be tested directly by these. Of course, these manual tests are *not* to be considered in lieu of complete automatic unit tests! Rather, they are simply intended to augment the automatic unit tests that are already in place for this integration feature on the Unity and Software Center sides. The advantage of having manual tests for this feature is that a person can more easily observe in detail the overall interaction between the two applications.
The added tests will thoroughly verify all of the pieces of the integration functionality for the various cases (install/
Finally, I've set this branch as depending on Bilal's ~bilalakhtar/
Many thanks for your review!
| Gary Lasker (gary-lasker) wrote : | # |
| Christopher Lee (veebers) wrote : | # |
Hi Gary,
I stepped through the instructions but had to do something different to get the 'More Info' option. I think this needs to be made a bit more explicit so it doesn't confuse anyone.
The default view that I get when selecting the 'Games' category is a list of genres and the top rated games (clicking on any of these games takes me to its details, no 'More Info' available).
I had to select a genre to get a list of games which allowed me to select it and get the 'More Info' option.
If you could clarify that please.
| Gary Lasker (gary-lasker) wrote : | # |
Hello Christopher! Thanks very much for reviewing this so carefully, I really appreciate it! Indeed, the section for selecting a game is unclear in exactly the way you described. I've updated the branch and hopefully now it is clarified.
Thank you again and please let me know if there are any other changes needed!
Best regards,
Gary
| Unity Merger (unity-merger) wrote : | # |
The prerequisite lp:~bilalakhtar/unity/5.0series-sru-software-center-integration-fixes has not yet been merged into lp:unity/5.0.
| Brandon Schaefer (brandontschaefer) wrote : | # |
Hmm the last manual test didn't work for me. When you remove a program that icon still stays in the the launcher.
| Gary Lasker (gary-lasker) wrote : | # |
Hi Brandon, and thanks very much for testing this. Actually, you need to be using the corresponding updated version of Software Center as well for the integration to fully work. This can be found in my branch here:
lp:~gary-lasker/software-center/unity-launcher-integration-fixes
I have a test build of this currently building in my PPA so that it's easier to get and use, but feel free to use the branch directly. To run it, simply get a copy of the branch above and run as follows from the top level of the branch:
$ ./software-center
I'll add a note here when my PPA build is complete.
Thanks again!
Gary
| Brandon Schaefer (brandontschaefer) wrote : | # |
Opps! Sorry, missed that in the info above! (Retesting) annd it works :).
| Marco Trevisan (Treviño) (3v1n0) wrote : | # |
+ The game begins to install and the progress bar moves in Software Center.
37 + At the very *end* of the installation process (when the progress bar has completed), the icon for the game "flies" into the Unity Launcher.
Mhm... shouldn't the fly happen when the installation process begins?
| Bilal Akhtar (bilalakhtar) wrote : | # |
On Mon, Jul 16, 2012 at 1:21 PM, Marco Trevisan (Treviño)
<mail@3v1n0.net> wrote:
>
> Mhm... shouldn't the fly happen when the installation process begins?
Not any more. Gary's software-center branch (linked above) makes the change.
> --
> https:/
> You are subscribed to branch lp:~bilalakhtar/unity/5.0series-sru-software-center-integration-fixes.
| Gary Lasker (gary-lasker) wrote : | # |
Hi Marco and Bilal, yes, in order to fix a few important bugs we had to move the icon flying event to happen at the very end of the application install rather than at the beginning. For Quantal (still TODO), we will want to revert this behavior -- I believe to do that we just need some additional support for reacting aptdaemon events on the Unity side (see bug 1011681, and please correct me if there is more than that, Bilal).
In the meantime, this delay has been deemed a worthy compromise by the user experience folks based on the number of high-target bugs we are fixing with this.
Thanks all for your help!!
Gary
| Gary Lasker (gary-lasker) wrote : | # |
And thanks to you also, Brandon!! I appreciate your retesting!
Just fyi, I prepared a test version of Ubuntu Software Center that includes this functionality and it is now ready in my PPA at:
https:/
The version of interest is the one for Precise, version 5.2.5~unitylaun
| Unity Merger (unity-merger) wrote : | # |
No proposals found for merge of lp:~bilalakhtar/unity/5.0series-sru-software-center-integration-fixes into lp:unity/5.0.


P.S. It would be nice, if you decide to merge these tests, to also get them into trunk. Will a second MP be needed for that? Or do you automatically merge 5.0 changes to trunk?
Please let me know and I will write a second proposal if needed.
Thanks again!