Merge lp://staging/~xavi-garcia-mena/indicator-sound/re-add-integration-tests-notifications into lp://staging/indicator-sound/15.10

Proposed by Xavi Garcia
Status: Merged
Approved by: Charles Kerr
Approved revision: 539
Merged at revision: 540
Proposed branch: lp://staging/~xavi-garcia-mena/indicator-sound/re-add-integration-tests-notifications
Merge into: lp://staging/indicator-sound/15.10
Diff against target: 103 lines (+14/-19)
2 files modified
tests/integration/CMakeLists.txt (+2/-2)
tests/integration/test-indicator.cpp (+12/-17)
To merge this branch: bzr merge lp://staging/~xavi-garcia-mena/indicator-sound/re-add-integration-tests-notifications
Reviewer Review Type Date Requested Status
Charles Kerr (community) Approve
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+293508@code.staging.launchpad.net

Commit message

Re-adding some integration tests for notifications after the fix for Qt has landed.

Description of the change

Re-adding some integration tests for notifications after the fix for Qt has landed.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
539. By Xavi Garcia

Re-add notifications integration tests

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Charles Kerr (charlesk) wrote :

LGTM.

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.

Subscribers

People subscribed via source and target branches