Merge lp:~3v1n0/unity8/fake-indicators-model into lp:unity8
| Status: | Merged |
|---|---|
| Approved by: | Lukáš Tinkl on 2016-09-08 |
| Approved revision: | 2449 |
| Merged at revision: | 2647 |
| Proposed branch: | lp:~3v1n0/unity8/fake-indicators-model |
| Merge into: | lp:unity8 |
| Prerequisite: | lp:~3v1n0/unity8/indicators-client-load-fix |
| Diff against target: |
1251 lines (+1141/-7) 4 files modified
tests/mocks/Unity/Indicators/CMakeLists.txt (+1/-0) tests/mocks/Unity/Indicators/IndicatorsModel.qml (+32/-7) tests/mocks/Unity/Indicators/fakeindicatorsmodeldata.js (+1106/-0) tests/mocks/Unity/Indicators/fakeplugin.cpp (+2/-0) |
| To merge this branch: | bzr merge lp:~3v1n0/unity8/fake-indicators-model |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Unity8 CI Bot | continuous-integration | Needs Fixing on 2016-09-09 | |
| Lukáš Tinkl (community) | Approve on 2016-09-08 | ||
| Michał Sawicz | Needs Fixing on 2016-08-10 | ||
| Albert Astals Cid (community) | Approve on 2016-08-08 | ||
| Michael Terry | 2016-06-15 | Needs Fixing on 2016-06-30 | |
| Nick Dedekind | 2016-06-15 | Pending | |
|
Review via email:
|
|||
Commit Message
Indicators, mocks: add fake indicators menuitem to populate mocks with different menu types
Also allow to use fake indicators model in IndicatorsClient
Description of the Change
* Are there any related MPs required for this MP to build/function as expected? Please list.
Nope
* Did you perform an exploratory manual test run of your code change and any related functionality?
Yes. make tryShell (or make tryMenuContent) is enough, or this will also allow to launch indicators-client with a fake data model when QML2_IMPORT_PATH points to the mocks path (i.e. builddir/
* If you changed the packaging (debian), did you subscribe the ubuntu-unity team to this MP?
No debian change here
* If you changed the UI, has there been a design review?
No UI change, although the indicators used in mocks will show some fake data.
| Unity8 CI Bot (unity8-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:2445
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
| Michael Terry (mterry) wrote : | # |
Looks like we're missing modelprinter.cpp and modelprinter.h
| Marco Trevisan (Treviño) (3v1n0) wrote : | # |
> Looks like we're missing modelprinter.cpp and modelprinter.h
Actually they are already provided by unity8. I'm just ensuring they are added to the fake lib here.
| Unity8 CI Bot (unity8-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:2447
https:/
Executed test runs:
SUCCESS: https:/
UNSTABLE: https:/
UNSTABLE: https:/
UNSTABLE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
| Albert Astals Cid (aacid) wrote : | # |
Text conflict in tests/mocks/
1 conflicts encountered.
| Marco Trevisan (Treviño) (3v1n0) wrote : | # |
And... Fixed :-)
| Unity8 CI Bot (unity8-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:2448
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
FAILURE: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
| Albert Astals Cid (aacid) 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.
Yes, except broken yakkety
| Unity8 CI Bot (unity8-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:2448
https:/
Executed test runs:
SUCCESS: https:/
UNSTABLE: https:/
UNSTABLE: https:/
UNSTABLE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/
| Michał Sawicz (saviq) wrote : | # |
Failing tests:
qmltestrunner.
qmltestrunner.
qmltestrunner.
qmltestrunner.
| Lukáš Tinkl (lukas-kde) wrote : | # |
Likely "conflicting" with https:/
- 2448. By Marco Trevisan (Treviño) on 2016-09-08
-
Merging with trunk
- 2449. By Marco Trevisan (Treviño) on 2016-09-08
-
fakeindicatorsm
odeldata. js: mark indicator-files always visible This fixes tests using mocks
| Marco Trevisan (Treviño) (3v1n0) wrote : | # |
Tests should run properly here... Let's wait CI confirmation.
| Lukáš Tinkl (lukas-kde) wrote : | # |
Yup, tests work fine now, thanks!
* 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.
Nope. not yet
| Unity8 CI Bot (unity8-ci-bot) wrote : | # |
FAILED: Continuous integration, rev:2449
https:/
Executed test runs:
SUCCESS: https:/
SUCCESS: https:/
FAILURE: https:/
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
SUCCESS: https:/
deb: https:/
Click here to trigger a rebuild:
https:/

FAILED: Continuous integration, rev:2445 /unity8- jenkins. ubuntu. com/job/ lp-unity8- ci/1489/ /unity8- jenkins. ubuntu. com/job/ build/1985/ console /unity8- jenkins. ubuntu. com/job/ build-0- fetch/2011/ console
https:/
Executed test runs:
FAILURE: https:/
FAILURE: https:/
Click here to trigger a rebuild: /unity8- jenkins. ubuntu. com/job/ lp-unity8- ci/1489/ rebuild
https:/