Unity 8

Merge lp:~cimi/unity8/more_stable_lazy_image_test into lp:unity8

  1. more_stable_lazy_image_test
  2. Merge into trunk
Proposed by Andrea Cimitan on 2016-02-01
Status: Superseded
Proposed branch: lp:~cimi/unity8/more_stable_lazy_image_test
Merge into: lp:unity8
Diff against target: 86 lines (+27/-9)
1 file modified
tests/qmltests/Components/tst_LazyImage.qml (+27/-9)
To merge this branch: bzr merge lp:~cimi/unity8/more_stable_lazy_image_test
Reviewer Review Type Date Requested Status
PS Jenkins bot continuous-integration Needs Fixing on 2016-02-01
Unity8 CI Bot continuous-integration Needs Fixing on 2016-02-01
Unity Team 2016-02-01 Pending
Review via email: mp+284596@code.launchpad.net

This proposal supersedes a proposal from 2016-02-01.

This proposal has been superseded by a proposal from 2016-02-02.

To post a comment you must log in.
Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2172
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~cimi/unity8/more_stable_lazy_image_test/+merge/284596/+edit-commit-message

https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/232/
Executed test runs:

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/232/rebuild

review: Needs Fixing (continuous-integration)
Albert Astals Cid (aacid) wrote :

Nope:

FAIL! : qmltestrunner::LazyImage::test_lazyimage(Unbound Wide) property running
   Actual (): false
   Expected (): true
   Loc: [/home/tsdgeos_work/phablet/unity8/unity8-ubuntu-xenial-landing-019/tests/qmltests/Components/tst_LazyImage.qml(180)]

PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:2172
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~cimi/unity8/more_stable_lazy_image_test/+merge/284596/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity8-ci/7181/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/6277/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-xenial-touch/596/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/1886
    FAILURE: http://jenkins.qa.ubuntu.com/job/unity8-qmluitest-xenial-amd64/589/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/1781
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/1781
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-amd64-ci/588
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-i386-ci/587
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-touch/4829/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6288
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6288/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27198
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-xenial-touch/313/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/594
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/594/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27197

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/7181/rebuild

review: Needs Fixing (continuous-integration)
lp:~cimi/unity8/more_stable_lazy_image_test updated on 2016-02-01
2173. By Andrea Cimitan on 2016-02-01

Yet another attempt

Unity8 CI Bot (unity8-ci-bot) wrote :

FAILED: Continuous integration, rev:2173
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~cimi/unity8/more_stable_lazy_image_test/+merge/284596/+edit-commit-message

https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/236/
Executed test runs:

Click here to trigger a rebuild:
https://unity8-jenkins.ubuntu.com/job/lp-unity8-1-ci/236/rebuild

review: Needs Fixing (continuous-integration)
PS Jenkins bot (ps-jenkins) wrote :

FAILED: Continuous integration, rev:2173
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~cimi/unity8/more_stable_lazy_image_test/+merge/284596/+edit-commit-message

http://jenkins.qa.ubuntu.com/job/unity8-ci/7185/
Executed test runs:
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-vivid-touch/6283/console
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-xenial-touch/600/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-phablet-qmluitests-vivid/1890
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-qmluitest-xenial-amd64/593
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-amd64-ci/1785
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-vivid-i386-ci/1785
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-amd64-ci/592
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity8-xenial-i386-ci/591
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-vivid-touch/4831/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6294
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-vivid-armhf/6294/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27204
    FAILURE: http://jenkins.qa.ubuntu.com/job/generic-deb-autopilot-runner-xenial-touch/315/console
    SUCCESS: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/598
        deb: http://jenkins.qa.ubuntu.com/job/generic-mediumtests-builder-xenial-armhf/598/artifact/work/output/*zip*/output.zip
    SUCCESS: http://s-jenkins.ubuntu-ci:8080/job/touch-flash-device/27203

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity8-ci/7185/rebuild

review: Needs Fixing (continuous-integration)
lp:~cimi/unity8/more_stable_lazy_image_test updated on 2016-02-02
2174. By Andrea Cimitan on 2016-02-02

merged el trunko

Unmerged revisions

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'tests/qmltests/Components/tst_LazyImage.qml'
2--- tests/qmltests/Components/tst_LazyImage.qml 2016-01-28 17:45:51 +0000
3+++ tests/qmltests/Components/tst_LazyImage.qml 2016-02-02 10:22:27 +0000
4@@ -112,6 +112,11 @@
5 }
6 }
7
8+ SignalSpy {
9+ id: signalSpy
10+ signalName: "runningChanged"
11+ }
12+
13 UT.UnityTestCase {
14 name: "LazyImage"
15 when: windowShown
16@@ -123,6 +128,18 @@
17 tryCompare(lazy2, "height", units.gu(10));
18 controls3.blank();
19 tryCompare(lazy3, "width", units.gu(10));
20+
21+ tryCompare(lazy1, "state", "default");
22+ var transition = findChildIn(lazy1, "transitions", "genericTransition");
23+ tryCompare(transition, "running", false);
24+
25+ tryCompare(lazy2, "state", "default");
26+ transition = findChildIn(lazy2, "transitions", "genericTransition");
27+ tryCompare(transition, "running", false);
28+
29+ tryCompare(lazy3, "state", "default");
30+ transition = findChildIn(lazy3, "transitions", "genericTransition");
31+ tryCompare(transition, "running", false);
32 }
33
34 function test_lazyimage_data() {
35@@ -131,33 +148,34 @@
36 {tag: "Unbound Wide", image: lazy1, func: controls1.wide, transition: "readyTransition", width: 160, height: 80, imageWidth: 160, imageHeight: 80, initialWidth: units.gu(10), initialHeight: units.gu(10)},
37 {tag: "Unbound Square", image: lazy1, func: controls1.square, transition: "readyTransition", width: 160, height: 160, imageWidth: 160, imageHeight: 160, initialWidth: units.gu(10), initialHeight: units.gu(10)},
38 {tag: "Unbound Portrait", image: lazy1, func: controls1.portrait, transition: "readyTransition", width: 80, height: 160, imageWidth: 80, imageHeight: 160, initialWidth: units.gu(10), initialHeight: units.gu(10)},
39- {tag: "Unbound Bad path", image: lazy1, func: controls1.badpath, transition: "genericTransition", width: units.gu(10), height: units.gu(10), imageWidth: units.gu(10), imageHeight: units.gu(10), initialWidth: units.gu(10), initialHeight: units.gu(10), placeholder: true, error: true},
40+ {tag: "Unbound Bad path", image: lazy1, func: controls1.badpath, transition: "genericTransition", state: "error", width: units.gu(10), height: units.gu(10), imageWidth: units.gu(10), imageHeight: units.gu(10), initialWidth: units.gu(10), initialHeight: units.gu(10), placeholder: true, error: true},
41 {tag: "Width-bound Blank", image: lazy2, func: controls2.blank, width: units.gu(30), height: units.gu(10), imageWidth: units.gu(30), imageHeight: units.gu(10), initialWidth: units.gu(30), initialHeight: units.gu(10), placeholder: true},
42 {tag: "Width-bound Wide", image: lazy2, func: controls2.wide, transition: "readyTransition", width: units.gu(30), height: units.gu(15), imageWidth: units.gu(30), imageHeight: units.gu(15), initialWidth: units.gu(30), initialHeight: units.gu(10)},
43 {tag: "Width-bound Square", image: lazy2, func: controls2.square, transition: "readyTransition", width: units.gu(30), height: units.gu(30), imageWidth: units.gu(30), imageHeight: units.gu(30), initialWidth: units.gu(30), initialHeight: units.gu(10)},
44 {tag: "Width-bound Portrait", image: lazy2, func: controls2.portrait, transition: "readyTransition", width: units.gu(30), height: units.gu(60), imageWidth: units.gu(30), imageHeight: units.gu(60), initialWidth: units.gu(30), initialHeight: units.gu(10)},
45- {tag: "Width-bound Bad path", image: lazy2, func: controls2.badpath, transition: "genericTransition", width: units.gu(30), height: units.gu(10), imageWidth: units.gu(30), imageHeight: units.gu(10), initialWidth: units.gu(30), initialHeight: units.gu(10), placeholder: true, error: true},
46+ {tag: "Width-bound Bad path", image: lazy2, func: controls2.badpath, transition: "genericTransition", state: "error", width: units.gu(30), height: units.gu(10), imageWidth: units.gu(30), imageHeight: units.gu(10), initialWidth: units.gu(30), initialHeight: units.gu(10), placeholder: true, error: true},
47 {tag: "Height-bound Blank", image: lazy3, func: controls3.blank, width: units.gu(10), height: units.gu(12), imageWidth: units.gu(10), imageHeight: units.gu(12), initialWidth: units.gu(10), initialHeight: units.gu(12), placeholder: true},
48 {tag: "Height-bound Wide", image: lazy3, func: controls3.wide, transition: "readyTransition", width: units.gu(24), height: units.gu(12), imageWidth: units.gu(24), imageHeight: units.gu(12), initialWidth: units.gu(10), initialHeight: units.gu(12)},
49 {tag: "Height-bound Square", image: lazy3, func: controls3.square, transition: "readyTransition", width: units.gu(12), height: units.gu(12), imageWidth: units.gu(12), imageHeight: units.gu(12), initialWidth: units.gu(10), initialHeight: units.gu(12)},
50 {tag: "Height-bound Portrait", image: lazy3, func: controls3.portrait, transition: "readyTransition", width: units.gu(6), height: units.gu(12), imageWidth: units.gu(6), imageHeight: units.gu(12), initialWidth: units.gu(10), initialHeight: units.gu(12)},
51- {tag: "Height-bound Bad path", image: lazy3, func: controls3.badpath, transition: "genericTransition", width: units.gu(10), height: units.gu(12), imageWidth: units.gu(10), imageHeight: units.gu(12), initialWidth: units.gu(10), initialHeight: units.gu(12), placeholder: true, error: true},
52+ {tag: "Height-bound Bad path", image: lazy3, func: controls3.badpath, transition: "genericTransition", state: "error", width: units.gu(10), height: units.gu(12), imageWidth: units.gu(10), imageHeight: units.gu(12), initialWidth: units.gu(10), initialHeight: units.gu(12), placeholder: true, error: true},
53 {tag: "Fit Blank", image: lazy4, func: controls4.blank, width: units.gu(12), height: units.gu(12), imageWidth: units.gu(12), imageHeight: units.gu(12), initialWidth: units.gu(12), initialHeight: units.gu(12), placeholder: true},
54 {tag: "Fit Wide", image: lazy4, func: controls4.wide, transition: "readyTransition", width: units.gu(12), height: units.gu(12), imageWidth: units.gu(12), imageHeight: units.gu(6), initialWidth: units.gu(12), initialHeight: units.gu(12)},
55 {tag: "Fit Square", image: lazy4, func: controls4.square, transition: "readyTransition", width: units.gu(12), height: units.gu(12), imageWidth: units.gu(12), imageHeight: units.gu(12), initialWidth: units.gu(12), initialHeight: units.gu(12)},
56 {tag: "Fit Portrait", image: lazy4, func: controls4.portrait, transition: "readyTransition", width: units.gu(12), height: units.gu(12), imageWidth: units.gu(6), imageHeight: units.gu(12), initialWidth: units.gu(12), initialHeight: units.gu(12)},
57- {tag: "Fit Bad path", image: lazy4, func: controls4.badpath, transition: "genericTransition", width: units.gu(12), height: units.gu(12), imageWidth: units.gu(12), imageHeight: units.gu(12), initialWidth: units.gu(12), initialHeight: units.gu(12), placeholder: true, error: true},
58+ {tag: "Fit Bad path", image: lazy4, func: controls4.badpath, transition: "genericTransition", state: "error", width: units.gu(12), height: units.gu(12), imageWidth: units.gu(12), imageHeight: units.gu(12), initialWidth: units.gu(12), initialHeight: units.gu(12), placeholder: true, error: true},
59 ]
60 }
61
62 function test_lazyimage(data) {
63+ var transition = findChildIn(data.image, "transitions", data.transition);
64+ signalSpy.target = transition;
65+ signalSpy.clear();
66+
67 data.func();
68
69 if (data.transition) {
70- waitForRendering(data.image);
71- // wait for the transition to complete
72- var transition = findChildIn(data.image, "transitions", data.transition);
73- tryCompare(transition, "running", true);
74+ tryCompare(data.image, "state", data.state ? data.state : "ready");
75 tryCompare(transition, "running", false);
76 }
77
78@@ -186,7 +204,7 @@
79
80 // check the error image
81 var error = findChild(data.image, "errorImage");
82- compare(error.visible, data.error ? true : false);
83+ tryCompare(error, "visible", data.error ? true : false);
84 }
85 }
86 }

Subscribers

People subscribed via source and target branches