Merge lp:~unity-team/unity8/refactor-qmltests into lp:unity8
| Status: | Superseded |
|---|---|
| Proposed branch: | lp:~unity-team/unity8/refactor-qmltests |
| Merge into: | lp:unity8 |
| Prerequisite: | lp:~aacid/unity8/more_stable_lvwpht |
| Diff against target: |
1581 lines (+635/-478) 34 files modified
CMakeLists.txt (+9/-0) cmake/modules/QmlPlugins.cmake (+1/-3) cmake/modules/QmlTest.cmake (+308/-214) plugins/Wizard/PageList.cpp (+1/-1) runtests.sh (+1/-1) tests/CMakeLists.txt (+122/-10) tests/copyright/CMakeLists.txt (+1/-1) tests/libs/UbuntuGestures/CMakeLists.txt (+4/-8) tests/plugins/AccountsService/CMakeLists.txt (+5/-4) tests/plugins/Dash/CMakeLists.txt (+10/-17) tests/plugins/Greeter/Unity/Launcher/CMakeLists.txt (+4/-5) tests/plugins/LightDM/CMakeLists.txt (+13/-4) tests/plugins/Ubuntu/Gestures/CMakeLists.txt (+23/-25) tests/plugins/Unity/Indicators/CMakeLists.txt (+9/-12) tests/plugins/Unity/Indicators/IndicatorsManagerTest.cpp (+1/-1) tests/plugins/Unity/Indicators/IndicatorsModelTest.cpp (+1/-1) tests/plugins/Unity/Indicators/MenuContentActivatorTest.cpp (+1/-1) tests/plugins/Unity/Indicators/RootActionStateTest.cpp (+1/-1) tests/plugins/Unity/Indicators/SharedUnityMenuModelTest.cpp (+1/-1) tests/plugins/Unity/Indicators/UnityMenuModelStackTest.cpp (+1/-1) tests/plugins/Unity/Launcher/CMakeLists.txt (+5/-5) tests/plugins/Unity/Session/CMakeLists.txt (+5/-5) tests/plugins/Utils/CMakeLists.txt (+12/-26) tests/plugins/Utils/HomeKeyWatcherTest.cpp (+1/-1) tests/plugins/Utils/ModelTest.cpp (+1/-1) tests/plugins/Utils/QLimitProxyModelTest.cpp (+2/-2) tests/plugins/Utils/TimeFormatterTest.cpp (+1/-11) tests/plugins/Utils/UnitySortFilterProxyModelTest.cpp (+2/-2) tests/plugins/Wizard/CMakeLists.txt (+2/-2) tests/qmltests/CMakeLists.txt (+76/-93) tests/qmltests/Components/CMakeLists.txt (+4/-10) tests/qmltests/Stages/tst_WindowMoveResizeArea.qml (+1/-0) tests/qmltests/utils/Unity/Test/tst_UnityTest.qml (+5/-8) tests/whitespace/CMakeLists.txt (+1/-1) |
| To merge this branch: | bzr merge lp:~unity-team/unity8/refactor-qmltests |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Michael Zanetti (community) | 2015-05-19 | Approve on 2015-05-19 | |
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-05-19 | |
| Albert Astals Cid (community) | 2015-04-29 | Abstain on 2015-05-19 | |
|
Review via email:
|
|||
This proposal has been superseded by a proposal from 2015-05-27.
Commit Message
Refactor QmlTest.cmake module so that all tests can go through it.
Also a bit of cleanup around tests.
ARTIFACTS_DIR directs xunitxml output to that dir
QUIET_LOGGER lets you disable stdout logging
Description of the Change
* Are there any related MPs required for this MP to build/function as expected? Please list.
Prerequisite
* Did you perform an exploratory manual test run of your code change and any related functionality?
Y
* Did you make sure that your branch does not contain spurious tags?
Y
* 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
- 1762. By Michał Sawicz on 2015-04-30
-
Make gdb and xvfb-run optional
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1762
http://
Executed test runs:
None: http://
None: http://
FAILURE: http://
FAILURE: http://
None: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1762
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 1763. By Michał Sawicz on 2015-04-30
-
Transition gesture tests to new macros
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1763
http://
Executed test runs:
UNSTABLE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1764. By Michał Sawicz on 2015-04-30
-
PageList only wants testing env in the unit test.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1764
http://
Executed test runs:
UNSTABLE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1765. By Michał Sawicz on 2015-04-30
-
GreeterDBus test needs dbus.
- 1766. By Michał Sawicz on 2015-04-30
-
Drop quotes
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1766
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
function(
import_
Should this be qmltestrunner?
Also the docu is add_manual_qml_test instead of add_qml_unittest?
- 1767. By Michał Sawicz on 2015-05-01
-
Refactor library and import path handling
- 1768. By Michał Sawicz on 2015-05-01
-
Add testcase name in tst_WindowMoveR
esizeArea. qml - 1769. By Michał Sawicz on 2015-05-01
-
Add LIGHTDM where needed
- 1770. By Michał Sawicz on 2015-05-01
-
Fix add_qml_unittest.
- 1771. By Michał Sawicz on 2015-05-01
-
Drop unneeded qmltestrunner import
| Michał Sawicz (saviq) wrote : | # |
W dniu 01.05.2015 o 11:41, Albert Astals Cid pisze:
> Should this be qmltestrunner?
Yup.
> Also the docu is add_manual_qml_test instead of add_qml_unittest?
Fixed.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1768
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1772. By Michał Sawicz on 2015-05-01
-
Add UnityTest name
- 1773. By Michał Sawicz on 2015-05-01
-
Singleton bug is fixed.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1770
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1773
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 1774. By Michał Sawicz on 2015-05-07
-
Use CMAKE_LIBRARY_
ARCHITECTURE instead of asking gcc
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1774
http://
Executed test runs:
UNSTABLE: http://
FAILURE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1774
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in runtests.sh
Text conflict in tests/plugins/
Text conflict in tests/qmltests/
3 conflicts encountered.
- 1775. By Albert Astals Cid on 2015-05-19
-
Merge lp:unity8
| Albert Astals Cid (aacid) wrote : | # |
> Text conflict in runtests.sh
> Text conflict in tests/plugins/
> Text conflict in tests/qmltests/
> 3 conflicts encountered.
Merged
- 1776. By Albert Astals Cid on 2015-05-19
-
We never need to ld library path the qmenumodel
| Albert Astals Cid (aacid) wrote : | # |
After speaking with Michael we decided to leave the possible refactoring for dbus-test-runner tests for later
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:1776
http://
Executed test runs:
UNSTABLE: http://
SUCCESS: http://
SUCCESS: http://
UNSTABLE: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Michael Zanetti (mzanetti) wrote : | # |
success!
looks good to me.
* Did you perform an exploratory manual test run of the code change and any related functionality?
just test changes, jenkins tested them
* Did CI run pass? If not, please explain why.
unrelated AP failures :/
* Did you make sure that the branch does not contain spurious tags?
yes
- 1777. By Albert Astals Cid on 2015-05-21
-
Merge unity8
- 1778. By Albert Astals Cid on 2015-05-27
-
Merge oxide_regressio
n_workaround - 1779. By Michael Zanetti on 2015-06-17
-
merge trunk
- 1780. By Michael Zanetti on 2015-06-17
-
fix bad merge
- 1781. By Michael Zanetti on 2015-06-18
-
merge with trunk

FAILED: Continuous integration, rev:1761 jenkins. qa.ubuntu. com/job/ unity8- ci/5643/ jenkins. qa.ubuntu. com/job/ generic- deb-autopilot- vivid-touch/ 2548/console jenkins. qa.ubuntu. com/job/ unity-phablet- qmluitests- vivid/806/ console jenkins. qa.ubuntu. com/job/ unity8- vivid-amd64- ci/808/ console jenkins. qa.ubuntu. com/job/ unity8- vivid-i386- ci/808/ console jenkins. qa.ubuntu. com/job/ generic- mediumtests- builder- vivid-armhf/ 2546/console
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/unity8- ci/5643/ rebuild
http://