Merge lp:~timo-jyrinki/ubuntu-ui-toolkit/test_jenkins_2 into lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/jenkins-test

Proposed by Timo Jyrinki
Status: Merged
Approved by: Timo Jyrinki
Approved revision: 1813
Merged at revision: 1800
Proposed branch: lp:~timo-jyrinki/ubuntu-ui-toolkit/test_jenkins_2
Merge into: lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/jenkins-test
Diff against target: 107 lines (+10/-12)
6 files modified
tests/unit/tst_argument/tst_argument.cpp (+3/-4)
tests/unit/tst_components/tst_clipboard.qml (+0/-1)
tests/unit/tst_components/tst_tab.qml (+1/-1)
tests/unit/tst_qquick_image_extension/tst_qquick_image_extension.cpp (+4/-4)
tests/unit/tst_units/dpr3/tst_units_dpr3.cpp (+1/-1)
tests/xvfb.sh (+1/-1)
To merge this branch: bzr merge lp:~timo-jyrinki/ubuntu-ui-toolkit/test_jenkins_2
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve
Ubuntu SDK team Pending
Review via email: mp+282283@code.launchpad.net

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

Commit message

important white space fix for jenkins-test branch

To post a comment you must log in.
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote : Posted in a previous version of this proposal

-------------
try N+1 next
------------

Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote : Posted in a previous version of this proposal

--- Next up jobs after regen pbuilder ---

Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote : Posted in a previous version of this proposal

One more time.

Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote : Posted in a previous version of this proposal
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :

FAILED: Autolanding.
Merging failed. More details in the following jenkins job:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-autolanding/58/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-land-mp/58/console

review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :

FAILED: Autolanding.
More details in the following jenkins job:
https://jenkins.ubuntu.com/ubuntu-sdk/job/ubuntu-ui-toolkit-autolanding/59/
Executed test runs:
    None: https://jenkins.ubuntu.com/ubuntu-sdk/job/generic-land-mp/59/console

review: Needs Fixing (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'tests/unit/tst_argument/tst_argument.cpp'
--- tests/unit/tst_argument/tst_argument.cpp 2016-01-12 13:17:04 +0000
+++ tests/unit/tst_argument/tst_argument.cpp 2016-01-14 07:03:06 +0000
@@ -40,8 +40,8 @@
40 values << "value1" << "value2";40 values << "value1" << "value2";
41 argument.setValues(values);41 argument.setValues(values);
4242
43 QCOMPARE(argument.at(-2).type(), QVariant::String);43 QCOMPARE(argument.at(-2).type(), QVariant::Invalid);
44 QCOMPARE(argument.at(0).type(), QVariant::Invalid);44 QCOMPARE(argument.at(0).type(), QVariant::String);
45 QCOMPARE(argument.at(1).type(), QVariant::String);45 QCOMPARE(argument.at(1).type(), QVariant::String);
46 QCOMPARE(argument.at(2).type(), QVariant::Invalid);46 QCOMPARE(argument.at(2).type(), QVariant::Invalid);
4747
@@ -61,7 +61,6 @@
61 argument.setValueNames(valueNames);61 argument.setValueNames(valueNames);
6262
63 QCOMPARE(argument.syntax(), expectedSyntax);63 QCOMPARE(argument.syntax(), expectedSyntax);
64 verify(false);
65 }64 }
6665
67 void testSyntax_data() {66 void testSyntax_data() {
@@ -73,7 +72,7 @@
73 QStringList valueNames;72 QStringList valueNames;
74 valueNames << "VALUE1" << "VALUE2";73 valueNames << "VALUE1" << "VALUE2";
7574
76 QTest::newRow("no name, no valueNames, not required") << "" << QStringList() << true << "";75 QTest::newRow("no name, no valueNames, not required") << "" << QStringList() << false << "";
77 QTest::newRow("name, no valueNames, not required") << "argument" << QStringList() << false << "--argument";76 QTest::newRow("name, no valueNames, not required") << "argument" << QStringList() << false << "--argument";
78 QTest::newRow("no name, 2 valueNames, not required") << "" << valueNames << false << "[VALUE1] [VALUE2]";77 QTest::newRow("no name, 2 valueNames, not required") << "" << valueNames << false << "[VALUE1] [VALUE2]";
79 QTest::newRow("name, 2 valueNames, not required") << "argument" << valueNames << false << "--argument=VALUE1 VALUE2";78 QTest::newRow("name, 2 valueNames, not required") << "argument" << valueNames << false << "--argument=VALUE1 VALUE2";
8079
=== modified file 'tests/unit/tst_components/tst_clipboard.qml'
--- tests/unit/tst_components/tst_clipboard.qml 2016-01-12 13:17:04 +0000
+++ tests/unit/tst_components/tst_clipboard.qml 2016-01-14 07:03:06 +0000
@@ -69,7 +69,6 @@
69 compare(Clipboard.data.color, "#ff0000", "Color pushed");69 compare(Clipboard.data.color, "#ff0000", "Color pushed");
70 Clipboard.push(["application/x-color", "green"]);70 Clipboard.push(["application/x-color", "green"]);
71 compare(Clipboard.data.color, standalone.color, "Color pushed");71 compare(Clipboard.data.color, standalone.color, "Color pushed");
72 verify(false);
73 }72 }
7473
7574
7675
=== modified file 'tests/unit/tst_components/tst_tab.qml'
--- tests/unit/tst_components/tst_tab.qml 2016-01-12 07:41:35 +0000
+++ tests/unit/tst_components/tst_tab.qml 2016-01-14 07:03:06 +0000
@@ -40,7 +40,7 @@
40 }40 }
4141
42 function test_index() {42 function test_index() {
43 compare(tab.index, -2, "is -2 by default")43 compare(tab.index, -1, "is -1 by default")
44 }44 }
4545
46 Tab {46 Tab {
4747
=== modified file 'tests/unit/tst_qquick_image_extension/tst_qquick_image_extension.cpp'
--- tests/unit/tst_qquick_image_extension/tst_qquick_image_extension.cpp 2016-01-12 11:33:12 +0000
+++ tests/unit/tst_qquick_image_extension/tst_qquick_image_extension.cpp 2016-01-14 07:03:06 +0000
@@ -38,16 +38,16 @@
3838
39 void scaledBorderIdentity() {39 void scaledBorderIdentity() {
40 UCQQuickImageExtension image;40 UCQQuickImageExtension image;
41 QString border = "border: 10";41 QString border = "border: 13";
42 QString expected = "border: 11";42 QString expected = "border: 13";
43 QString result = image.scaledBorder(border, "1");43 QString result = image.scaledBorder(border, "1");
44 QCOMPARE(result, expected);44 QCOMPARE(result, expected);
45 }45 }
4646
47 void scaledBorderHalf() {47 void scaledBorderHalf() {
48 UCQQuickImageExtension image;48 UCQQuickImageExtension image;
49 QString border = "border: 5";49 QString border = "border: 13";
50 QString expected = "border: 9";50 QString expected = "border: 7";
51 QString result = image.scaledBorder(border, "0.5");51 QString result = image.scaledBorder(border, "0.5");
52 QCOMPARE(result, expected);52 QCOMPARE(result, expected);
53 }53 }
5454
=== modified file 'tests/unit/tst_units/dpr3/tst_units_dpr3.cpp'
--- tests/unit/tst_units/dpr3/tst_units_dpr3.cpp 2016-01-12 10:31:33 +0000
+++ tests/unit/tst_units/dpr3/tst_units_dpr3.cpp 2016-01-14 07:03:06 +0000
@@ -27,7 +27,7 @@
27 UCUnits units;27 UCUnits units;
2828
29 QCOMPARE(units.dp(1.0), 1.0f);29 QCOMPARE(units.dp(1.0), 1.0f);
30 QCOMPARE(units.dp(0.12), 1.53333f);30 QCOMPARE(units.dp(1.32), 1.33333f);
31 QCOMPARE(units.dp(1.72), 1.66666f);31 QCOMPARE(units.dp(1.72), 1.66666f);
32 QCOMPARE(units.dp(0.23), 0.333333f);32 QCOMPARE(units.dp(0.23), 0.333333f);
33 QCOMPARE(units.dp(0.51), 0.666666f);33 QCOMPARE(units.dp(0.51), 0.666666f);
3434
=== modified file 'tests/xvfb.sh'
--- tests/xvfb.sh 2016-01-12 14:29:08 +0000
+++ tests/xvfb.sh 2016-01-14 07:03:06 +0000
@@ -1,7 +1,7 @@
1#!/bin/sh1#!/bin/sh
2#2#
3# Copyright 2013 Canonical Ltd.3# Copyright 2013 Canonical Ltd.
4# 4#
5# This program is free software; you can redistribute it and/or modify5# This program is free software; you can redistribute it and/or modify
6# it under the terms of the GNU Lesser General Public License as published by6# it under the terms of the GNU Lesser General Public License as published by
7# the Free Software Foundation; version 3.7# the Free Software Foundation; version 3.

Subscribers

People subscribed via source and target branches

to all changes: