The AMP-based machinery is tested using a real reactor
Bug #1169102 reported by
Free Ekanayaka
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Landscape Client |
Fix Released
|
High
|
Free Ekanayaka |
Bug Description
We should rather mock the transport and make all tests synchronous.
Related branches
lp:~free.ekanayaka/landscape-client/testable-amp
- Alberto Donato (community): Approve
- Geoff Teale (community): Approve
-
Diff: 911 lines (+260/-166)4 files modifiedlandscape/broker/tests/test_amp.py (+2/-2)
landscape/lib/amp.py (+55/-34)
landscape/lib/tests/test_amp.py (+202/-129)
landscape/tests/test_amp.py (+1/-1)
tags: | removed: kanban |
Changed in landscape-client: | |
status: | In Progress → Fix Committed |
Changed in landscape-client: | |
status: | Fix Committed → Fix Released |
tags: |
added: verification-done removed: verification-needed |
To post a comment you must log in.
Hello Free, or anyone else affected,
Accepted landscape-client into lucid-proposed. The package will build now and be available at http:// launchpad. net/ubuntu/ +source/ landscape- client/ 13.07.3- 0ubuntu0. 10.04 in a few hours, and then in the -proposed repository.
Please help us by testing this new package. See https:/ /wiki.ubuntu. com/Testing/ EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.
If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification- failed. In either case, details of your testing will help us make a better decision.
Further information regarding the verification process can be found at https:/ /wiki.ubuntu. com/QATeam/ PerformingSRUVe rification . Thank you in advance!