Unity

Merge lp://staging/~bregma/unity/lp-1435830 into lp://staging/unity

  1. lp-1435830
  2. Merge into trunk
Proposed by Stephen M. Webb
Status: Merged
Approved by: Christopher Townsend
Approved revision: no longer in the source branch.
Merged at revision: 3946
Proposed branch: lp://staging/~bregma/unity/lp-1435830
Merge into: lp://staging/unity
Diff against target: 19 lines (+8/-3)
1 file modified
tests/autopilot/unity/tests/launcher/test_keynav.py (+8/-3)
To merge this branch: bzr merge lp://staging/~bregma/unity/lp-1435830
Reviewer Review Type Date Requested Status
Christopher Townsend (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+254014@code.staging.launchpad.net

Commit message

fixed the Panel title string expected by Launcher keynav AP tests

Description of the change

Fixes the Panel title string expected by Launcher keynav AP tests.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:3930
http://jenkins.qa.ubuntu.com/job/unity-ci/1172/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-vivid-amd64-ci/60
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-vivid-armhf-ci/59
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-vivid-i386-ci/59

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-ci/1172/rebuild

review: Approve (continuous-integration)
Revision history for this message
Christopher Townsend (townsend) wrote :

As discussed on IRC, the reason this test now fails is due to LIM being enabled. A design decision was made to not display the tooltip string in the Panel when LIM is enabled.

Although I think this design makes no sense since what is displayed in the Panel in Launcher keynav mode should have nothing to do with LIM being enabled or not, but I disgress. At any rate, this needs fixing.

review: Needs Fixing
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:3931
http://jenkins.qa.ubuntu.com/job/unity-ci/1174/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity-vivid-amd64-ci/62/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-vivid-armhf-ci/61
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-vivid-i386-ci/61

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-ci/1174/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:3931
http://jenkins.qa.ubuntu.com/job/unity-ci/1177/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-vivid-amd64-ci/65
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-vivid-armhf-ci/64
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-vivid-i386-ci/64

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-ci/1177/rebuild

review: Approve (continuous-integration)
Revision history for this message
Christopher Townsend (townsend) wrote :

Ok. I still think the design is silly and I'd like to know where this is defined (besides in someone's head).

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.