From acfb378245fa4a4b2c7d90b106244d28d33caf6c Mon Sep 17 00:00:00 2001 From: Mateusz Mandera Date: Fri, 27 Mar 2020 16:39:59 +0100 Subject: [PATCH] tests: Change some assertTrue usage where assertEqual was meant. These were typos, causing the asserts to pass always, because the syntax was meant for assertEqual. --- zerver/tests/test_queue_worker.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/zerver/tests/test_queue_worker.py b/zerver/tests/test_queue_worker.py index f22112c4f5..56f2816323 100644 --- a/zerver/tests/test_queue_worker.py +++ b/zerver/tests/test_queue_worker.py @@ -105,8 +105,8 @@ class WorkerTest(ZulipTestCase): user_profile = user.id, client = get_client('ios') ) - self.assertTrue(len(activity_records), 1) - self.assertTrue(activity_records[0].count, 1) + self.assertEqual(len(activity_records), 1) + self.assertEqual(activity_records[0].count, 1) # Now process the event a second time and confirm count goes # up to 2. Ideally, we'd use an event with a slightly never @@ -124,8 +124,8 @@ class WorkerTest(ZulipTestCase): user_profile = user.id, client = get_client('ios') ) - self.assertTrue(len(activity_records), 1) - self.assertTrue(activity_records[0].count, 2) + self.assertEqual(len(activity_records), 1) + self.assertEqual(activity_records[0].count, 2) def test_missed_message_worker(self) -> None: cordelia = self.example_user('cordelia')