Merge lp:~om26er/address-book-app/abook_navigation_favorites into lp:address-book-app

[Omer Akram]

autopilot: add tests for favoriting contacts, using the new dummy backend service

Note: I only fixed the code to use the dummy backend service, no other changes are from me.

Depends on lp:~renatofilho/address-book-service/dummy-mode to be merged.

[lp:~om26er/address-book-app/abook_navigation_favorites]

163. By Omer Akram on 2014-03-18

remove syncevolution dep

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:163
http://jenkins.qa.ubuntu.com/job/address-book-app-ci/455/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-amd64-ci/165
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/165
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/165/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-i386-ci/165
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/4001
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/3586/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/3493
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4042
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4042/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3588
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3588/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5918/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/4891

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/address-book-app-ci/455/rebuild

review: Needs Fixing (continuous-integration)

[lp:~om26er/address-book-app/abook_navigation_favorites]

164. By Omer Akram on 2014-03-18

remove unused test code, fix things

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:164
http://jenkins.qa.ubuntu.com/job/address-book-app-ci/456/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-amd64-ci/166
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/166
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/166/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-i386-ci/166
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/4005
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/3590/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/3495
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4046
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4046/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3592
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3592/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5922/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/4895

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/address-book-app-ci/456/rebuild

review: Needs Fixing (continuous-integration)

[lp:~om26er/address-book-app/abook_navigation_favorites]

165. By Omer Akram on 2014-03-18

make dummy backend code reusable by other test cases

166. By Omer Akram on 2014-03-18

add docstrings for new tests

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:165
http://jenkins.qa.ubuntu.com/job/address-book-app-ci/458/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-amd64-ci/168
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/168
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/168/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-i386-ci/168
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/4016
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/3601/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/3507
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4057
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4057/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3603
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3603/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5932/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/4907

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/address-book-app-ci/458/rebuild

review: Needs Fixing (continuous-integration)

[PS Jenkins bot (ps-jenkins)]

FAILED: Continuous integration, rev:166
http://jenkins.qa.ubuntu.com/job/address-book-app-ci/460/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-amd64-ci/170
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/170
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-armhf-ci/170/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-trusty-i386-ci/170
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty/4021
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-trusty-touch/3606/console
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-trusty/3512
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4062
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-amd64/4062/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3608
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-trusty-armhf/3608/artifact/work/output/*zip*/output.zip
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-runner-mako/5937/console
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/4912

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/address-book-app-ci/460/rebuild

review: Needs Fixing (continuous-integration)

[lp:~om26er/address-book-app/abook_navigation_favorites]

167. By Omer Akram on 2014-03-24

make starting of dummy backend a reusable helper for other apps to consume

168. By Omer Akram on 2014-03-26

adapt autopilot tests to UI changes in trunk

169. By Omer Akram on 2014-03-26

merge trunk

Unmerged revisions

169. By Omer Akram on 2014-03-26

merge trunk

168. By Omer Akram on 2014-03-26

adapt autopilot tests to UI changes in trunk

167. By Omer Akram on 2014-03-24

make starting of dummy backend a reusable helper for other apps to consume

166. By Omer Akram on 2014-03-18

add docstrings for new tests

165. By Omer Akram on 2014-03-18

make dummy backend code reusable by other test cases

164. By Omer Akram on 2014-03-18

remove unused test code, fix things

163. By Omer Akram on 2014-03-18

remove syncevolution dep

162. By Omer Akram on 2014-03-17

fix pep8 and pyflakes warnings for test_favorite, did not touch other modules

161. By Omer Akram on 2014-03-17

remove unused test code

160. By Omer Akram on 2014-03-17

don't use memory backend for favorite tests, use the new dummy backend from the service

=== modified file 'src/imports/ContactView/ContactDetailFavoriteView.qml'
--- src/imports/ContactView/ContactDetailFavoriteView.qml	2014-02-25 21:52:58 +0000
+++ src/imports/ContactView/ContactDetailFavoriteView.qml	2014-03-26 22:14:30 +0000
@@ -28,6 +28,7 @@
 
     Icon {
         id: icon
+        objectName: 'favoriteIcon'
 
         anchors.fill: parent
         name: root.detail && root.detail.favorite ? "favorite-selected" : "favorite-unselected"

=== modified file 'src/imports/Ubuntu/Contacts/ContactListView.qml'
--- src/imports/Ubuntu/Contacts/ContactListView.qml	2014-02-25 20:20:21 +0000
+++ src/imports/Ubuntu/Contacts/ContactListView.qml	2014-03-26 22:14:30 +0000
@@ -57,6 +57,7 @@
             anchors.fill: parent
             Label {
                 id: lblAll
+                objectName: "labelAll"
 
                 anchors {
                     top: parent.top
@@ -84,6 +85,7 @@
 
             Label {
                 id: lblFavourites
+                objectName: "labelFavorite"
 
                 anchors {
                     top: parent.top

=== modified file 'tests/autopilot/address_book_app/emulators/__init__.py'
--- tests/autopilot/address_book_app/emulators/__init__.py	2013-07-09 18:42:30 +0000
+++ tests/autopilot/address_book_app/emulators/__init__.py	2014-03-26 22:14:30 +0000
@@ -4,3 +4,15 @@
 # This program is free software: you can redistribute it and/or modify it
 # under the terms of the GNU General Public License version 3, as published
 # by the Free Software Foundation.
+
+
+def get_nearest_parent_matching_type(child, desired_type):
+    """Climb down the widget tree, return first instance of desired_type"""
+    while type(child).__name__ != desired_type:
+        parent = child.get_parent()
+        if type(child).__name__ == 'AddressBookApp':
+            # we reached the root and found nothing
+            return None
+        else:
+            child = parent
+    return child

=== modified file 'tests/autopilot/address_book_app/emulators/contact_list_page.py'
--- tests/autopilot/address_book_app/emulators/contact_list_page.py	2014-02-13 16:44:12 +0000
+++ tests/autopilot/address_book_app/emulators/contact_list_page.py	2014-03-26 22:14:30 +0000
@@ -1,20 +1,24 @@
 # -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
-
-""" ContactListPage emulator for Addressbook App tests """
-
 # Copyright 2014 Canonical
 #
 # This program is free software: you can redistribute it and/or modify it
 # under the terms of the GNU General Public License version 3, as published
 # by the Free Software Foundation.
 
+""" ContactListPage emulator for Addressbook App tests """
+
 import logging
+from time import sleep
 
+from autopilot import logging as autopilot_logging
 from autopilot.introspection.dbus import StateNotFoundError
+from autopilot.matchers import Eventually
+from testtools.matchers import GreaterThan
 from ubuntuuitoolkit import emulators as uitk
 
-LOGGER = logging.getLogger(__name__)
-from time import sleep
+from address_book_app.emulators import get_nearest_parent_matching_type
+
+logger = logging.getLogger(__name__)
 
 
 class ContactListPage(uitk.UbuntuUIToolkitEmulatorBase):
@@ -27,9 +31,12 @@
         super(ContactListPage, self).__init__(*args)
 
     def get_contacts(self):
-        """
-        Returns a list of ContactDelegate objects and populate
-        self.selection_marks
+        """Returns a list of ContactDelegate objects.
+
+        Also populate self.selection_marks.
+
+        :return: The list of ContactDelegate objects.
+
         """
         sleep(1)
         self.contacts = self.select_many("ContactDelegate")
@@ -41,10 +48,12 @@
             self.selection_marks.append(mark)
         return self.contacts
 
+    @autopilot_logging.log_action(logger.info)
     def select_contacts_by_index(self, indices):
-        """ Select contacts corresponding to the list of index in indices
-
-        :param indices: List of integers
+        """Select contacts corresponding to the list of index in indices.
+
+        :param indices: List of integer indices of contacts to select.
+
         """
         self.deselect_all()
 
@@ -53,8 +62,9 @@
             self.selected_marks.append(self.selection_marks[idx])
             self.pointing_device.click_object(self.selection_marks[idx])
 
+    @autopilot_logging.log_action(logger.info)
     def deselect_all(self):
-        """Deselect every contacts"""
+        """Deselect every contact."""
         contacts = self.select_many("ContactDelegate")
         self.selected_marks = []
         for contact in contacts:
@@ -63,28 +73,137 @@
                                              objectName="selectionMark")
                 self.pointing_device.click_object(mark)
 
+    @autopilot_logging.log_action(logger.info)
     def click_button(self, objectname):
         """Press a button that matches objectname
 
-        :param objectname: Name of the object
+        :param objectname: The name of the object.
+        :raise StateNotFoundError: When a matching button is not found.
+
         """
-        try:
-            buttons = self.select_many("Button",
-                                       objectName=objectname)
-            for button in buttons:
-                if button.visible:
-                    self.pointing_device.click_object(button)
-        except StateNotFoundError:
-            LOGGER.error(
-                'Button with objectName "{0}" not found.'.format(objectname)
-            )
-            raise
+        button = self.select_single("Button",
+                                     objectName=objectname,
+                                     visible=True)
+        self.pointing_device.click_object(button)
 
+    @autopilot_logging.log_action(logger.info)
     def cancel(self):
-        """Press the cancel button displayed when pick mode is enabled"""
+        """Press the cancel button displayed when pick mode is enabled."""
         self.click_button("DialogButtons.rejectButton")
 
+    @autopilot_logging.log_action(logger.info)
     def delete(self):
-        """Press the delete button displayed when pick mode is enabled"""
+        """Press the delete button displayed when pick mode is enabled."""
         self.click_button("DialogButtons.acceptButton")
         self.get_contacts()
+
+    def get_contact_by_name(self,
+                            contact_name,
+                            parent_delegate_type='ContactDelegate'):
+        """Find a label with text matching contact name.
+
+        :param contact_name: The name of the contact, e.g. 'Fulano de Tal'.
+        :param parent_delegate_type: 'ContactDelegate' or 'FavoriteDelegate'.
+        :return: The label for a matching contact.
+        :raises StateNotFoundError: If the contact_name is not found.
+
+        """
+        try:
+            assert(lambda: len(self.select_many("Label",
+                                                text=contact_name)),
+                   Eventually(GreaterThan(0)))
+            contact_name_labels = self.select_many(
+                "Label",
+                text=contact_name)
+            for contact_name_label in contact_name_labels:
+                # we could have a contact or a favorite
+                if get_nearest_parent_matching_type(
+                        contact_name_label,
+                        parent_delegate_type):
+                    return contact_name_label
+            raise StateNotFoundError('Label')
+        except StateNotFoundError:
+            logger.error("Contact {} not found.".format(contact_name))
+            raise
+
+    def get_favorite_by_name(self, contact_name):
+        """Find a label with text matching contact name under Favorites.
+
+        :param contact_name: Name of the contact, e.g. 'Fulano de Tal'.
+        :return: The label for the metching favorite.
+        :raises StateNotFoundError: If the contact_name is not found.
+
+        """
+        return self.get_contact_by_name(contact_name).visible
+
+    @autopilot_logging.log_action(logger.info)
+    def click_contact_by_name(self,
+                              contact_name,
+                              parent_delegate_type='ContactDelegate'):
+        """Click a contact with label matching the given contact name.
+
+        :param contact_name: Name of a contact, e.g. 'Fulano de Tal'.
+        :param parent_delegate_type: 'ContactDelegate' or 'FavoriteDelegate'.
+        :raises StateNotFoundError: If the contact_name is not found.
+
+        """
+        contact_name_label = self.get_contact_by_name(
+            contact_name, parent_delegate_type)
+        self.pointing_device.click_object(contact_name_label)
+
+    @autopilot_logging.log_action(logger.info)
+    def open_contact_view_by_contact_name(self, contact_name):
+        """Open the contact view page for a contact with the given name.
+
+        :param contact_name: The name of the contact, e.g. 'Abe Lincoln'.
+        :return: The ContactView for the contact.
+        :raises StateNotFoundError: If the name is not found.
+
+        """
+        self.click_contact_by_name(contact_name)
+
+    @autopilot_logging.log_action(logger.info)
+    def click_favorite_by_name(self,
+                               contact_name):
+        """Click a favorite with label matching the given contact name.
+
+        :param contact_name: Name of a contact, e.g. 'Fulano de Tal'
+        :raises StateNotFoundError: If the contact_name is not found.
+
+        """
+        contact_name_label = self.get_favorite_by_name(
+            contact_name)
+        self.pointing_device.click_object(contact_name_label)
+
+    @autopilot_logging.log_action(logger.info)
+    def open_contact_view_by_favorite_name(self, favorite_name):
+        """Open the contact view page for a favorite with the given name.
+
+        :param favorite_name: The name of the favorite, e.g. 'Abe Lincoln'.
+        :return: The ContactView for the contact.
+        :raises StateNotFoundError: If the name is not found.
+
+        """
+        self.click_favorite_by_name(favorite_name)
+
+    @autopilot_logging.log_action(logger.info)
+    def click_phone_number(self, phone_number):
+        """Click a label matching the given phone number.
+
+        :param phone_number: A phone number, e.g. '3321 1232'.
+        :raises StateNotFoundError: If the phone_number is not found.
+
+        """
+        phone_number_label = self.wait_select_single('Label',
+                                                     text=phone_number)
+        self.pointing_device.click_object(phone_number_label)
+
+    def contact_is_favorite(self, name):
+        """Is the given contact name a favorite?
+
+        :param contact_name: The name of a contact e.g. John.
+        :raises StateNotFoundError: If the contact name is not found.
+        :returns: Boolean.
+
+        """
+        self.open_contact_view_by_contact_name(name)

=== added file 'tests/autopilot/address_book_app/emulators/contact_view.py'
--- tests/autopilot/address_book_app/emulators/contact_view.py	1970-01-01 00:00:00 +0000
+++ tests/autopilot/address_book_app/emulators/contact_view.py	2014-03-26 22:14:30 +0000
@@ -0,0 +1,57 @@
+# -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
+# Copyright 2014 Canonical
+#
+# This program is free software: you can redistribute it and/or modify it
+# under the terms of the GNU General Public License version 3, as published
+# by the Free Software Foundation.
+
+""" ContactView emulator for Addressbook App tests """
+
+import logging
+
+from autopilot import logging as autopilot_logging
+from autopilot.introspection.dbus import StateNotFoundError
+from ubuntuuitoolkit import emulators as uitk
+
+logger = logging.getLogger(__name__)
+
+
+class ContactView(uitk.UbuntuUIToolkitEmulatorBase):
+    """ContactView emulator class"""
+
+    def __init__(self, *args):
+        super(ContactView, self).__init__(*args)
+
+    @autopilot_logging.log_action(logger.info)
+    def click_button(self, objectname):
+        """Press a button that matches objectname.
+
+        :param objectname: The name of the object.
+        :raises StateNotFoundError: If the matching object is not found.
+
+        """
+        buttons = self.select_single("Button",
+                                     objectName=objectname,
+                                     visible=True)
+        self.pointing_device.click_object(button)
+
+    @autopilot_logging.log_action(logger.info)
+    def cancel(self):
+        """Press the cancel button displayed when pick mode is enabled."""
+        self.click_button("DialogButtons.rejectButton")
+
+    @autopilot_logging.log_action(logger.info)
+    def delete(self):
+        """Press the delete button displayed when pick mode is enabled."""
+        self.click_button("DialogButtons.acceptButton")
+        self.get_contacts()
+
+    def favorite_icon(self):
+        return self.wait_select_single('Icon', objectName='favoriteIcon')
+
+    @autopilot_logging.log_action(logger.info)
+    def click_favorite_icon(self):
+        """Tap/click on the favorite icon to make a contact favorite."""
+        favorite_icon = self.favorite_icon()
+
+        self.pointing_device.click_object(favorite_icon)

=== modified file 'tests/autopilot/address_book_app/emulators/main_window.py'
--- tests/autopilot/address_book_app/emulators/main_window.py	2014-02-28 16:00:02 +0000
+++ tests/autopilot/address_book_app/emulators/main_window.py	2014-03-26 22:14:30 +0000
@@ -21,7 +21,7 @@
 
     def get_contact_list_page(self):
         return self. wait_select_single("ContactListPage",
-                                        objectName="contactListPage")
+                                       objectName="contactListPage")
 
     def get_contact_edit_page(self):
         return self.wait_select_single(ContactEditor,
@@ -43,7 +43,7 @@
         """
         Returns a ContactListView iobject for the current window
         """
-        return self.wait_select_single( "ContactListView",
+        return self.wait_select_single("ContactListView",
                                        objectName="contactListView")
 
     def get_button(self, name):
@@ -53,7 +53,7 @@
         Arguments:
             name: Name of the button
         """
-        return self.wait_select_single( "Button", objectName=name)
+        return self.wait_select_single("Button", objectName=name)
 
     def cancel(self):
         """
@@ -67,6 +67,30 @@
         """
         self.pointing_device.click_object(self.get_button("accept"))
 
+    def _get_label(self, name):
+        """
+        Returns a Label object matching 'name'
+
+        Arguments:
+            name: objectName of the label
+        """
+        return self.wait_select_single('Label', objectName=name, visible=True)
+
+    def switch_to_favorites(self):
+        """
+        switch to the favorite contacts view
+        """
+        self.pointing_device.click_object(self._get_label('labelFavorite'))
+
+    def switch_to_all_contact(self):
+        """
+        Switch to all contacts view
+        """
+        self.pointing_device.click_object(self._get_label('labelAll'))
+
+    def are_favorites_shown(self):
+        return self.get_contact_list_view().showFavourites
+
     @autopilot_logging.log_action(logger.info)
     def go_to_add_contact(self):
         """

=== added file 'tests/autopilot/address_book_app/helpers.py'
--- tests/autopilot/address_book_app/helpers.py	1970-01-01 00:00:00 +0000
+++ tests/autopilot/address_book_app/helpers.py	2014-03-26 22:14:30 +0000
@@ -0,0 +1,74 @@
+# -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
+# Copyright 2014 Canonical
+#
+# This program is free software: you can redistribute it and/or modify it
+# under the terms of the GNU General Public License version 3, as published
+# by the Free Software Foundation.
+
+import subprocess
+
+from autopilot.testcase import AutopilotTestCase
+from autopilot.matchers import Eventually
+from testtools.matchers import Equals
+
+import dbus
+
+DUMMY_BACKEND_ADDRESS = 'com.canonical.test.pim'
+
+
+class DummyServiceBackendSetup(AutopilotTestCase):
+
+    def _set_contact_service_dummy_backend_environment_variables(self):
+        self.patch_environment(
+            'CANONICAL_PIN_SERVICE_NAME', DUMMY_BACKEND_ADDRESS
+        )
+        self.patch_environment(
+            'FOLKS_BACKEND_PATH', self._get_dummy_backend_so_location()
+        )
+        self.patch_environment('FOLKS_BACKENDS_ALLOWED', 'dummy')
+        self.patch_environment('FOLKS_PRIMARY_STORE', 'dummy')
+
+    def _get_dummy_backend_so_location(self):
+        output = subprocess.check_output(
+            ['locate', '/usr/lib', '--basename', 'dummy.so']
+            ).split()
+
+        if len(output) is 1:
+            return output[0]
+        elif len(output) > 1:
+            for path in output:
+                if 'address-book-service' in path:
+                    return path
+        else:
+            raise RuntimeError(
+                'Dummy backend .so not found is address-book-service-dummy'
+                'installed ?'
+            )
+
+    def _get_address_book_service_binary_location(self):
+        output = subprocess.check_output(
+            ['locate', '/usr/lib', '--basename', 'address-book-service']
+            ).split()
+
+        for path in output:
+            if 'address-book-service/address-book-service' in path:
+                return path
+
+    def _start_service_dummy_backend(self):
+        self.service = subprocess.Popen(
+            [self._get_address_book_service_binary_location()]
+        )
+        self.addCleanup(self.service.kill)
+
+        self.assertThat(
+            lambda: self._is_dummy_backend_up(), Eventually(Equals(True))
+        )
+
+    def _is_dummy_backend_up(self):
+        interfaces = dbus.SessionBus().list_names()
+
+        return DUMMY_BACKEND_ADDRESS in interfaces
+
+    def ensure_dummy_service_backend_running(self):
+        self._set_contact_service_dummy_backend_environment_variables()
+        self._start_service_dummy_backend()

=== modified file 'tests/autopilot/address_book_app/tests/__init__.py'
--- tests/autopilot/address_book_app/tests/__init__.py	2014-01-30 08:35:04 +0000
+++ tests/autopilot/address_book_app/tests/__init__.py	2014-03-26 22:14:30 +0000
@@ -18,6 +18,7 @@
 from testtools.matchers import Equals
 
 from address_book_app.emulators.main_window import MainWindow
+
 from ubuntuuitoolkit import emulators as toolkit_emulators
 
 
@@ -31,6 +32,7 @@
     VCARD_PATH_DEV = os.path.abspath("../data/vcard.vcf")
     ARGS = []
     PRELOAD_VCARD = False
+    QTCONTACTS_MANAGER_OVERRIDE_MEMORY = True
 
     def setUp(self):
         self.pointing_device = toolkit_emulators.get_pointing_device()
@@ -45,8 +47,17 @@
         else:
             self.app_bin = AddressBookAppTestCase.DEFAULT_DEV_LOCATION
 
-        os.environ['QTCONTACTS_MANAGER_OVERRIDE'] = 'memory'
+        print "Running from: %s" % (self.app_bin)
+        # NOTE defeats favoriting: contacts don't show up in "Favorites" header
+        if AddressBookAppTestCase.QTCONTACTS_MANAGER_OVERRIDE_MEMORY:
+            os.environ['QTCONTACTS_MANAGER_OVERRIDE'] = 'memory'
+        else:
+            try:
+                del(os.environ['QTCONTACTS_MANAGER_OVERRIDE'])
+            except KeyError:
+                pass
         vcard_data = ""
+
         if AddressBookAppTestCase.PRELOAD_VCARD:
             # Use vcard from source tree and fallback on installed vcard (from
             #  address-book-app-autopilot package)
@@ -55,16 +66,16 @@
             else:
                 vcard_data = AddressBookAppTestCase.VCARD_PATH_BIN
 
-        os.environ["ADDRESS_BOOK_TEST_DATA"] = vcard_data
-        if vcard_data != "": print "Using vcard %s" % vcard_data
+        os.environ['ADDRESS_BOOK_TEST_DATA'] = vcard_data
+        if vcard_data != '': print 'Using vcard %s' % vcard_data
         if os.path.exists(self.app_bin):
-            print "Running from: %s" % (self.app_bin)
+            print 'Running from: %s' % (self.app_bin)
             self.launch_test_local()
         elif os.path.exists(self.DEB_LOCALTION):
-            print "Running from: %s" % (self.DEB_LOCALTION)
+            print 'Running from: %s' % (self.DEB_LOCALTION)
             self.launch_test_installed()
         else:
-            print "Running from click package: address-book-app"
+            print 'Running from click package: address-book-app'
             self.launch_click_installed()
 
         AddressBookAppTestCase.ARGS = []

=== modified file 'tests/autopilot/address_book_app/tests/test_add_contact.py'
--- tests/autopilot/address_book_app/tests/test_add_contact.py	2014-02-28 15:36:09 +0000
+++ tests/autopilot/address_book_app/tests/test_add_contact.py	2014-03-26 22:14:30 +0000
@@ -20,6 +20,10 @@
 class TestAddContact(AddressBookAppTestCase):
     """ Tests the Add contact """
 
+    def setUp(self):
+        AddressBookAppTestCase.QTCONTACTS_MANAGER_OVERRIDE_MEMORY = True
+        super(TestAddContact, self).setUp()
+
     def test_go_to_add_contact(self):
         """Test to launch the add contact screen using emulator method"""
         self.assertRaises(

=== modified file 'tests/autopilot/address_book_app/tests/test_contactlist.py'
--- tests/autopilot/address_book_app/tests/test_contactlist.py	2013-11-21 18:53:19 +0000
+++ tests/autopilot/address_book_app/tests/test_contactlist.py	2014-03-26 22:14:30 +0000
@@ -18,6 +18,10 @@
 class TestContactList(AddressBookAppTestCase):
     """Tests the contact list features"""
 
+    def setUp(self):
+        AddressBookAppTestCase.QTCONTACTS_MANAGER_OVERRIDE_MEMORY = True
+        super(TestContactList, self).setUp()
+
     def test_contact_list(self):
         contact_list = self.main_window.get_contact_list_page()
         self.assertThat(contact_list.visible, Eventually(Equals(True)))

=== modified file 'tests/autopilot/address_book_app/tests/test_delete_contact.py'
--- tests/autopilot/address_book_app/tests/test_delete_contact.py	2014-02-13 16:44:12 +0000
+++ tests/autopilot/address_book_app/tests/test_delete_contact.py	2014-03-26 22:14:30 +0000
@@ -40,6 +40,7 @@
     ]
 
     def setUp(self):
+        AddressBookAppTestCase.QTCONTACTS_MANAGER_OVERRIDE_MEMORY = True
         AddressBookAppTestCase.PRELOAD_VCARD = True
         super(TestDeleteSelectContact, self).setUp()
 

=== modified file 'tests/autopilot/address_book_app/tests/test_edit_contact.py'
--- tests/autopilot/address_book_app/tests/test_edit_contact.py	2014-03-18 21:13:04 +0000
+++ tests/autopilot/address_book_app/tests/test_edit_contact.py	2014-03-26 22:14:30 +0000
@@ -17,6 +17,10 @@
 class TestEditContact(AddressBookAppTestCase):
     """Tests edit a contact"""
 
+    def setUp(self):
+        AddressBookAppTestCase.QTCONTACTS_MANAGER_OVERRIDE_MEMORY = True
+        super(TestEditContact, self).setUp()
+
     def test_add_new_phone(self):
         self.add_contact("Fulano", "de Tal", ["3321 2300"])
         edit_page = self.edit_contact(0)

=== added file 'tests/autopilot/address_book_app/tests/test_favorites.py'
--- tests/autopilot/address_book_app/tests/test_favorites.py	1970-01-01 00:00:00 +0000
+++ tests/autopilot/address_book_app/tests/test_favorites.py	2014-03-26 22:14:30 +0000
@@ -0,0 +1,96 @@
+# -*- Mode: Python; coding: utf-8; indent-tabs-mode: nil; tab-width: 4 -*-
+# Copyright 2014 Canonical
+#
+# This program is free software: you can redistribute it and/or modify it
+# under the terms of the GNU General Public License version 3, as published
+# by the Free Software Foundation.
+
+"""Tests of favoriting contacts for the Addressbook App"""
+
+from __future__ import absolute_import
+
+import subprocess
+
+from autopilot.introspection.dbus import StateNotFoundError
+from autopilot.matchers import Eventually
+from testtools.matchers import Equals
+
+from address_book_app.helpers import DummyServiceBackendSetup
+from address_book_app.tests import AddressBookAppTestCase
+from address_book_app.emulators.contact_list_page import ContactListPage
+from address_book_app.emulators.contact_view import ContactView
+
+
+class TestFavorite(AddressBookAppTestCase, DummyServiceBackendSetup):
+    """Test for favoriting contacts"""
+
+    def setUp(self):
+        AddressBookAppTestCase.QTCONTACTS_MANAGER_OVERRIDE_MEMORY = False
+        AddressBookAppTestCase.PRELOAD_VCARD = True
+        self.ensure_dummy_service_backend_running()
+        super(TestFavorite, self).setUp()
+
+    def _add_contact_as_favorite(self, name):
+        contact_list_page = self.main_window.get_contact_list_page()
+        contact_list_page.open_contact_view_by_contact_name(name)
+        contact_view = self.main_window.get_contact_view_page()
+        contact_view.click_favorite_icon()
+        self.main_window.go_back()
+        self._ensure_favorites_shown()
+
+        return contact_list_page
+
+    def _ensure_favorites_shown(self):
+        if not self.main_window.are_favorites_shown():
+            self.main_window.switch_to_favorites()
+            self.main_window.are_favorites_shown().wait_for(True)
+
+    def is_contact_favorite(self, name):
+        contact_list_page = self.main_window.get_contact_list_page()
+        contact_list_page.open_contact_view_by_contact_name(name)
+
+        contact_view = self.main_window.get_contact_view_page()
+        favorite_icon = contact_view.favorite_icon()
+
+        if favorite_icon.name == 'favorite-selected':
+            return True
+        else:
+            return False
+
+    def test_add_favorite(self):
+        """Ensure a contact can be made a favorite."""
+        self._add_contact_as_favorite('teste test34')
+        
+        self.assertTrue(self.is_contact_favorite('teste test34'))
+
+    def test_save_multiple_favorites(self):
+        """Ensure multiple contacts can be made favorites."""
+        self._add_contact_as_favorite('teste test34')
+        self.assertTrue(self.is_contact_favorite('teste test34'))
+
+        self.main_window.go_back()
+        self.main_window.switch_to_all_contact()
+
+        contact_list_page = self._add_contact_as_favorite('teste teste2')        
+        self.assertTrue(self.is_contact_favorite('teste teste2')) 
+
+    def test_remove_favorite(self):
+        """Ensure an already favorite contact can be un-favorited."""
+        contact_list_page = self._add_contact_as_favorite('teste test34')
+        contact_list_page.open_contact_view_by_contact_name('teste test34')
+        contact_view = self.main_window.get_contact_view_page()
+        contact_view.click_favorite_icon()
+        
+        self.assertFalse(contact_list_page.get_favorite_by_name('teste test34'))
+
+    def test_delete_favorite(self):
+        """Ensure deleting a favorite contact also removes it from favorite
+        category."""
+        contact_list_page = self._add_contact_as_favorite('teste test34')
+        contact_list_page.open_contact_view_by_contact_name('teste test34')
+        toolbar = self.main_window.open_toolbar()
+        toolbar.click_button('delete')
+        
+        self.assertRaises(
+            StateNotFoundError,
+            lambda: contact_list_page.get_favorite_by_name('teste test34'))

=== modified file 'tests/autopilot/address_book_app/tests/test_multiple_pick_mode.py'
--- tests/autopilot/address_book_app/tests/test_multiple_pick_mode.py	2013-12-13 19:31:33 +0000
+++ tests/autopilot/address_book_app/tests/test_multiple_pick_mode.py	2014-03-26 22:14:30 +0000
@@ -20,6 +20,7 @@
 
     def setUp(self):
         self.ARGS.append("addressbook:///pick?single=false")
+        AddressBookAppTestCase.QTCONTACTS_MANAGER_OVERRIDE_MEMORY = True
         AddressBookAppTestCase.PRELOAD_VCARD = True
         super(TestMultiplePickerMode, self).setUp()
 

=== modified file 'tests/autopilot/address_book_app/tests/test_single_pick_mode.py'
--- tests/autopilot/address_book_app/tests/test_single_pick_mode.py	2013-12-13 19:31:33 +0000
+++ tests/autopilot/address_book_app/tests/test_single_pick_mode.py	2014-03-26 22:14:30 +0000
@@ -21,6 +21,7 @@
     def setUp(self):
         AddressBookAppTestCase.ARGS.append("addressbook:///pick?single=true")
         AddressBookAppTestCase.PRELOAD_VCARD = True
+        AddressBookAppTestCase.QTCONTACTS_MANAGER_OVERRIDE_MEMORY = True
         super(TestSinglePickerMode, self).setUp()
 
     def test_select_single_contact(self):