Merge lp://qastaging/~rhuddie/ubuntu-autopilot-tests/create-u1-user-install-app into lp://qastaging/ubuntu-autopilot-tests/ubuntu-experience-tests

Proposed by Richard Huddie
Status: Work in progress
Proposed branch: lp://qastaging/~rhuddie/ubuntu-autopilot-tests/create-u1-user-install-app
Merge into: lp://qastaging/ubuntu-autopilot-tests/ubuntu-experience-tests
Diff against target: 74 lines (+70/-0)
1 file modified
ubuntu_integration_tests/tests/test_u1_create_user_install_app.py (+70/-0)
To merge this branch: bzr merge lp://qastaging/~rhuddie/ubuntu-autopilot-tests/create-u1-user-install-app
Reviewer Review Type Date Requested Status
Javier Collado Pending
Víctor R. Ruiz Pending
Leo Arias Pending
Ubuntu Testcase Admins Pending
Review via email: mp+209878@code.qastaging.launchpad.net

Description of the change

Psuedo code for proposed integration test covering creating a new u1 user, installing an app and then launcing it:
1. Open system settings.
2. Go to online accounts.
3. Go to Ubuntu One.
4. Fill the form for a new user.
5. Close online accounts, or swipe it out.
6. Open the applications unity scope.
7. Click an application from the more suggestions section.
8. Click install.
9. Wait for the install to complete.
10. Click open.
11. Validate that the installed app is running

To post a comment you must log in.
Revision history for this message
Javier Collado (javier.collado) wrote :

The code proposed for the test case looks good to me.

The only comment that I have is that probably the `open_app` method can return
better information (maybe a PID) to be passed to the `assert_app_is_running`
method rather than the application name.

Following with that, maybe the assertion can be executed by the app object
itself:
selected_app.assert_is_running()

Anyway, this is something that will be clearer once implementation progress
gets to that point.

Revision history for this message
Víctor R. Ruiz (vrruiz) wrote :

65 + selected_app, selected_app_name = suggested_apps.press_first_available_app()

This should check that the values returned aren't null (no available app to install).

Revision history for this message
Leo Arias (elopio) wrote :

53 + system_settings = self.open_system_settings() -> Javier

54 + online_accounts = system_settings.open_online_accounts() -> Victor

65 + selected_app, selected_app_name = suggested_apps.press_first_available_app() -> Richard

70 + selected_app.open_app() -> Leo

Unmerged revisions

12. By Richard Huddie

added pseudo code for test steps

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