Merge lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/orientationManual into lp:ubuntu-ui-toolkit/staging
| Status: | Rejected | ||||||||||||
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| Rejected by: | Christian Dywan on 2015-08-04 | ||||||||||||
| Proposed branch: | lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/orientationManual | ||||||||||||
| Merge into: | lp:ubuntu-ui-toolkit/staging | ||||||||||||
| Diff against target: |
664 lines (+304/-37) 19 files modified
modules/Ubuntu/Components/1.3/MainView.qml (+1/-0) modules/Ubuntu/Components/1.3/MainViewBase.qml (+19/-0) modules/Ubuntu/Components/1.3/OrientationHelper.qml (+3/-9) modules/Ubuntu/Components/1.3/TextCursor.qml (+10/-1) modules/Ubuntu/Components/Popups/1.2/popupUtils.js (+1/-1) modules/Ubuntu/Components/Popups/1.3/PopupBase.qml (+6/-7) modules/Ubuntu/Components/plugin/plugin.cpp (+0/-7) modules/Ubuntu/Components/plugin/ucapplication.cpp (+41/-1) modules/Ubuntu/Components/plugin/ucapplication.h (+5/-0) tests/autopilot/ubuntuuitoolkit/_custom_proxy_objects/_mainview.py (+13/-0) tests/autopilot/ubuntuuitoolkit/tests/components/test_popover.py (+44/-0) tests/autopilot/ubuntuuitoolkit/tests/components/test_popover.qml (+56/-5) tests/autopilot/ubuntuuitoolkit/tests/components/test_textinput.header.qml (+1/-1) tests/autopilot/ubuntuuitoolkit/tests/components/test_textinput.py (+26/-0) tests/autopilot/ubuntuuitoolkit/tests/components/test_textinput.textarea.qml (+3/-1) tests/autopilot/ubuntuuitoolkit/tests/components/test_textinput.textfield.qml (+11/-3) tests/autopilot/ubuntuuitoolkit/tests/components/test_textinput.textfield_custom.qml (+3/-1) tests/autopilot/ubuntuuitoolkit/tests/custom_proxy_objects/test_main_view.orientation.qml (+29/-0) tests/autopilot/ubuntuuitoolkit/tests/custom_proxy_objects/test_main_view.py (+32/-0) |
||||||||||||
| To merge this branch: | bzr merge lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/orientationManual | ||||||||||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-06-18 | |
| Leo Arias (community) | Approve on 2015-04-13 | ||
| Tim Peeters | Approve on 2015-04-13 | ||
| Zsombor Egri | 2015-03-31 | Pending | |
|
Review via email:
|
|||
This proposal supersedes a proposal from 2015-03-09.
Commit Message
Allow manual orientation changes through OrientationHelp
Description of the Change
Autopilot tests for orientation issues in both text input and popovers/ dialogs:
env APP_ID=uitl LANGUAGE=en ./run_tests.sh ubuntuuitoolkit
env APP_ID=uitl LANGUAGE=en ./run_tests.sh ubuntuuitoolkit
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1178
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1179
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Tim Peeters (tpeeters) wrote : | # |
If someone would be setting OrientationHelp
| Tim Peeters (tpeeters) wrote : | # |
22 + // Functions to allow Autopilot to handle orientation
23 + objectName: "MainViewBaseSt
24 + function setOrientation(
25 + if (newOrientation == 'Landscape')
26 + UbuntuApplicati
27 + else if (newOrientation == 'Portrait')
28 + UbuntuApplicati
29 + else
30 + UbuntuApplicati
31 + }
32 + function getOrientation() {
33 + return UbuntuApplicati
34 + }
Could this be implemented in autopilot without adding the JS functions here? Ca we get to the UbuntuApplication in AP?
| Tim Peeters (tpeeters) wrote : | # |
46 - property int orientationAngle: automaticOrient
47 + readonly property int orientationAngle: automaticOrient
48 + ? Screen.
49 + : 0
This breaks API. Perhaps we should split up OrientationHelper in an 1.0 version an 1.3 version.
Was there any use case for setting the value previously? And did you check that apps don't set the orientationAngle?
| Tim Peeters (tpeeters) wrote : | # |
53 - onOrientationAn
54 + onOrientationAn
55 + : d.tryUpdateState();
why did you remove the { }?
Personally I would use the old-fashioned if-then-else ;) but that's a matter of taste I guess.
| Tim Peeters (tpeeters) wrote : | # |
63 - /*
64 - The attached property Screen.orientation is only valid inside Item or
65 - derived components. Inside Object it evaluates to 0 with no error.
66 - Also be aware that some apps eg. webbrowser-app set window.
67 - and thus can hide failure to update it from this code.
68 - See http://
69 - */
(why) is this comment no longer valid?
The link would require an update though.
| Tim Peeters (tpeeters) wrote : | # |
nevermind that^ comment. Got it.
> 63 - /*
> 64 - The attached property Screen.orientation is only valid inside
> Item or
> 65 - derived components. Inside Object it evaluates to 0 with no
> error.
> 66 - Also be aware that some apps eg. webbrowser-app set
> window.
> 67 - and thus can hide failure to update it from this code.
> 68 - See http://
> window2-screen.html
> 69 - */
>
> (why) is this comment no longer valid?
>
> The link would require an update though.
| Tim Peeters (tpeeters) wrote : | # |
107 - automaticOrient
108 - stateWrapper.
109 + rotation: stateWrapper.
110 + ? Screen.
111 anchorToKeyboard: true
112
113 LayoutMirroring
114 @@ -180,6 +179,7 @@
115 Item {
116 id: stateWrapper
117 property Item rootItem: QuickUtils.
118 + property bool automaticOrient
This change doesn't really make it prettier. Does the automaticOrient
| Tim Peeters (tpeeters) wrote : | # |
130 - parent: QuickUtils.
131 + // Parent to the contentsItem to be able to inherit rotation
132 + parent: {
133 + var mainView = QuickUtils.
134 + // Check in case the root isn't a MainView
135 + return mainView.
136 + }
Can you add a comment explaining what you expect to be mainView.
| Tim Peeters (tpeeters) wrote : | # |
201 +/*!
202 + * \qmlproperty Qt.ScreenOrient
203 + * \internal
204 + * The current orientation of the application - can be overridden manually for testing.
205 + */
Why is the component name here Application and not UbuntuApplication? (same question would apply to Applcation:
And why is it internal? Don't we want this to be documented? (again, same question for applicationName).
| Tim Peeters (tpeeters) wrote : | # |
214 +void UCApplication:
215 + if (m_orientation == -1) {
216 + // we're now in manual override mode
217 + disconnect(
218 + this, &UCApplication:
219 + }
220 +
221 + if (m_orientation != orientation) {
222 + m_orientation = orientation;
223 + Q_EMIT orientationChan
224 + }
225 +}
You want to re-connect when the orientation is set to -1. Also, document that you can set it to -1 to make it follow the Qt value.
| Tim Peeters (tpeeters) wrote : | # |
276 + def _set_orientatio
277 + self.select_single(
278 + objectName=
279 + orientation)
280 + # Short delay to ensure introspection is uptodate
281 + time.sleep(1)
282 +
283 + @property
284 + def _orientation(self):
285 + return self.select_single(
286 + objectName=
Since we are using _orientation outside of the CPOs themselves (in tests/components), perhaps we should expose it properly? So no _-prefix?
Also, we should also test the property in tests/custom_
| Tim Peeters (tpeeters) wrote : | # |
281 + time.sleep(1)
When changing orientation, there can be an animation going on because of the orientation change. It is better to have a proper wait for that (similar to waitForAnimation() in _header.py)?
| Tim Peeters (tpeeters) wrote : | # |
324 + # Give rendering a chance to finish
325 + time.sleep(2)
Here you would call the proper waitForOrientat
Anyway I think if you physically rotate a device, the rotation animation is more than 2ms.
| Tim Peeters (tpeeters) wrote : | # |
311 + scenarios = [
312 + ('Popover', dict(button=
313 + ('Small dialog', dict(button=
314 + ('Huge dialog', dict(button=
315 + ]
I prefer buttonName, popoverName etc to name the variables, to make the code that follows more clear (the first time I read popover = self.main_
| Tim Peeters (tpeeters) wrote : | # |
+ def _rotate(self, orientation):
I don't know if there is a policy on this, but I noticed that in the other python component tests, we don't prefix functions with _.
| Tim Peeters (tpeeters) wrote : | # |
330 + def test_popover_
331 + self._rotate(
332 +
333 + def test_popover_
334 + self._rotate(
Minor detail: test_rotate_
| Tim Peeters (tpeeters) wrote : | # |
304 +class PopoverOpenTest
you are not testing opening of a Popover, but its rotation.
| Tim Peeters (tpeeters) wrote : | # |
440 + def _rotate(self, orientation):
other functions (like select_cursor() in the same class) don't use the _ prefix.
454 + def test_rotate_
455 + self._rotate(
456 +
457 + def test_rotate_
458 + self._rotate(
459
same as above, I would prefer test_rotate_
| Tim Peeters (tpeeters) wrote : | # |
> env APP_ID=uitl LANGUAGE=en ./run_tests.sh >ubuntuuitoolki
This works on my desktop without the APP_ID. Is that because I'm on desktop?
Where does the uitl id come from? Is that something common for all our tests?
| Christian Dywan (kalikiana) wrote : | # |
> 201 +/*!
> 202 + * \qmlproperty Qt.ScreenOrient
> 203 + * \internal
> 204 + * The current orientation of the application - can be overridden
> manually for testing.
> 205 + */
>
> Why is the component name here Application and not UbuntuApplication? (same
> question would apply to Applcation:
>
> And why is it internal? Don't we want this to be documented? (again, same
> question for applicationName).
That whole object is internal API that exists for things we can't implement in QML.
| Christian Dywan (kalikiana) wrote : | # |
> 214 +void UCApplication:
> {
> 215 + if (m_orientation == -1) {
> 216 + // we're now in manual override mode
> 217 + disconnect(
> &QScreen:
> 218 + this, &UCApplication:
> 219 + }
> 220 +
> 221 + if (m_orientation != orientation) {
> 222 + m_orientation = orientation;
> 223 + Q_EMIT orientationChan
> 224 + }
> 225 +}
>
> You want to re-connect when the orientation is set to -1. Also, document that
> you can set it to -1 to make it follow the Qt value.
I don't see a use case for that. As the documentation says you can override it for testing, and if you do that it makes no sense to go back to physical orientation.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1180
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Christian Dywan (kalikiana) wrote : | # |
> > env APP_ID=uitl LANGUAGE=en ./run_tests.sh >ubuntuuitoolki
> .test_textinput
>
>
> This works on my desktop without the APP_ID. Is that because I'm on desktop?
> Where does the uitl id come from? Is that something common for all our tests?
I use it to get rid of some of the error spam which you get despite the fact that it is indeed neither required nor usually set on the desktop.
| Christian Dywan (kalikiana) wrote : | # |
> 324 + # Give rendering a chance to finish
> 325 + time.sleep(2)
>
> Here you would call the proper waitForOrientat
> suggested above.
>
> Anyway I think if you physically rotate a device, the rotation animation is
> more than 2ms.
I added a rotating.wait_for. But it's not apparently enough for AP to update its coordinate space when the animation is in fact finished. So I left a sleep(1), not sure what else I could check for here. The one for the cursor is in fact unnecessary, though, so I removed it.
| Christian Dywan (kalikiana) wrote : | # |
I addressed all the naming suggestions, made the set_orientation function public and added a test for it. Note there's no actual way to verify rotation and as such absent from the cpo test because one of the orientations is always a no-op depending on the primary orientation and the real tests don't need to be aware of it… after having written this I actually convinced myself that this isn't acceptable… I'll have to look into a way to address this.
| Leo Arias (elopio) wrote : | # |
I left some comments.
| Tim Peeters (tpeeters) wrote : | # |
9 + // Function provided here because Autopilot can't change property
Is the function that you refer to getOrientation(), which you removed in a later commit?
| Tim Peeters (tpeeters) wrote : | # |
264 + def set_orientation
265 + self.select_single(
266 + objectName=
267 + orientation)
268 + self.select_
269 + # Short delay to ensure introspection is uptodate
270 + time.sleep(1)
I wonder if the problem here is that when you do the wait_for, perhaps the value of the property was not updated to be True yet? In that case it is better to add the sleep(1) before the wait_for.
| Tim Peeters (tpeeters) wrote : | # |
In two occasions, you use select_
I think it is better to use an objectName for selecting the OrientationHelper. In the past we had problems some times (with versioning or custom subclasses of components) when using the type.
| Tim Peeters (tpeeters) wrote : | # |
118 + do {
119 + orientationHelper = orientationHelp
120 + } while (orientationHelper && !orientationHel
isn't orientationHelper != null already implied by orientationHelper? Anyway it would be weird to put the != null in the end.
Also, if you are looking for orientationHelper, you should be checking orientationHelp
| Christian Dywan (kalikiana) wrote : | # |
> 9 + // Function provided here because Autopilot can't change
> property
>
> Is the function that you refer to getOrientation(), which you removed in a
> later commit?
There still is a function right below the comment…
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1186
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Leo Arias (elopio) wrote : | # |
<elopio> kalikiana: FOLLOW_
<elopio> is it really scream?
<kalikiana> erm
<kalikiana> well, timp asked me to fix the typo in SCREEM…
<kalikiana> ^_^'
<kalikiana> brainfart…
<elopio> other than that, lgt.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Autolanding.
More details in the following jenkins job:
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1191
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Tim Peeters (tpeeters) wrote : | # |
Found 1 license problems:
tests/autopilot
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Autolanding.
More details in the following jenkins job:
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
deb: http://
UNSTABLE: http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1192
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1193. By Christian Dywan on 2015-04-15
-
Only test non-default orientation for clear results
Otherwise some properties may not change if nothing did rotate.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1193
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1194. By Christian Dywan on 2015-04-17
- 1195. By Christian Dywan on 2015-04-20
-
Take Component-based popover rootItem from the caller
- 1196. By Christian Dywan on 2015-04-20
-
FakeCursor parent should fallback to toplevel
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1195
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1197. By Christian Dywan on 2015-04-20
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1197
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1198. By Christian Dywan on 2015-04-20
-
Put back accidentally removed "import os"
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1198
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1199. By Christian Dywan on 2015-05-12
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1199
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
UNSTABLE: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1200. By Christian Dywan on 2015-05-13
-
Empty commit
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1200
http://
Executed test runs:
UNSTABLE: http://
UNSTABLE: http://
UNSTABLE: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1201. By Christian Dywan on 2015-05-21
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1201
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1202. By Christian Dywan on 2015-05-21
-
Update components.api
- 1203. By Christian Dywan on 2015-05-21
-
Undo accidental changes
- 1204. By Christian Dywan on 2015-05-21
-
Move all orientation tests to 1.3
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1204
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1205. By Christian Dywan on 2015-05-22
-
Revert bogus components.api changes
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1205
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
UNSTABLE: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1206. By Christian Dywan on 2015-06-01
-
Normalize rotation angle in tests
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1206
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1207. By Christian Dywan on 2015-06-02
- 1208. By Christian Dywan on 2015-06-02
-
Use Popups 1.3 in test_popover.qml
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1207
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1208
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1209. By Christian Dywan on 2015-06-02
-
Assert focus after tapping TextField in popover tests
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1209
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1210. By Christian Dywan on 2015-06-03
-
Anchor Column in test_textinput.
textfield. qml to the top
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1210
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1211. By Christian Dywan on 2015-06-05
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1211
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1212. By Christian Dywan on 2015-06-10
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1212
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1213. By Christian Dywan on 2015-06-11
-
Use Eventually and wait_ instead of time.sleep
- 1214. By Christian Dywan on 2015-06-11
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1214
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1215. By Christian Dywan on 2015-06-12
-
Make flake8 happy once again
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1215
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1216. By Christian Dywan on 2015-06-18
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1216
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
Unmerged revisions
- 1216. By Christian Dywan on 2015-06-18
- 1215. By Christian Dywan on 2015-06-12
-
Make flake8 happy once again
- 1214. By Christian Dywan on 2015-06-11
- 1213. By Christian Dywan on 2015-06-11
-
Use Eventually and wait_ instead of time.sleep
- 1212. By Christian Dywan on 2015-06-10
- 1211. By Christian Dywan on 2015-06-05
- 1210. By Christian Dywan on 2015-06-03
-
Anchor Column in test_textinput.
textfield. qml to the top - 1209. By Christian Dywan on 2015-06-02
-
Assert focus after tapping TextField in popover tests
- 1208. By Christian Dywan on 2015-06-02
-
Use Popups 1.3 in test_popover.qml
- 1207. By Christian Dywan on 2015-06-02

FAILED: Continuous integration, rev:1177 jenkins. qa.ubuntu. com/job/ ubuntu- sdk-team- ubuntu- ui-toolkit- staging- ci/1613/ jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- vivid-touch/ 2056/console jenkins. qa.ubuntu. com/job/ ubuntu- sdk-team- ubuntu- ui-toolkit- staging- vivid-amd64- ci/340/ console jenkins. qa.ubuntu. com/job/ ubuntu- sdk-team- ubuntu- ui-toolkit- staging- vivid-armhf- ci/343/ console jenkins. qa.ubuntu. com/job/ ubuntu- sdk-team- ubuntu- ui-toolkit- staging- vivid-i386- ci/340/ console jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- vivid-armhf/ 2054/console
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/ubuntu- sdk-team- ubuntu- ui-toolkit- staging- ci/1613/ rebuild
http://