Merge lp:~michihenning/storage-framework/upload-download-fixes into lp:storage-framework/devel
Proposed by
Michi Henning
Status: | Merged |
---|---|
Approved by: | James Henstridge |
Approved revision: | 80 |
Merged at revision: | 42 |
Proposed branch: | lp:~michihenning/storage-framework/upload-download-fixes |
Merge into: | lp:storage-framework/devel |
Prerequisite: | lp:~michihenning/storage-framework/more-coverage |
Diff against target: |
334 lines (+115/-14) 10 files modified
include/unity/storage/qt/client/Uploader.h (+8/-0) include/unity/storage/qt/client/internal/UploaderBase.h (+4/-1) include/unity/storage/qt/client/internal/remote_client/UploaderImpl.h (+0/-1) src/qt/client/Uploader.cpp (+5/-0) src/qt/client/internal/UploaderBase.cpp (+11/-2) src/qt/client/internal/local_client/UploaderImpl.cpp (+12/-5) src/qt/client/internal/remote_client/DownloaderImpl.cpp (+4/-2) src/qt/client/internal/remote_client/UploaderImpl.cpp (+4/-3) tests/local-client/local-client_test.cpp (+65/-0) tests/remote-client/remote-client_test.cpp (+2/-0) |
To merge this branch: | bzr merge lp:~michihenning/storage-framework/upload-download-fixes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
James Henstridge | Approve | ||
unity-api-1-bot | continuous-integration | Needs Fixing | |
Review via email: mp+301865@code.launchpad.net |
Commit message
Added size() accessor to Uploader. Fixed upload/download finalization for remote client.
Description of the change
Added size() accessor to Uploader.
Added tests for upload size mismatch and that calling finish_upload() more than once continues to return the same error future.
Fixed upload/download finalization for remote client.
To post a comment you must log in.
FAILED: Continuous integration, rev:80 /jenkins. canonical. com/unity- api-1/job/ lp-storage- framework- ci/46/ /jenkins. canonical. com/unity- api-1/job/ build/273/ console /jenkins. canonical. com/unity- api-1/job/ build-0- fetch/279 /jenkins. canonical. com/unity- api-1/job/ build-1- sourcepkg/ release= vivid+overlay/ 213 /jenkins. canonical. com/unity- api-1/job/ build-1- sourcepkg/ release= xenial+ overlay/ 213 /jenkins. canonical. com/unity- api-1/job/ build-1- sourcepkg/ release= yakkety/ 213 /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= vivid+overlay/ 142/console /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= xenial+ overlay/ 142/console /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=amd64, release= yakkety/ 142/console /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= vivid+overlay/ 142/console /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= xenial+ overlay/ 142/console /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=armhf, release= yakkety/ 142/console /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= vivid+overlay/ 142/console /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= xenial+ overlay/ 142/console /jenkins. canonical. com/unity- api-1/job/ build-2- binpkg/ arch=i386, release= yakkety/ 142/console
https:/
Executed test runs:
FAILURE: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
SUCCESS: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
FAILURE: https:/
Click here to trigger a rebuild: /jenkins. canonical. com/unity- api-1/job/ lp-storage- framework- ci/46/rebuild
https:/