pulp/pulp_rpm — pull request #143 by rbarlow (2013-03-12 21:14:50+00:00)
Title: Removing a non-deterministic unit test. We have a Rally task to reimplim...

Removing a non-deterministic unit test. We have a Rally task to reimpliment it more suitably when we rewrite the distributor.

This test is difficult to follow, and I believe more complicated than necessary. I think it would be better to assert cancellation behaviors by asserting appropriate calls are made by using Mock to wrap objects. Due to this test using threads of its own to test cancellation, it is not deterministic likely due to race conditions, and does occasionally fail. We will write a better test to ensure cancellation works when we rewrite the distributor.

We also benefit because this test was slow (there is a loop with a sleep(1) in it).

I also removed some stray print statements.