Merge lp:~aacid/unity8/use_dbus_test_runner into lp:unity8

Proposed by Albert Astals Cid
Status: Merged
Approved by: Michael Zanetti
Approved revision: 1756
Merged at revision: 1767
Proposed branch: lp:~aacid/unity8/use_dbus_test_runner
Merge into: lp:unity8
Prerequisite: lp:~aacid/unity8/fixruntests
Diff against target: 35 lines (+10/-8)
1 file modified
cmake/modules/QmlTest.cmake (+10/-8)
To merge this branch: bzr merge lp:~aacid/unity8/use_dbus_test_runner
Reviewer Review Type Date Requested Status
Michael Zanetti (community) Approve
PS Jenkins bot (community) continuous-integration Needs Fixing
Review via email: mp+257784@code.launchpad.net

This proposal supersedes a proposal from 2015-04-28.

Commit message

Use dbus-test-runner instead of dbus-launch

Since we actually depend on it in the control file

Description of the change

* Are there any related MPs required for this MP to build/function as expected?
Prerequisite

 * Did you perform an exploratory manual test run of your code change and any related functionality?
Yes

 * Did you make sure that your branch does not contain spurious tags?
Yes

 * If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
N/A

 * If you changed the UI, has there been a design review?
N/A

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote : Posted in a previous version of this proposal
review: Needs Fixing (continuous-integration)
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
1756. By Albert Astals Cid

Merge fixruntests

Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Michael Zanetti (mzanetti) wrote :

 * Did you perform an exploratory manual test run of the code change and any related functionality?

This is only for jenkins, and it behaves as it should now.

 * Did CI run pass? If not, please explain why.

no, which is the purpose of this branch.

 * Did you make sure that the branch does not contain spurious tags?

yes

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'cmake/modules/QmlTest.cmake'
2--- cmake/modules/QmlTest.cmake 2015-02-05 20:21:00 +0000
3+++ cmake/modules/QmlTest.cmake 2015-05-04 10:34:48 +0000
4@@ -155,10 +155,11 @@
5 set(testCommand
6 ${ENVVAR}
7 LD_LIBRARY_PATH=${LD_PATH}
8- dbus-launch
9- ${CMAKE_CURRENT_BINARY_DIR}/${CLASS_NAME}TestExec
10- -o ${CMAKE_BINARY_DIR}/test${CLASS_NAME}.xml,xunitxml
11- -o -,txt)
12+ dbus-test-runner
13+ -m 300
14+ -t ${CMAKE_CURRENT_BINARY_DIR}/${CLASS_NAME}TestExec
15+ -p -o -p ${CMAKE_BINARY_DIR}/test${CLASS_NAME}.xml,xunitxml
16+ -p -o -p -,txt)
17
18 add_qmltest_target(test${CLASS_NAME} "${testCommand}" FALSE TRUE)
19 add_dependencies(test${CLASS_NAME} ${CLASS_NAME}TestExec ${DEPS})
20@@ -173,10 +174,11 @@
21 ${ENVVAR}
22 LD_LIBRARY_PATH=${LD_PATH}
23 xvfb-run --server-args "-screen 0 1024x768x24" --auto-servernum
24- dbus-launch
25- ${CMAKE_CURRENT_BINARY_DIR}/${CLASS_NAME}TestExec
26- -o ${CMAKE_BINARY_DIR}/test${CLASS_NAME}.xml,xunitxml
27- -o -,txt)
28+ dbus-test-runner
29+ -m 300
30+ -t ${CMAKE_CURRENT_BINARY_DIR}/${CLASS_NAME}TestExec
31+ -p -o -p ${CMAKE_BINARY_DIR}/test${CLASS_NAME}.xml,xunitxml
32+ -p -o -p -,txt)
33
34 add_qmltest_target(xvfbtest${CLASS_NAME} "${xvfbtestCommand}" FALSE TRUE)
35 add_dependencies(qmluitests xvfbtest${CLASS_NAME})

Subscribers

People subscribed via source and target branches