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

Proposed by Albert Astals Cid
Status: Merged
Approved by: Michał Sawicz
Approved revision: 1116
Merged at revision: 1132
Proposed branch: lp:~aacid/unity8/prepend_test_xml
Merge into: lp:unity8
Diff against target: 57 lines (+4/-3)
4 files modified
cmake/modules/QmlTest.cmake (+2/-2)
plugins/Dash/CardCreator.js (+0/-1)
qml/Dash/DashApplication.qml (+1/-0)
qml/ScopeTool.qml (+1/-0)
To merge this branch: bzr merge lp:~aacid/unity8/prepend_test_xml
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Needs Fixing
Michał Sawicz Approve
Review via email: mp+229766@code.launchpad.net

Commit message

Add test prefix to xml output, seems CI needs it

Description of the change

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

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

 * 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
Michał Sawicz (saviq) :
review: Needs Fixing
Revision history for this message
Albert Astals Cid (aacid) wrote :

Suffix dropped

lp:~aacid/unity8/prepend_test_xml updated
1116. By Albert Astals Cid

drop Test suffix

Revision history for this message
Michał Sawicz (saviq) wrote :

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

 * Did CI run pass? If not, please explain why.
Not yet, but this can't cause anything.

review: Approve
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Needs Fixing (continuous-integration)
Revision history for this message
Michał Sawicz (saviq) wrote :

WooHoo, almost 50% more tests reported ;)

Revision history for this message
Albert Astals Cid (aacid) wrote :

Now we'll have to fix this test :D

lp:~aacid/unity8/prepend_test_xml updated
1117. By Albert Astals Cid

Move the declaration of Ubuntu.Thumbnailer 0.1 to the application root file

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

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'cmake/modules/QmlTest.cmake'
--- cmake/modules/QmlTest.cmake 2014-07-29 16:14:51 +0000
+++ cmake/modules/QmlTest.cmake 2014-08-06 19:30:14 +0000
@@ -135,7 +135,7 @@
135 set(testCommand135 set(testCommand
136 LD_LIBRARY_PATH=${LD_PATH}136 LD_LIBRARY_PATH=${LD_PATH}
137 ${CMAKE_CURRENT_BINARY_DIR}/${CLASS_NAME}TestExec137 ${CMAKE_CURRENT_BINARY_DIR}/${CLASS_NAME}TestExec
138 -o ${CMAKE_BINARY_DIR}/${CLASS_NAME}Test.xml,xunitxml138 -o ${CMAKE_BINARY_DIR}/test${CLASS_NAME}.xml,xunitxml
139 -o -,txt)139 -o -,txt)
140140
141 add_qmltest_target(test${CLASS_NAME} "${testCommand}" FALSE TRUE)141 add_qmltest_target(test${CLASS_NAME} "${testCommand}" FALSE TRUE)
@@ -151,7 +151,7 @@
151 LD_LIBRARY_PATH=${LD_PATH}151 LD_LIBRARY_PATH=${LD_PATH}
152 xvfb-run --server-args "-screen 0 1024x768x24" --auto-servernum152 xvfb-run --server-args "-screen 0 1024x768x24" --auto-servernum
153 ${CMAKE_CURRENT_BINARY_DIR}/${CLASS_NAME}TestExec153 ${CMAKE_CURRENT_BINARY_DIR}/${CLASS_NAME}TestExec
154 -o ${CMAKE_BINARY_DIR}/${CLASS_NAME}Test.xml,xunitxml154 -o ${CMAKE_BINARY_DIR}/test${CLASS_NAME}.xml,xunitxml
155 -o -,txt)155 -o -,txt)
156156
157 add_qmltest_target(xvfbtest${CLASS_NAME} "${xvfbtestCommand}" FALSE TRUE)157 add_qmltest_target(xvfbtest${CLASS_NAME} "${xvfbtestCommand}" FALSE TRUE)
158158
=== modified file 'plugins/Dash/CardCreator.js'
--- plugins/Dash/CardCreator.js 2014-07-28 09:28:50 +0000
+++ plugins/Dash/CardCreator.js 2014-08-06 19:30:14 +0000
@@ -593,7 +593,6 @@
593function createCardComponent(parent, template, components) {593function createCardComponent(parent, template, components) {
594 var imports = 'import QtQuick 2.2; \n\594 var imports = 'import QtQuick 2.2; \n\
595 import Ubuntu.Components 0.1; \n\595 import Ubuntu.Components 0.1; \n\
596 import Ubuntu.Thumbnailer 0.1;\n\
597 import Dash 0.1;\n';596 import Dash 0.1;\n';
598 var card = cardString(template, components);597 var card = cardString(template, components);
599 var code = imports + 'Component {\n' + card + '}\n';598 var code = imports + 'Component {\n' + card + '}\n';
600599
=== modified file 'qml/Dash/DashApplication.qml'
--- qml/Dash/DashApplication.qml 2014-07-29 14:28:16 +0000
+++ qml/Dash/DashApplication.qml 2014-08-06 19:30:14 +0000
@@ -16,6 +16,7 @@
1616
17import QtQuick 2.217import QtQuick 2.2
18import Ubuntu.Components 1.018import Ubuntu.Components 1.0
19import Ubuntu.Thumbnailer 0.1 // Register support for image://thumbnailer/ and image://albumart/
1920
20MainView {21MainView {
21 width: units.gu(40)22 width: units.gu(40)
2223
=== modified file 'qml/ScopeTool.qml'
--- qml/ScopeTool.qml 2014-07-03 09:08:22 +0000
+++ qml/ScopeTool.qml 2014-08-06 19:30:14 +0000
@@ -17,6 +17,7 @@
17import QtQuick 2.017import QtQuick 2.0
18import Ubuntu.Components 0.118import Ubuntu.Components 0.1
19import Ubuntu.Components.Popups 0.119import Ubuntu.Components.Popups 0.1
20import Ubuntu.Thumbnailer 0.1 // Register support for image://thumbnailer/ and image://albumart/
20import Utils 0.121import Utils 0.1
21import Unity 0.222import Unity 0.2
22import "Components"23import "Components"

Subscribers

People subscribed via source and target branches