Merge lp:~canonical-platform-qa/address-book-app/fix_app_proxy into lp:address-book-app

Proposed by Leo Arias on 2015-02-25
Status: Merged
Approved by: Renato Araujo Oliveira Filho on 2015-02-25
Approved revision: 387
Merged at revision: 388
Proposed branch: lp:~canonical-platform-qa/address-book-app/fix_app_proxy
Merge into: lp:address-book-app
Diff against target: 14 lines (+2/-2)
1 file modified
tests/autopilot/address_book_app/__init__.py (+2/-2)
To merge this branch: bzr merge lp:~canonical-platform-qa/address-book-app/fix_app_proxy
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Approve on 2015-02-25
Renato Araujo Oliveira Filho (community) 2015-02-25 Approve on 2015-02-25
Review via email: mp+250969@code.launchpad.net

Commit Message

Fixed the application autopilot proxy.

To post a comment you must log in.

looks good

review: Approve
Leo Arias (elopio) wrote :

The failed tests are settle after and settle before.
All the tests affected by this change passed. I'll kick it again.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/autopilot/address_book_app/__init__.py'
2--- tests/autopilot/address_book_app/__init__.py 2015-02-19 07:02:09 +0000
3+++ tests/autopilot/address_book_app/__init__.py 2015-02-25 17:44:15 +0000
4@@ -42,8 +42,8 @@
5 @classmethod
6 def validate_dbus_object(cls, path, state):
7 name = introspection.get_classname_from_path(path)
8- return (name == b'webbrowser-app' and
9- state['applicationName'][1] == 'webbrowser-app')
10+ return (name == b'AddressBookApp' and
11+ state['applicationName'][1] == 'AddressBookApp')
12
13 @property
14 def main_window(self):

Subscribers

People subscribed via source and target branches