Merge lp:~ted/pay-service/verify-after-purchase into lp:pay-service/14.10
Proposed by
Ted Gould
Status: | Merged |
---|---|
Approved by: | dobey |
Approved revision: | 25 |
Merged at revision: | 21 |
Proposed branch: | lp:~ted/pay-service/verify-after-purchase |
Merge into: | lp:pay-service/14.10 |
Diff against target: |
172 lines (+53/-27) 6 files modified
libpay/pay-package.h (+2/-1) service/item-memory.cpp (+9/-15) service/verification-curl.cpp (+15/-8) tests/item-memory-tests.cpp (+15/-3) tests/verification-curl-tests.cpp (+9/-0) tests/verification-test.h (+3/-0) |
To merge this branch: | bzr merge lp:~ted/pay-service/verify-after-purchase |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
dobey (community) | Approve | ||
PS Jenkins bot (community) | continuous-integration | Approve | |
Review via email: mp+226728@code.launchpad.net |
Commit message
Verify purchase status after running Purchase UI
Description of the change
This makes it so that we always verify the status after each time we run the UI so that we don't have to trust its return code. It also fixes some bugs found with multiple verify calls.
To post a comment you must log in.
PASSED: Continuous integration, rev:25 jenkins. qa.ubuntu. com/job/ pay-service- ci/32/ jenkins. qa.ubuntu. com/job/ pay-service- utopic- amd64-ci/ 33 jenkins. qa.ubuntu. com/job/ pay-service- utopic- armhf-ci/ 33 jenkins. qa.ubuntu. com/job/ pay-service- utopic- armhf-ci/ 33/artifact/ work/output/ *zip*/output. zip
http://
Executed test runs:
SUCCESS: http://
SUCCESS: http://
deb: http://
Click here to trigger a rebuild: s-jenkins. ubuntu- ci:8080/ job/pay- service- ci/32/rebuild
http://