From d92022bb832238a5b493b687f8e1ff46bd79b551 Mon Sep 17 00:00:00 2001 From: Vincent Driessen Date: Thu, 5 Sep 2013 00:24:06 +0200 Subject: [PATCH] Record new behaviour of Queue.all() explicitly in a test. --- tests/test_queue.py | 20 ++++++++++++++------ 1 file changed, 14 insertions(+), 6 deletions(-) diff --git a/tests/test_queue.py b/tests/test_queue.py index b161b0c..abd97da 100644 --- a/tests/test_queue.py +++ b/tests/test_queue.py @@ -2,6 +2,7 @@ from tests import RQTestCase from tests.fixtures import Number, div_by_zero, say_hello, some_calculation from rq import Queue, get_failed_queue from rq.job import Job, Status +from rq.worker import Worker from rq.exceptions import InvalidJobOperationError @@ -253,18 +254,18 @@ class TestQueue(RQTestCase): def test_all_queues(self): """All queues""" - q = Queue('first-queue') - r = Queue('second-queue') - s = Queue('third-queue') + q1 = Queue('first-queue') + q2 = Queue('second-queue') + q3 = Queue('third-queue') # Ensure a queue is added only once a job is enqueued self.assertEquals(len(Queue.all()), 0) - q.enqueue(say_hello) + q1.enqueue(say_hello) self.assertEquals(len(Queue.all()), 1) # Ensure this holds true for multiple queues - r.enqueue(say_hello) - s.enqueue(say_hello) + q2.enqueue(say_hello) + q3.enqueue(say_hello) names = [q.name for q in Queue.all()] self.assertEquals(len(Queue.all()), 3) @@ -273,6 +274,13 @@ class TestQueue(RQTestCase): self.assertTrue('second-queue' in names) self.assertTrue('third-queue' in names) + # Now empty two queues + w = Worker([q2, q3]) + w.work(burst=True) + + # Queue.all() should still report the empty queues + self.assertEquals(len(Queue.all()), 3) + def test_enqueue_waitlist(self): """Enqueueing a waitlist pushes all jobs in waitlist to queue""" q = Queue()