Merge lp:~om26er/kubuntu-packaging/qtsystems_unittests into lp:~kubuntu-packagers/kubuntu-packaging/qtsystems-opensource-src

Proposed by Omer Akram
Status: Merged
Merged at revision: 23
Proposed branch: lp:~om26er/kubuntu-packaging/qtsystems_unittests
Merge into: lp:~kubuntu-packagers/kubuntu-packaging/qtsystems-opensource-src
Diff against target: 79 lines (+48/-0)
4 files modified
debian/control (+1/-0)
debian/patches/series (+1/-0)
debian/patches/skip_failing_tests.patch (+42/-0)
debian/rules (+4/-0)
To merge this branch: bzr merge lp:~om26er/kubuntu-packaging/qtsystems_unittests
Reviewer Review Type Date Requested Status
Timo Jyrinki Approve
Review via email: mp+198439@code.launchpad.net

Commit message

Run unittests during build, disable failing tests

Description of the change

Run unittests during build, disable failing tests

To post a comment you must log in.
Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :

bzr bd fails:
Applying patch disable_qservicemanager_ipc.patch
can't find file to patch at input line 4
Perhaps you used the wrong -p or --strip option?
The text leading up to this was:
--------------------------
|=== modified file 'tests/auto/serviceframework/serviceframework.pro'
|--- tests/auto/serviceframework/serviceframework.pro 2013-06-14 10:32:13 +0000
|+++ tests/auto/serviceframework/serviceframework.pro 2013-12-10 17:11:31 +0000
--------------------------
No file to patch. Skipping patch.
1 out of 1 hunk ignored
Patch disable_qservicemanager_ipc.patch does not apply (enforce with -f)

review: Needs Fixing
24. By Omer Akram

fix the patch

Revision history for this message
Omer Akram (om26er) wrote :

sorry about that. Fixed the patch now.

25. By Omer Akram

add patch to skip failing tests

26. By Omer Akram

revert last commit

27. By Omer Akram

skip more failing tests

28. By Omer Akram

delete newly added patches

29. By Omer Akram

add DEP3 compiliant patch to skip failing tests

30. By Omer Akram

add missing build-dep for successfully running unittests

Revision history for this message
Timo Jyrinki (timo-jyrinki) wrote :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/control'
--- debian/control 2013-07-11 12:58:03 +0000
+++ debian/control 2013-12-17 09:25:21 +0000
@@ -7,6 +7,7 @@
7 qt5-default,7 qt5-default,
8 qtbase5-private-dev,8 qtbase5-private-dev,
9 qtdeclarative5-private-dev,9 qtdeclarative5-private-dev,
10 libqt5sql5-sqlite,
10Standards-Version: 3.9.411Standards-Version: 3.9.4
11Homepage: http://qt-project.org/12Homepage: http://qt-project.org/
12Vcs-Git: git://anonscm.debian.org/pkg-kde/qt/qtsystems.git13Vcs-Git: git://anonscm.debian.org/pkg-kde/qt/qtsystems.git
1314
=== modified file 'debian/patches/series'
--- debian/patches/series 2013-09-30 06:12:54 +0000
+++ debian/patches/series 2013-12-17 09:25:21 +0000
@@ -2,3 +2,4 @@
2add_license_files.patch2add_license_files.patch
3build_with_udisk.patch3build_with_udisk.patch
4wlan_detect.patch4wlan_detect.patch
5skip_failing_tests.patch
56
=== added file 'debian/patches/skip_failing_tests.patch'
--- debian/patches/skip_failing_tests.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/skip_failing_tests.patch 2013-12-17 09:25:21 +0000
@@ -0,0 +1,42 @@
1Description: Disable the failing tests
2Forwarded: not-needed
3Bug: https://bugreports.qt-project.org/browse/QTBUG-35662
4Bug-Ubuntu: https://launchpad.net/bugs/1261411
5Author: Omer Akram <om26er@ubuntu.com>
6
7=== modified file 'tests/auto/serviceframework/qremoteserviceregister/tst_qremoteserviceregister.cpp'
8--- a/tests/auto/serviceframework/qremoteserviceregister/tst_qremoteserviceregister.cpp 2013-06-14 10:32:13 +0000
9+++ b/tests/auto/serviceframework/qremoteserviceregister/tst_qremoteserviceregister.cpp 2013-12-12 13:18:44 +0000
10@@ -229,6 +229,7 @@
11
12 void tst_QRemoteServiceRegister::tst_instanceClosed()
13 {
14+ QSKIP("QTBUG-35662: fails due to missing /etc/machine-id");
15 qRegisterMetaType<QRemoteServiceRegister::Entry>("QRemoteServiceRegister::Entry");
16 if(!servicePublished)
17 serviceRegister->publishEntries("qt_sfw_example_rsr_unittest");
18
19=== modified file 'tests/auto/serviceframework/servicedeletion/tst_servicedeletion.cpp'
20--- a/tests/auto/serviceframework/servicedeletion/tst_servicedeletion.cpp 2013-06-14 10:32:13 +0000
21+++ b/tests/auto/serviceframework/servicedeletion/tst_servicedeletion.cpp 2013-12-12 13:26:29 +0000
22@@ -116,6 +116,7 @@
23
24 void tst_QServiceDeletion::publishService()
25 {
26+ QSKIP("LP: #1261411 fails due to missing /etc/machine-id");
27 QRemoteServiceRegister *serviceRegister = new QRemoteServiceRegister(this);
28
29 QString serviceName = "TestService";
30
31=== modified file 'tests/auto/serviceframework/serviceframework.pro'
32--- a/tests/auto/serviceframework/serviceframework.pro 2013-06-14 10:32:13 +0000
33+++ b/tests/auto/serviceframework/serviceframework.pro 2013-12-10 17:11:31 +0000
34@@ -5,7 +5,7 @@
35 qservicefilter \
36 qserviceinterfacedescriptor \
37 qservicemanager \
38- qservicemanager_ipc \
39+# qservicemanager_ipc \ #(is failing on Ubuntu)
40 qservicemetadata \
41 servicedeletion
42 # serviceobject
043
=== modified file 'debian/rules'
--- debian/rules 2013-08-09 08:44:14 +0000
+++ debian/rules 2013-12-17 09:25:21 +0000
@@ -40,3 +40,7 @@
40override_dh_builddeb:40override_dh_builddeb:
41 dh_builddeb -- -Zxz41 dh_builddeb -- -Zxz
4242
43override_dh_auto_test:
44 export LD_LIBRARY_PATH=$(CURDIR)/lib
45 cd tests/auto/ && ( test -e Makefile || qmake auto.pro -o Makefile ) && make -f Makefile check
46

Subscribers

People subscribed via source and target branches

to all changes: