Merge lp:~nick-dedekind/qtmir/qtmir-test-build into lp:qtmir
| Status: | Merged |
|---|---|
| Approved by: | Gerry Boland on 2015-12-16 |
| Approved revision: | 393 |
| Merged at revision: | 441 |
| Proposed branch: | lp:~nick-dedekind/qtmir/qtmir-test-build |
| Merge into: | lp:qtmir |
| Diff against target: |
3404 lines (+1807/-752) 57 files modified
CMakeLists.txt (+1/-1) tests/CMakeLists.txt (+1/-0) tests/framework/CMakeLists.txt (+45/-0) tests/framework/fake_desktopfilereader.cpp (+68/-0) tests/framework/fake_desktopfilereader.h (+22/-21) tests/framework/fake_mirsurface.cpp (+212/-0) tests/framework/fake_mirsurface.h (+48/-138) tests/framework/fake_session.cpp (+119/-0) tests/framework/fake_session.h (+35/-62) tests/framework/mock_application_controller.cpp (+129/-0) tests/framework/mock_application_controller.h (+22/-99) tests/framework/mock_desktop_file_reader.cpp (+117/-0) tests/framework/mock_desktop_file_reader.h (+20/-85) tests/framework/mock_display.cpp (+35/-0) tests/framework/mock_display.h (+6/-2) tests/framework/mock_display_configuration.cpp (+25/-0) tests/framework/mock_display_configuration.h (+4/-0) tests/framework/mock_gl_display_buffer.cpp (+30/-0) tests/framework/mock_gl_display_buffer.h (+3/-9) tests/framework/mock_main_loop.cpp (+28/-0) tests/framework/mock_main_loop.h (+5/-8) tests/framework/mock_mir_session.cpp (+63/-0) tests/framework/mock_mir_session.h (+13/-20) tests/framework/mock_proc_info.cpp (+35/-0) tests/framework/mock_proc_info.h (+8/-7) tests/framework/mock_prompt_session.cpp (+33/-0) tests/framework/mock_prompt_session.h (+3/-0) tests/framework/mock_prompt_session_manager.cpp (+33/-0) tests/framework/mock_prompt_session_manager.h (+3/-0) tests/framework/mock_renderable.cpp (+33/-0) tests/framework/mock_renderable.h (+2/-1) tests/framework/mock_session.cpp (+69/-0) tests/framework/mock_session.h (+7/-30) tests/framework/mock_settings.cpp (+40/-0) tests/framework/mock_settings.h (+7/-11) tests/framework/mock_shared_wakelock.cpp (+61/-0) tests/framework/mock_shared_wakelock.h (+8/-33) tests/framework/mock_surface.cpp (+45/-0) tests/framework/mock_surface.h (+5/-4) tests/framework/qtmir_test.cpp (+99/-1) tests/framework/qtmir_test.h (+15/-112) tests/framework/stub_input_channel.cpp (+52/-0) tests/framework/stub_input_channel.h (+8/-18) tests/framework/stub_scene_surface.cpp (+103/-0) tests/framework/stub_scene_surface.h (+41/-49) tests/mirserver/Screen/CMakeLists.txt (+1/-1) tests/mirserver/ScreenController/CMakeLists.txt (+4/-1) tests/modules/Application/CMakeLists.txt (+7/-6) tests/modules/ApplicationManager/CMakeLists.txt (+7/-5) tests/modules/DesktopFileReader/CMakeLists.txt (+0/-6) tests/modules/SessionManager/CMakeLists.txt (+6/-4) tests/modules/SessionManager/session_manager_test.cpp (+5/-5) tests/modules/SessionManager/session_test.cpp (+5/-5) tests/modules/SharedWakelock/CMakeLists.txt (+0/-1) tests/modules/SurfaceManager/CMakeLists.txt (+4/-4) tests/modules/SurfaceManager/mirsurfaceitem_test.cpp (+1/-1) tests/modules/TaskController/CMakeLists.txt (+6/-2) |
| To merge this branch: | bzr merge lp:~nick-dedekind/qtmir/qtmir-test-build |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| MichaĆ Sawicz | Abstain on 2016-02-11 | ||
| Gerry Boland | 2015-09-30 | Approve on 2015-12-16 | |
| PS Jenkins bot | continuous-integration | Needs Fixing on 2015-12-16 | |
|
Review via email:
|
|||
Commit Message
Moved test framework into a static library for quicker recompilation.
Description of the Change
Moved test framework into a static library for quicker recompilation.
- 382. By Nick Dedekind on 2015-09-30
-
Move test framework to library
- 383. By Nick Dedekind on 2015-10-01
-
undo new line removal
- 384. By Nick Dedekind on 2015-10-01
-
undo new line removal
| Gerry Boland (gerboland) wrote : | # |
Code style nit:
+void qtmir::
should be changed to be:
+void qtmir::
+{
These slipped in sometime before this, but would you mind fixing them please?
The exception is for 1 liners, those are fine as they are:
+Mir::Type qtmir::
=== modified file 'CMakeLists.txt'
I think your editor dropped the newline at the file end, can you restore it please.
=== added file 'tests/
these includes are not necessary:
+ ${CMAKE_
+ ${LTTNG_
+ ${Qt5Gui_
=== added file 'tests/
=== added file 'tests/
Why not wrap everything in "namespace qtmir {" instead of using "qtmir::" everywhere?
=== added file 'tests/
=== added file 'tests/
so::many:
Are these taken from Mir? There's no copyright header.
This improves builds measurably, thank you so much. Phone OOM killer isn't striking while compiling appManTest any more!
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:384
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
- 385. By Nick Dedekind on 2015-10-02
-
namespace rework
- 386. By Nick Dedekind on 2015-10-02
-
moved qtmir_test impl to cpp
- 387. By Nick Dedekind on 2015-10-02
-
removed unnecessary const
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:387
http://
Executed test runs:
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 388. By Nick Dedekind on 2015-10-05
-
merge trunk
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:388
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Nick Dedekind (nick-dedekind) wrote : | # |
> Code style nit:
> +void qtmir::
> should be changed to be:
> +void qtmir::
> +{
> These slipped in sometime before this, but would you mind fixing them please?
>
> The exception is for 1 liners, those are fine as they are:
> +Mir::Type qtmir::
>
>
>
> === modified file 'CMakeLists.txt'
> I think your editor dropped the newline at the file end, can you restore it
> please.
>
>
> === added file 'tests/
> these includes are not necessary:
> + ${CMAKE_
> + ${LTTNG_
> + ${Qt5Gui_
>
>
> === added file 'tests/
> === added file 'tests/
> Why not wrap everything in "namespace qtmir {" instead of using "qtmir::"
> everywhere?
>
>
> === added file 'tests/
> === added file 'tests/
> so::many:
> Are these taken from Mir? There's no copyright header.
>
> This improves builds measurably, thank you so much. Phone OOM killer isn't
> striking while compiling appManTest any more!
Think I got everything.
| Gerry Boland (gerboland) wrote : | # |
Thanks for cleaning up the namespaces in the tests. qtmir everywhere is simpler.
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
=== added file 'tests/
all missing licence headers.
That's my only complaint tho. But I do anticipate merge conflicts appearing when the multimonitor silo lands. Hopefully not many
- 389. By Nick Dedekind on 2015-10-19
-
added copywrite
| Nick Dedekind (nick-dedekind) wrote : | # |
> Thanks for cleaning up the namespaces in the tests. qtmir everywhere is
> simpler.
>
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> === added file 'tests/
> all missing licence headers.
>
> That's my only complaint tho. But I do anticipate merge conflicts appearing
> when the multimonitor silo lands. Hopefully not many
Fixed
| PS Jenkins bot (ps-jenkins) wrote : | # |
PASSED: Continuous integration, rev:389
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild:
http://
| Gerry Boland (gerboland) wrote : | # |
Conflicts with trunk :(
- 390. By Nick Dedekind on 2015-11-11
-
merged with trunk
| Nick Dedekind (nick-dedekind) wrote : | # |
> Conflicts with trunk :(
Merged.
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:390
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 391. By Nick Dedekind on 2015-11-20
-
merged with trunk
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:391
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 392. By Nick Dedekind on 2015-12-15
-
merged with trunk
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:392
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
| Gerry Boland (gerboland) wrote : | # |
Am getting cmake warnings: http://
- 393. By Nick Dedekind on 2015-12-16
-
removed dep loop
| PS Jenkins bot (ps-jenkins) wrote : | # |
FAILED: Continuous integration, rev:393
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
FAILURE: http://
FAILURE: http://
FAILURE: http://
Click here to trigger a rebuild:
http://
- 394. By Nick Dedekind on 2016-02-01
-
merged with trunk
- 395. By Nick Dedekind on 2016-02-10
-
merged with trunk
| MichaĆ Sawicz (saviq) wrote : | # |
/«BUILDDIR»
/«BUILDDIR»

PASSED: Continuous integration, rev:382 jenkins. qa.ubuntu. com/job/ qtmir-ci/ 463/ jenkins. qa.ubuntu. com/job/ qtmir-vivid- amd64-ci/ 159 jenkins. qa.ubuntu. com/job/ qtmir-vivid- armhf-ci/ 159 jenkins. qa.ubuntu. com/job/ qtmir-vivid- armhf-ci/ 159/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ qtmir-vivid- i386-ci/ 41 jenkins. qa.ubuntu. com/job/ qtmir-wily- amd64-ci/ 196 jenkins. qa.ubuntu. com/job/ qtmir-wily- armhf-ci/ 196 jenkins. qa.ubuntu. com/job/ qtmir-wily- armhf-ci/ 196/artifact/ work/output/ *zip*/output. zip jenkins. qa.ubuntu. com/job/ qtmir-wily- i386-ci/ 41
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
SUCCESS: http://
SUCCESS: http://
deb: http://
SUCCESS: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/qtmir- ci/463/ rebuild
http://