Merge into trunk : ux-call-msg-helpers : Code : address-book-app

Status:	Work in progress
Proposed branch:	lp:~canonical-platform-qa/address-book-app/ux-call-msg-helpers
Merge into:	lp:address-book-app
Diff against target:	218 lines (+165/-0) 4 files modified debian/control (+1/-0) tests/autopilot/address_book_app/pages/_contact_list_page.py (+33/-0) tests/autopilot/address_book_app/pages/_contact_view.py (+69/-0) tests/autopilot/address_book_app/tests/test_contactlist.py (+62/-0)
To merge this branch:	bzr merge lp:~canonical-platform-qa/address-book-app/ux-call-msg-helpers
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Leo Arias (community)		2014-07-22	Needs Fixing on 2015-02-20
PS Jenkins bot	continuous-integration	2014-07-22	Needs Fixing on 2014-08-13
Canonical Platform QA Team		2014-08-04	Pending
Ubuntu Phablet Team		2014-07-22	Pending
Review via email: mp+227682@code.launchpad.net

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-11: Posted in a previous version of this proposal

#

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

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-11: Posted in a previous version of this proposal

#

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

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-14: Posted in a previous version of this proposal

#

FAILED: Continuous integration, rev:240
http://jenkins.qa.ubuntu.com/job/address-book-app-ci/603/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-amd64-ci/56
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-armhf-ci/56
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-armhf-ci/56/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-i386-ci/56
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1928
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/1606
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2179
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3036
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3036/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/9769
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/1344
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/1809
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/1809/artifact/work/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)

Revision history for this message

Leo Arias (elopio) wrote on 2014-07-14: Posted in a previous version of this proposal

#

Thanks Richard. This is nice.

125 + def get_contact_index_by_name(self, display_name):

This needs some love, but feel free to leave it for the future because it's not your fault. Many things would be a lot easier if we had a method called get_contacts_info on the contacts list page. That method should return a list of contact tuples composed of all the visible information, ordered by the y coordinate. Reminders has a good example of this.
Then you could look for the contact index on that list, instead of in the QML tree.
I'm just saying it would be nice :)

174 + def _get_contact(self, index, phone_number):
This would be clearer as two methods: _get_contact_by_index and _get_contact_by_phone_number

203 + def call_contact(self, index=0, phone_number=None):
215 + def message_contact(self, index=0, phone_number=None):
Just like this, I would make two methods. And for a high level helper, I think index might not be a good parameter. I would be inclined to make a call_contact_by_number and call_contact_by_name. But in our tests we will generally have only one contact, so if you want to call him, index=0 sounds like the easiest thing to do. Feel free to keep it if you prefer that way.

231 + def get_call_button(self):
234 + def get_message_button(self):

Please make these two private method. And make two additional methods click_call_button and click_message_button. That helps encapsulating the responsibilities in the right custom proxy objects.

273 + self.add_contact("Fulano", "de Tal", [self.PHONE_NUMBER])

This is the old helper, it needs to be removed but I didn't have time to refactor all the tests.
This is the new way:

test_contact = data.Contact(
first_name='Fulano', last_name='de Tal',
phones=[self.PHONE_NUMBER])

contact_editor = self.app.main_window.go_to_add_contact()
contact_editor.fill_form(test_contact)

self.app.main_window.save()

You can wrap that in an add_contact method. Please don't use the other one, so it's easier to remove it once we have time.

277 + def convert_phone_number_to_url(self, phone_number):

There's a phone object on the data.py module. This sounds like a good method to have in that object.

Thanks Richard. This is nice.

125	+ def get_contact_index_by_name(self, display_name):

This needs some love, but feel free to leave it for the future because it's not your fault. Many things would be a lot easier if we had a method called get_contacts_info on the contacts list page. That method should return a list of contact tuples composed of all the visible information, ordered by the y coordinate. Reminders has a good example of this.
Then you could look for the contact index on that list, instead of in the QML tree.
I'm just saying it would be nice :)

174	+ def _get_contact(self, index, phone_number):
This would be clearer as two methods: _get_contact_by_index and _get_contact_by_phone_number

203	+ def call_contact(self, index=0, phone_number=None):
215	+ def message_contact(self, index=0, phone_number=None):
Just like this, I would make two methods. And for a high level helper, I think index might not be a good parameter. I would be inclined to make a call_contact_by_number and call_contact_by_name. But in our tests we will generally have only one contact, so if you want to call him, index=0 sounds like the easiest thing to do. Feel free to keep it if you prefer that way.

231	+ def get_call_button(self):
234	+ def get_message_button(self):

Please make these two private method. And make two additional methods click_call_button and click_message_button. That helps encapsulating the responsibilities in the right custom proxy objects.

273	+ self.add_contact("Fulano", "de Tal", [self.PHONE_NUMBER])

This is the old helper, it needs to be removed but I didn't have time to refactor all the tests.
This is the new way:

test_contact = data.Contact(
    first_name='Fulano', last_name='de Tal',
    phones=[self.PHONE_NUMBER])

contact_editor = self.app.main_window.go_to_add_contact()
contact_editor.fill_form(test_contact)

self.app.main_window.save()

You can wrap that in an add_contact method. Please don't use the other one, so it's easier to remove it once we have time.

277	+ def convert_phone_number_to_url(self, phone_number):

There's a phone object on the data.py  module. This sounds like a good method to have in that object.

review: Needs Fixing (code review)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-21: Posted in a previous version of this proposal

#

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

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-22: Posted in a previous version of this proposal

#

FAILED: Continuous integration, rev:243
http://jenkins.qa.ubuntu.com/job/address-book-app-ci/624/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-amd64-ci/77
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-armhf-ci/77
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-armhf-ci/77/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-i386-ci/77
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/2292
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/1893
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2482
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3497
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3497/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/10198
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/1589
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/2123
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/2123/artifact/work/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-22:

#

FAILED: Continuous integration, rev:243
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~canonical-platform-qa/address-book-app/ux-call-msg-helpers/+merge/227682/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/address-book-app-ci/625/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-amd64-ci/78
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-armhf-ci/78
        deb: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-armhf-ci/78/artifact/work/output/*zip*/output.zip
    SUCCESS: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-i386-ci/78
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/2293
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/1894
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-mako/2483
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3498
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/3498/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/10200
    UNSTABLE: http://jenkins.qa.ubuntu.com/job/autopilot-testrunner-otto-utopic/1590
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/2124
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/2124/artifact/work/output/*zip*/output.zip

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

review: Needs Fixing (continuous-integration)

Reply

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-07-29:

#

FAILED: Continuous integration, rev:244
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~canonical-platform-qa/address-book-app/ux-call-msg-helpers/+merge/227682/+edit-commit-message

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

review: Needs Fixing (continuous-integration)

Reply

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-08-04:

#

FAILED: Continuous integration, rev:244
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~canonical-platform-qa/address-book-app/ux-call-msg-helpers/+merge/227682/+edit-commit-message

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

review: Needs Fixing (continuous-integration)

Reply

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-08-04:

#

FAILED: Continuous integration, rev:244
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~canonical-platform-qa/address-book-app/ux-call-msg-helpers/+merge/227682/+edit-commit-message

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

review: Needs Fixing (continuous-integration)

Reply

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-08-04:

#

FAILED: Continuous integration, rev:248
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~canonical-platform-qa/address-book-app/ux-call-msg-helpers/+merge/227682/+edit-commit-message

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

review: Needs Fixing (continuous-integration)

Reply

Revision history for this message

PS Jenkins bot (ps-jenkins) wrote on 2014-08-13:

#

FAILED: Continuous integration, rev:244
http://jenkins.qa.ubuntu.com/job/address-book-app-ci/664/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-amd64-ci/117/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-armhf-ci/117/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/address-book-app-utopic-i386-ci/117/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/3461/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-utopic/2720/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/4708/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-amd64/2997/console

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

review: Needs Fixing (continuous-integration)

Reply

Revision history for this message

Leo Arias (elopio) wrote on 2015-02-20:

#

Needs to be updated.
We will need this soon when automating the telephony stuff from the sanity suite.

review: Needs Fixing

Reply

lp:~canonical-platform-qa/address-book-app/ux-call-msg-helpers updated on 2015-02-24

249. By Richard Huddie on 2015-02-24: merge from trunk

address-book-app

Merge lp:~canonical-platform-qa/address-book-app/ux-call-msg-helpers into lp:address-book-app

Commit message

Description of the change

Unmerged revisions

Preview Diff

Subscribers

 === modified file 'debian/control'
 --- debian/control	2015-02-09 15:02:11 +0000
 +++ debian/control	2015-02-24 15:29:58 +0000
@@ -92,5 +92,6 @@
           ubuntu-ui-toolkit-autopilot (>= 0.1.46+14.10.20140527),
           address-book-app (>= ${binary:Version}),
           ubuntu-mobile-icons,
++         url-dispatcher-testability,
  Description: Test package for address-book-app
   Autopilot tests for the address-book-app package
 === modified file 'tests/autopilot/address_book_app/pages/_contact_list_page.py'
 --- tests/autopilot/address_book_app/pages/_contact_list_page.py	2015-02-19 07:02:09 +0000
 +++ tests/autopilot/address_book_app/pages/_contact_list_page.py	2015-02-24 15:29:58 +0000
@@ -126,6 +126,39 @@
+         ]
          return [label.text for label in name_labels]
++    def get_contact_index_by_name(self, display_name):
++        """
++        Return the index of the contact with the specified name
++        :param display_name: Display name of contact
++        :return: Index of the required contact within the list
++                 None if not found
++        """
++        counter = 0
++        match_index = None
++        contact_list = self.get_contacts()
++        for contact in contact_list:
++            try:
++                # try to find matching contact
++                contact.select_single(
++                    'ContactAvatar', displayName=display_name)
++            except StateNotFoundError:
++                # increase counter and carry on
++                counter += 1
++            else:
++                # found a match, record index and exit loop
++                match_index = counter
++                break
++        return match_index
++
++    def open_contact_by_name(self, display_name):
++        """
++        Open the contact specified by display_name
++        :param display_name: Name of contact to open
++        :return: Page with contact information
++        """
++        contact_index = self.get_contact_index_by_name(display_name)
++        return self.open_contact(contact_index)
++
  class RemoveContactsDialog(
          ubuntuuitoolkit.UbuntuUIToolkitCustomProxyObjectBase):
 === modified file 'tests/autopilot/address_book_app/pages/_contact_view.py'
 --- tests/autopilot/address_book_app/pages/_contact_view.py	2014-06-13 15:05:40 +0000
 +++ tests/autopilot/address_book_app/pages/_contact_view.py	2015-02-24 15:29:58 +0000
@@ -15,6 +15,7 @@
  # along with this program. If not, see <http://www.gnu.org/licenses/>.
  from address_book_app.pages import _common, _contact_editor
++from autopilot.introspection.dbus import StateNotFoundError
  class ContactView(_common.PageWithHeader):
@@ -26,3 +27,71 @@
              _contact_editor.ContactEditor,
              objectName='contactEditorPage',
              active=True)
++
++    def _get_contact_by_phone_number(self, phone_number):
++        """
++        Find the contact using phone number
++        :param phone_number: Phone number for required contact
++        :return: ContactDetailPhoneNumberView matching search requirements
++                 None if match is not found
++        """
++        required_contact = None
++        contacts = self.select_many(ContactDetailPhoneNumberView)
++        if phone_number:
++            # search for a specific phone number
++            for contact in contacts:
++                try:
++                    # try to find matching phone number
++                    contact.select_single('Label', text=phone_number)
++                except StateNotFoundError:
++                    # move onto the next one
++                    pass
++                else:
++                    # found the required contact
++                    required_contact = contact
++                    break
++        return required_contact
++
++    def _get_contact_by_index(self, index):
++        """
++        Find the contact using index
++        :param index: Index offset for required contact
++        :return: ContactDetailPhoneNumberView matching search requirements
++        """
++        return self.select_many(ContactDetailPhoneNumberView)[index]
++
++    def call_contact(self, phone_number):
++        """
++        Press the call button for the required phone number
++        :param phone_number: The specified phone number to search for
++        """
++        contact_phone_number = self._get_contact_by_phone_number(phone_number)
++        contact_phone_number.click_call_button()
++
++    def message_contact(self, phone_number):
++        """
++        Press the message button for the required phone number
++        :param phone_number: The specified phone number to search for
++        """
++        contact_phone_number = self._get_contact_by_phone_number(phone_number)
++        contact_phone_number.click_message_button()
++
++
++class ContactDetailPhoneNumberView(ContactView):
++    """Class to represent a phone number view with call and message buttons"""
++
++    def _get_call_button(self):
++        """Return the call button ActionButton object"""
++        return self.select_single('ActionButton', iconName='call-start')
++
++    def _get_message_button(self):
++        """Return the message button ActionButton object"""
++        return self.select_single('ActionButton', iconName='message')
++
++    def click_call_button(self):
++        """Click on the call button for this phone number"""
++        self.pointing_device.click_object(self._get_call_button())
++
++    def click_message_button(self):
++        """Click on the message button for this phone number"""
++        self.pointing_device.click_object(self._get_message_button())
 === modified file 'tests/autopilot/address_book_app/tests/test_contactlist.py'
 --- tests/autopilot/address_book_app/tests/test_contactlist.py	2014-05-22 13:42:45 +0000
 +++ tests/autopilot/address_book_app/tests/test_contactlist.py	2015-02-24 15:29:58 +0000
@@ -10,13 +10,75 @@
  from autopilot.matchers import Eventually
  from testtools.matchers import Equals
++from address_book_app import data
  from address_book_app.tests import AddressBookAppTestCase
++from url_dispatcher_testability import fixture_setup, fake_dispatcher
  class TestContactList(AddressBookAppTestCase):
      """Tests the contact list features"""
++    TEST_MOBILE_NUMBER = '073331234567'
++
++    def setUp(self):
++        super(TestContactList, self).setUp()
++        self.fake_url_dispatcher = fixture_setup.FakeURLDispatcher()
++        self.useFixture(self.fake_url_dispatcher)
++        self.phone_number = data.Phone(
++            type_='Mobile', number=self.TEST_MOBILE_NUMBER)
++        self.message_url = 'message:///{}'.format(self.phone_number.number)
++        self.tel_url = 'tel:///{}'.format(self.phone_number.number)
++
++    def add_test_contact(self):
++        """
++        Create a contact for testing
++        """
++        test_contact = data.Contact(
++            first_name='Fulano', last_name='de Tal',
++            phones=[self.phone_number])
++        contact_editor = self.app.main_window.go_to_add_contact()
++        contact_editor.fill_form(test_contact)
++        self.app.main_window.save()
++
++    def create_and_open_test_contact(self):
++        """
++        Create a contact for testing, return detail page for new contact
++        :return: Contact detail page
++        """
++        self.add_test_contact()
++        contacts_list = self.app.main_window.get_contact_list_page()
++        return contacts_list.open_contact(0)
++
++    def assert_last_dispatch_url(self, expected_url):
++        """
++        Assert that the last dispatch url matches the expected url
++        :param expected_url: The expected url
++        """
++        url_dispatcher = self.fake_url_dispatcher
++
++        def get_last_dispatch_url_call_parameter():
++            # Workaround for http://pad.lv/1312384
++            try:
++                return url_dispatcher.get_last_dispatch_url_call_parameter()
++            except fake_dispatcher.FakeDispatcherException:
++                return None
++        self.assertThat(
++            get_last_dispatch_url_call_parameter,
++            Eventually(Equals(expected_url)))
++
      def test_contact_list(self):
          contact_list = self.app.main_window.get_contact_list_page()
          self.assertThat(contact_list.visible, Eventually(Equals(True)))
          pass
++
++    def test_call_contact(self):
++        """Press the call button on a contact's phone number"""
++        contact_page = self.create_and_open_test_contact()
++        contact_page.call_contact(self.TEST_MOBILE_NUMBER)
++        self.assert_last_dispatch_url(self.tel_url)
++
++    def test_message_contact(self):
++        """Press the message button on a contact's phone number"""
++        contact_page = self.create_and_open_test_contact()
++        contact_page.message_contact(self.TEST_MOBILE_NUMBER)
++        self.assert_last_dispatch_url(self.message_url)