Merge lp:~mzanetti/unity/more-stats into lp:unity/phablet

Proposed by Michael Zanetti
Status: Merged
Approved by: Michael Zanetti
Approved revision: no longer in the source branch.
Merged at revision: 551
Proposed branch: lp:~mzanetti/unity/more-stats
Merge into: lp:unity/phablet
Diff against target: 67 lines (+9/-7)
4 files modified
CMakeLists.txt (+3/-3)
plugins/Utils/tests/CMakeLists.txt (+2/-0)
runtests.sh (+1/-1)
tests/qmluitests/tst_Greeter.qml (+3/-3)
To merge this branch: bzr merge lp:~mzanetti/unity/more-stats
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Michał Sawicz Approve
Review via email: mp+157441@code.launchpad.net

Commit message

generate more test statistics

* adjusts the coverage part of CMakeLists.txt to our other projects and makes it discoverable by our jenkins hooks
* enables test result file generation for plugins/Utils
* makes a randomly failing greeter test more robust
* renames coverage.xml for qml to avoid clashing with real lcov/gcov generated file

Description of the change

* adjusts the coverage part of CMakeLists.txt to our other projects and makes it discoverable by our jenkins hooks
* enables test result file generation for plugins/Utils
* makes a randomly failing greeter test more robust
* renames coverage.xml for qml to avoid clashing with real lcov/gcov generated file

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) :
review: Approve
Revision history for this message
Michał Sawicz (saviq) :
review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'CMakeLists.txt'
2--- CMakeLists.txt 2013-04-04 12:41:50 +0000
3+++ CMakeLists.txt 2013-04-08 08:12:19 +0000
4@@ -7,8 +7,8 @@
5 set(CMAKE_AUTOMOC ON)
6
7 # enable QML debugging
8-string(TOLOWER "${CMAKE_BUILD_TYPE}" buildtype_lower) # Build types should always be lowercase but sometimes they are not.
9-if(buildtype_lower MATCHES "debug")
10+string(TOLOWER "${CMAKE_BUILD_TYPE}" cmake_build_type_lower) # Build types should always be lowercase but sometimes they are not.
11+if(cmake_build_type_lower MATCHES "debug")
12 add_definitions(-DQT_QML_DEBUG)
13 endif()
14
15@@ -22,7 +22,7 @@
16 # * Find html report in subdir coveragereport
17 # * Find xml report feasible for jenkins in coverage.xml
18 #####################################################################
19-if(buildtype_lower MATCHES coverage)
20+if(cmake_build_type_lower MATCHES coverage)
21 set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} --coverage" )
22 set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} --coverage" )
23 set(CMAKE_MODULE_LINKER_FLAGS "${CMAKE_MODULE_LINKER_FLAGS} --coverage" )
24
25=== modified file 'plugins/Utils/tests/CMakeLists.txt'
26--- plugins/Utils/tests/CMakeLists.txt 2013-03-28 15:26:58 +0000
27+++ plugins/Utils/tests/CMakeLists.txt 2013-04-08 08:12:19 +0000
28@@ -9,6 +9,8 @@
29 foreach(_test ${ARGN})
30 add_test(NAME ${_test}
31 COMMAND ${CMAKE_CURRENT_BINARY_DIR}/${_test}
32+ -o ${CMAKE_BINARY_DIR}/${_test}.xml,xunitxml
33+ -o -,txt
34 )
35 add_executable(${_test} ${_test}.cpp)
36 qt5_use_modules(${_test} Test Core Qml)
37
38=== modified file 'runtests.sh'
39--- runtests.sh 2013-04-05 10:58:29 +0000
40+++ runtests.sh 2013-04-08 08:12:19 +0000
41@@ -79,7 +79,7 @@
42
43 cd -
44
45-coveragefile=coverage.xml
46+coveragefile=coverage-qml.xml
47
48 echo "<?xml version=\"1.0\" ?>" > $coveragefile
49 echo "<!DOCTYPE coverage" >> $coveragefile
50
51=== modified file 'tests/qmluitests/tst_Greeter.qml'
52--- tests/qmluitests/tst_Greeter.qml 2013-03-26 20:05:18 +0000
53+++ tests/qmluitests/tst_Greeter.qml 2013-04-08 08:12:19 +0000
54@@ -70,10 +70,10 @@
55 var account = findChild(greeter, "username1")
56 mouseClick(account, 1, 1)
57 account = findChild(greeter, "username0")
58- mouseClick(account, 1, 1)
59 var passwordInput = findChild(greeter, "passwordInput")
60- tryCompare(passwordInput, "opacity", 0)
61- tryCompare(passwordInput, "opacity", 1)
62+ mouseClick(account, 1, 1)
63+ wait(0) // spin event loop to start any pending animations
64+ tryCompare(passwordInput, "opacity", 1) // wait for opacity animation to be finished
65 mouseClick(passwordInput, 1, 1)
66 compare(unlockSpy.count, 0)
67 typeString("wr0ng p4ssw0rd")

Subscribers

People subscribed via source and target branches