Ubuntu Autopilot Tests

Merge lp://staging/~canonical-platform-qa/ubuntu-autopilot-tests/launcher_messaging_experience into lp://staging/ubuntu-autopilot-tests/ubuntu-experience-tests

launcher_messaging_experience
Merge into ubuntu-experience-tests

Proposed by Omer Akram on 2014-07-03

Status:

Needs review

Proposed branch:

lp://staging/~canonical-platform-qa/ubuntu-autopilot-tests/launcher_messaging_experience

Merge into:

lp://staging/ubuntu-autopilot-tests/ubuntu-experience-tests

Diff against target:

83 lines (+56/-1)

3 files modified

debian/control (+2/-0)
ubuntu_experience_tests/helpers.py (+1/-1)
ubuntu_experience_tests/tests/test_user_experiences.py (+53/-0)

To merge this branch:

bzr merge lp://staging/~canonical-platform-qa/ubuntu-autopilot-tests/launcher_messaging_experience

Undecided

In Progress

Link a bug report

Reviewer	Review Type	Date Requested	Status
Brendan Donegan (community)			Needs Fixing on 2014-07-23
PS Jenkins bot	continuous-integration		Needs Fixing on 2014-07-22
Leo Arias (community)		2014-07-03	Needs Fixing on 2014-07-08
Review via email: mp+225465@code.staging.launchpad.net

Commit message

Automate Launcher to Messaging App experience.

Revision history for this message

Omer Akram (om26er) wrote on 2014-07-08:

While the code is ready, this branch depends on four other branches to be merged.

Revision history for this message

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

FAILED: Continuous integration, rev:26
http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/17/
Executed test runs:
    ABORTED: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1692/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-utopic-amd64-ci/17/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-utopic-armhf-ci/17/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-utopic-i386-ci/17/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2733/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/17/rebuild

review: Needs Fixing (continuous-integration)

lp://staging/~canonical-platform-qa/ubuntu-autopilot-tests/launcher_messaging_experience updated on 2014-07-08

27. By Omer Akram on 2014-07-08: commit in working state, will need to adapt to use helpers from address-book-app once they are moved there.

Revision history for this message

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

FAILED: Continuous integration, rev:27
http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/18/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1706/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-utopic-amd64-ci/18/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-utopic-armhf-ci/18/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-utopic-i386-ci/18/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2749/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/18/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

Leo Arias (elopio) wrote on 2014-07-08:

103 + messaging_app = helpers.get_proxy_object_by_process_name(
104 + 'messaging-app',
105 + emulator_base=messaging_emulators.MainView
106 + )

Just as with Brendan's branch, the only emulator_base that we must be using is the one from the toolkit. Definitely not a MainView. There's something wrong here.

107 + messaging_app.start_new_message()
108 + messaging_app.type_contact_phone_num('3333333')
109 + messaging_app.type_message('Hello user!')
110 + messaging_app.click_send_button()

This needs to be wrapped in a helper on the messaging app that's called something like: def send_message(recipients, text)

review: Needs Fixing

lp://staging/~canonical-platform-qa/ubuntu-autopilot-tests/launcher_messaging_experience updated on 2014-07-10

28. By Omer Akram on 2014-07-10: don;t alter launcher icons go with stock
29. By Omer Akram on 2014-07-10: python3 ready
30. By Omer Akram on 2014-07-10: revert the change to get_proxy_object_by_process_name

Revision history for this message

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

FAILED: Continuous integration, rev:30
http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/21/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-utopic-touch/1827/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-utopic-amd64-ci/21/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-utopic-armhf-ci/21/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/ubuntu-autopilot-tests-ubuntu-experience-tests-utopic-i386-ci/21/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-utopic-armhf/2914/console

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/21/rebuild

review: Needs Fixing (continuous-integration)

lp://staging/~canonical-platform-qa/ubuntu-autopilot-tests/launcher_messaging_experience updated on 2014-07-10

31. By Omer Akram on 2014-07-10: use the helper from message_app to send message instead of duplicating the code
32. By Omer Akram on 2014-07-10: use helper from the messaging_app to send message
33. By Omer Akram on 2014-07-10: flake8

Revision history for this message

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

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/22/rebuild

review: Needs Fixing (continuous-integration)

lp://staging/~canonical-platform-qa/ubuntu-autopilot-tests/launcher_messaging_experience updated on 2014-07-11

34. By Omer Akram on 2014-07-11: also depend on messaging-app-autopilot
35. By Omer Akram on 2014-07-11: create a method to add contact

Revision history for this message

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

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/23/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

Brendan Donegan (brendan-donegan) wrote on 2014-07-21:

I'm missing how this code implements the use case in the linked bug:
https://bugs.launchpad.net/ubuntu-autopilot-tests/+bug/1328921. It's supposed to use the quicklist, but as far as I can tell it just fills in the details and sends the message.

review: Needs Information

lp://staging/~canonical-platform-qa/ubuntu-autopilot-tests/launcher_messaging_experience updated on 2014-07-22

36. By Omer Akram on 2014-07-22: adapt to contact fixture in address-book-service instead of doing that stuff in our code
37. By Omer Akram on 2014-07-22: merge trunk

Revision history for this message

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

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/27/rebuild

review: Needs Fixing (continuous-integration)

lp://staging/~canonical-platform-qa/ubuntu-autopilot-tests/launcher_messaging_experience updated on 2014-07-22

38. By Omer Akram on 2014-07-22: fix test as suggested by brendand

Revision history for this message

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

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-autopilot-tests-ubuntu-experience-tests-ci/28/rebuild

review: Needs Fixing (continuous-integration)

Revision history for this message

Brendan Donegan (brendan-donegan) wrote on 2014-07-23:

What is this 'select' option in send_message? I don't see it in messaging-app trunk. I think rather than just calling send_message you want to use the helper to compose a new message (start_new_message), then type in the MultiRecipientInput. You'll need a helper for that based on the one I'm adding here: https://code.launchpad.net/~canonical-platform-qa/messaging-app/more_helpers_test/+merge/227781. That's depending on your address-book-service helper so once that gets merged we can merge that one, then merge this one.

review: Needs Fixing

Unmerged revisions

38. By Omer Akram on 2014-07-22: fix test as suggested by brendand
37. By Omer Akram on 2014-07-22: merge trunk
36. By Omer Akram on 2014-07-22: adapt to contact fixture in address-book-service instead of doing that stuff in our code
35. By Omer Akram on 2014-07-11: create a method to add contact
34. By Omer Akram on 2014-07-11: also depend on messaging-app-autopilot
33. By Omer Akram on 2014-07-10: flake8
32. By Omer Akram on 2014-07-10: use helper from the messaging_app to send message
31. By Omer Akram on 2014-07-10: use the helper from message_app to send message instead of duplicating the code
30. By Omer Akram on 2014-07-10: revert the change to get_proxy_object_by_process_name
29. By Omer Akram on 2014-07-10: python3 ready