Merge lp:~osomon/webbrowser-app/workaround-bug-1417118 into lp:webbrowser-app
| Status: | Merged |
|---|---|
| Approved by: | Olivier Tilloy on 2015-02-17 |
| Approved revision: | 900 |
| Merged at revision: | 904 |
| Proposed branch: | lp:~osomon/webbrowser-app/workaround-bug-1417118 |
| Merge into: | lp:webbrowser-app |
| Diff against target: |
86 lines (+17/-11) 4 files modified
README (+1/-1) debian/control (+4/-4) tests/autopilot/webbrowser_app/emulators/browser.py (+5/-2) tests/autopilot/webbrowser_app/tests/test_addressbar_states.py (+7/-4) |
| To merge this branch: | bzr merge lp:~osomon/webbrowser-app/workaround-bug-1417118 |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-02-17 | |
| Riccardo Padovani (community) | Approve on 2015-02-04 | ||
| Ubuntu Phablet Team | 2015-02-04 | Pending | |
|
Review via email:
|
|||
Commit Message
Work around autopilot test failure by ensuring that the selection is cleared before clicking on the action button.
| Olivier Tilloy (osomon) wrote : | # |
Thanks for testing Riccardo :)
However this is not an acceptable fix. I merely submitted it as a MR to have a complete CI run on it to see if it resolves the issue in otto. If it does, we’ll still need to figure out what the root cause of the issue is, and fix it properly.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:894
http://
Executed test runs:
SUCCESS: http://
UNSTABLE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
- 895. By Olivier Tilloy on 2015-02-04
-
Revert previous revision.
- 896. By Olivier Tilloy on 2015-02-04
-
Remove autopilot-touch runtime dependency.
It was originally added as a convenience for when I want to run test packages on a device with phablet-test-run.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:896
http://
Executed test runs:
UNSTABLE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:896
http://
Executed test runs:
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:896
http://
Executed test runs:
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
- 897. By Olivier Tilloy on 2015-02-05
-
Click somewhere in the leftmost half of the action button to ensure the event is not intercepted by the text selection handle.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:897
http://
Executed test runs:
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:897
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
- 898. By Olivier Tilloy on 2015-02-06
-
Updated README with runtime dependencies needed for autopilot tests.
- 899. By Olivier Tilloy on 2015-02-06
-
More robust workaround.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:899
http://
Executed test runs:
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:899
http://
Executed test runs:
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:899
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 900. By Olivier Tilloy on 2015-02-17
-
Yet another workaround for bug #1417118: on devices, clicking again in the address bar might trigger the context menu to show instead of clearing the selection as initially expected.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:900
http://
Executed test runs:
UNSTABLE: http://
FAILURE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://

autopilot3 run webbrowser_ app.tests. test_addressbar _states. TestAddressBarS tates.test_ looses_ focus_when_ reloading /Documents/ ubuntu/ touch/system- apps/webbrowser -app/workaround -bug-1417118/tests/autopilot
Loading tests from: /home/rpadovani
Tests running... request. QueryExtension' >
<class 'Xlib.protocol.
Ran 1 test in 4.467s
OK
###
vivid on amd64
Thanks!