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

Proposed by Timo Jyrinki
Status: Merged
Approved by: Timo Jyrinki
Approved revision: 1816
Merged at revision: 1799
Proposed branch: lp:~timo-jyrinki/ubuntu-ui-toolkit/test_jenkins_3_fail
Merge into: lp:~ubuntu-sdk-team/ubuntu-ui-toolkit/jenkins-test
Diff against target: 117 lines (+13/-11)
7 files modified
tests/unit/tst_argument/tst_argument.cpp (+4/-3)
tests/unit/tst_components/tst_clipboard.qml (+1/-0)
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)
ubuntu-sdk.pro (+1/-1)
To merge this branch: bzr merge lp:~timo-jyrinki/ubuntu-ui-toolkit/test_jenkins_3_fail
Reviewer Review Type Date Requested Status
ubuntu-sdk-build-bot continuous-integration Approve
Ubuntu SDK team Pending
Review via email: mp+282284@code.launchpad.net

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

Commit message

fail

To post a comment you must log in.
Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) : 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) : 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)
1809. By Timo Jyrinki

break, break, break

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)
1810. By Timo Jyrinki

it just refuses to break

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)
1811. By Timo Jyrinki

verify_false

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: Needs Fixing (continuous-integration)
1812. By Timo Jyrinki

bump, fixed a job

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)
1813. By Timo Jyrinki

bump, jenkins updated

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)
1814. By Timo Jyrinki

bump, jenkins updated

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: 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)
1815. By Timo Jyrinki

bumpfail

Revision history for this message
ubuntu-sdk-build-bot (ubuntu-sdk-build-bot) wrote :
review: Needs Fixing (continuous-integration)
1816. By Timo Jyrinki

bumpbfail

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: Needs Fixing (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 2013-05-19 14:24:48 +0000
+++ tests/unit/tst_argument/tst_argument.cpp 2016-01-13 16:00:49 +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::Invalid);43 QCOMPARE(argument.at(-2).type(), QVariant::String);
44 QCOMPARE(argument.at(0).type(), QVariant::String);44 QCOMPARE(argument.at(0).type(), QVariant::Invalid);
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,6 +61,7 @@
61 argument.setValueNames(valueNames);61 argument.setValueNames(valueNames);
6262
63 QCOMPARE(argument.syntax(), expectedSyntax);63 QCOMPARE(argument.syntax(), expectedSyntax);
64 verify(false);
64 }65 }
6566
66 void testSyntax_data() {67 void testSyntax_data() {
@@ -72,7 +73,7 @@
72 QStringList valueNames;73 QStringList valueNames;
73 valueNames << "VALUE1" << "VALUE2";74 valueNames << "VALUE1" << "VALUE2";
7475
75 QTest::newRow("no name, no valueNames, not required") << "" << QStringList() << false << "";76 QTest::newRow("no name, no valueNames, not required") << "" << QStringList() << true << "";
76 QTest::newRow("name, no valueNames, not required") << "argument" << QStringList() << false << "--argument";77 QTest::newRow("name, no valueNames, not required") << "argument" << QStringList() << false << "--argument";
77 QTest::newRow("no name, 2 valueNames, not required") << "" << valueNames << false << "[VALUE1] [VALUE2]";78 QTest::newRow("no name, 2 valueNames, not required") << "" << valueNames << false << "[VALUE1] [VALUE2]";
78 QTest::newRow("name, 2 valueNames, not required") << "argument" << valueNames << false << "--argument=VALUE1 VALUE2";79 QTest::newRow("name, 2 valueNames, not required") << "argument" << valueNames << false << "--argument=VALUE1 VALUE2";
7980
=== modified file 'tests/unit/tst_components/tst_clipboard.qml'
--- tests/unit/tst_components/tst_clipboard.qml 2015-03-03 13:20:06 +0000
+++ tests/unit/tst_components/tst_clipboard.qml 2016-01-13 16:00:49 +0000
@@ -69,6 +69,7 @@
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);
72 }73 }
7374
7475
7576
=== modified file 'tests/unit/tst_components/tst_tab.qml'
--- tests/unit/tst_components/tst_tab.qml 2015-03-03 13:20:06 +0000
+++ tests/unit/tst_components/tst_tab.qml 2016-01-13 16:00:49 +0000
@@ -40,7 +40,7 @@
40 }40 }
4141
42 function test_index() {42 function test_index() {
43 compare(tab.index, -1, "is -1 by default")43 compare(tab.index, -2, "is -2 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 2013-05-06 16:33:54 +0000
+++ tests/unit/tst_qquick_image_extension/tst_qquick_image_extension.cpp 2016-01-13 16:00:49 +0000
@@ -38,16 +38,16 @@
3838
39 void scaledBorderIdentity() {39 void scaledBorderIdentity() {
40 UCQQuickImageExtension image;40 UCQQuickImageExtension image;
41 QString border = "border: 13";41 QString border = "border: 10";
42 QString expected = "border: 13";42 QString expected = "border: 11";
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: 13";49 QString border = "border: 5";
50 QString expected = "border: 7";50 QString expected = "border: 9";
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 2015-05-22 18:51:25 +0000
+++ tests/unit/tst_units/dpr3/tst_units_dpr3.cpp 2016-01-13 16:00:49 +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(1.32), 1.33333f);30 QCOMPARE(units.dp(0.12), 1.53333f);
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 2013-12-19 14:46:25 +0000
+++ tests/xvfb.sh 2016-01-13 16:00:49 +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.
88
=== modified file 'ubuntu-sdk.pro'
--- ubuntu-sdk.pro 2015-12-11 13:35:10 +0000
+++ ubuntu-sdk.pro 2016-01-13 16:00:49 +0000
@@ -1,5 +1,5 @@
1#include( documentation/documentation.pri )1#include( documentation/documentation.pri )
22
3BUILD_PATH_CONTENTS="SRC_DIR=\"$$PWD\"" \3BUILD_PATH_CONTENTS="SRC_DIR=\"$$PWD\"" \
4 "BUILD_DIR=\"$$OUT_PWD\""4 "BUILD_DIR=\"$$OUT_PWD\""
55

Subscribers

People subscribed via source and target branches

to all changes: