Merge lp://staging/~ken-vandine/content-hub/process_cpp into lp://staging/content-hub
Proposed by
Ken VanDine
Status: | Needs review |
---|---|
Proposed branch: | lp://staging/~ken-vandine/content-hub/process_cpp |
Merge into: | lp://staging/content-hub |
Prerequisite: | lp://staging/~ken-vandine/content-hub/dbus_test_runner |
Diff against target: |
961 lines (+221/-297) 12 files modified
CMakeLists.txt (+1/-0) debian/control (+1/-0) debian/rules (+1/-0) tests/acceptance-tests/CMakeLists.txt (+37/-17) tests/acceptance-tests/app_hub_communication_default_source.cpp (+28/-22) tests/acceptance-tests/app_hub_communication_handler.cpp (+39/-31) tests/acceptance-tests/app_hub_communication_known_sources.cpp (+30/-23) tests/acceptance-tests/app_hub_communication_paste.cpp (+23/-17) tests/acceptance-tests/app_hub_communication_stores.cpp (+27/-21) tests/acceptance-tests/app_hub_communication_transfer.cpp (+34/-41) tests/cross_process_sync.h (+0/-64) tests/fork_and_run.h (+0/-61) |
To merge this branch: | bzr merge lp://staging/~ken-vandine/content-hub/process_cpp |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
system-apps-ci-bot | continuous-integration | Needs Fixing | |
Michael Sheldon (community) | Approve | ||
PS Jenkins bot | continuous-integration | Pending | |
Review via email: mp+296935@code.staging.launchpad.net |
Commit message
Port tests to use process-cpp for forkandrun
Description of the change
Port tests to use process-cpp for forkandrun
To post a comment you must log in.
Unmerged revisions
- 298. By Ken VanDine
-
Ported paste test and updated the rest of tests for consistency
- 297. By Ken VanDine
-
Fixed last of the merge issues from the prereq. Commented out the paste test until it can be updated to work with process-cpp
- 296. By Ken VanDine
-
merged prereq
- 295. By Ken VanDine
-
Cleanup
- 294. By Ken VanDine
-
The service side of forkandrun shouldn't ever fail
- 293. By Ken VanDine
-
The service side of forkandrun shouldn't ever fail
- 292. By Ken VanDine
-
Run tests in serial to work around timeouts we get due to load in CI
- 291. By Ken VanDine
-
get ctest output on failure
- 290. By Ken VanDine
-
bumped the dbus-test-runner job timeout
- 289. By Ken VanDine
-
Bumped timeout in tests to 120 seconds
Couple of questions in comments, but otherwise this looks good to me :)