tests for rabbit server and a corresponding fake
Bug #1302471 reported by
Vincent Ladeuil
This bug affects 1 person
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Ubuntu CI Engine |
Fix Released
|
High
|
Vincent Ladeuil | ||
Bug Description
The tests should cover only the features we use currently in amqp_utils and amqp_server.
The fake should allow tests to be written and run without a real server so they should be focused around queue handling and the behaviors we care about.
The tests against the rabbit server can be run against a real deployment, being able to use a local one would be a bonus but not requirement.
Related branches
lp:~vila/uci-engine/1302471-fake-rabbit
- Andy Doan (community): Needs Fixing
- PS Jenkins bot (community): Approve (continuous-integration)
-
Diff: 631 lines (+307/-61)15 files modifiedbranch-source-builder/bsbuilder/run_worker.py (+1/-1)
ci-utils/ci_utils/amqp_utils.py (+33/-16)
ci-utils/ci_utils/amqp_worker.py (+11/-5)
ci-utils/ci_utils/gpg.py (+1/-1)
ci-utils/ci_utils/sourcecode.py (+1/-1)
ci-utils/ci_utils/testing/fixtures.py (+19/-0)
ci-utils/ci_utils/tests/test_amqp_worker.py (+2/-2)
image-builder/imagebuilder/run_worker.py (+1/-1)
lander/bin/json_status_cgi.py (+1/-1)
lander/bin/lander_process_ticket.py (+1/-1)
test_runner/tstrun/run_worker.py (+3/-3)
test_runner/tstrun/testbed.py (+3/-2)
test_runner/tstrun/tests/test_worker.py (+47/-0)
tests/deployers.py (+34/-27)
tests/test_rabbit.py (+149/-0)
Revision history for this message
| Andy Doan (doanac) wrote Re: [Bug 1302471] [NEW] tests for rabbit server and a corresponding fake | #1 |
| Changed in uci-engine: | |
| status: | Triaged → In Progress |
| Changed in uci-engine: | |
| status: | In Progress → Fix Committed |
| Changed in uci-engine: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
On 04/04/2014 05:33 AM, Vincent Ladeuil wrote:
> Public bug reported:
>
> The tests should cover only the features we use currently in amqp_utils
> and amqp_server.
We already have a bit of this in-place:
test_amqp.py
test_