Merge lp://staging/~elopio/webapps-demo/fix_autopilot_tests into lp://staging/webapps-demo

Proposed by Leo Arias
Status: Needs review
Proposed branch: lp://staging/~elopio/webapps-demo/fix_autopilot_tests
Merge into: lp://staging/webapps-demo
Diff against target: 113 lines (+4/-54)
3 files modified
tests/autopilot/webapps_click_packages_autopilot/emulators/__init__.py (+0/-8)
tests/autopilot/webapps_click_packages_autopilot/tests/__init__.py (+4/-36)
tests/autopilot/webapps_click_packages_autopilot/tests/test_click_packages.py (+0/-10)
To merge this branch: bzr merge lp://staging/~elopio/webapps-demo/fix_autopilot_tests
Reviewer Review Type Date Requested Status
Víctor R. Ruiz (community) Approve
Alexandre Abreu (community) Approve
PS Jenkins bot continuous-integration Approve
Allan LeSage Pending
Richard Huddie Pending
Ubuntu Phablet Team Pending
Review via email: mp+212529@code.staging.launchpad.net

Commit message

Update the autopilot tests.

To post a comment you must log in.
Revision history for this message
Leo Arias (elopio) wrote :

This is the smallest possible diff to get the tests passing. There's still some cleaning and improvements to do.

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Alexandre Abreu (abreu-alexandre) wrote :

LGTM & works on N4

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

Looks good to me.

review: Approve

Unmerged revisions

41. By Leo Arias

No need for tear down, autopilot kills the apps.

40. By Leo Arias

Removed unused imports.

39. By Leo Arias

Changed the name of the webview class.

38. By Leo Arias

Removed the unused code.

37. By Leo Arias

Assign the app proxy.

36. By Leo Arias

Use autopilot's launch_click_package.

35. By Leo Arias

Removed all the unused stuff.

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

to all changes: