From fb2a69ec78b36e588490a5320d29aae5ad0945fb Mon Sep 17 00:00:00 2001 From: Steve Howell Date: Fri, 24 Jul 2020 17:09:06 +0000 Subject: [PATCH] outgoing webhook tests: Remove dead code. The first assignment is redundant, and the second assignment is ignored. --- zerver/tests/test_outgoing_webhook_system.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/zerver/tests/test_outgoing_webhook_system.py b/zerver/tests/test_outgoing_webhook_system.py index dc0ec4e4e7..c1b04f36cf 100644 --- a/zerver/tests/test_outgoing_webhook_system.py +++ b/zerver/tests/test_outgoing_webhook_system.py @@ -81,7 +81,6 @@ class DoRestCallTests(ZulipTestCase): def test_retry_request(self) -> None: response = ResponseMock(500) - self.mock_event['failed_tries'] = 3 with mock.patch('requests.request', return_value=response), mock.patch('logging.warning'): do_rest_call('', None, self.mock_event, self.service_handler) @@ -92,7 +91,6 @@ The webhook got a response with status code *500*.''') assert self.bot_user.bot_owner is not None self.assertEqual(bot_owner_notification.recipient_id, self.bot_user.bot_owner.id) - self.mock_event['failed_tries'] = 0 def test_fail_request(self) -> None: response = ResponseMock(400)