Merge lp:~mardy/signon-keyring-extension/mock-keyring into lp:signon-keyring-extension
Proposed by
Alberto Mardegan
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | David King | ||||
Approved revision: | 14 | ||||
Merged at revision: | 14 | ||||
Proposed branch: | lp:~mardy/signon-keyring-extension/mock-keyring | ||||
Merge into: | lp:signon-keyring-extension | ||||
Diff against target: |
417 lines (+345/-18) 4 files modified
tests/keyring-test.cpp (+3/-17) tests/mocked-gnome-keyring.c (+304/-0) tests/mocked-gnome-keyring.h (+36/-0) tests/tests.pro (+2/-1) |
||||
To merge this branch: | bzr merge lp:~mardy/signon-keyring-extension/mock-keyring | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
David King (community) | Approve | ||
jenkins (community) | continuous-integration | Approve | |
Review via email: mp+119524@code.launchpad.net |
Description of the change
Don't use the real GNOME keyring when running tests
Mock the keyring APIs which we are using and act on a fake in-memory keyring.
To post a comment you must log in.
PASSED: Continuous integration, rev:14 s-jenkins: 8080/job/ online- accounts- keyring- extension- ci/2/ s-jenkins: 8080/job/ online- accounts- keyring- extension- ci/./label= precise- clean-i386/ 2/console
http://
Executed test runs:
SUCCESS: http://