From e73d2fff970a88c6b066de6fe222c04dbdff7aa8 Mon Sep 17 00:00:00 2001 From: Sahil Batra Date: Sun, 1 Aug 2021 13:42:26 +0530 Subject: [PATCH] tests: Remove separate event tests for "enable_drafts_synchronization". We do not require separate tests for checking events when changing "enable_drafts_synchronization" as we already do this in the display settings test because this setting is included in property_types. --- zerver/tests/test_events.py | 13 ------------- 1 file changed, 13 deletions(-) diff --git a/zerver/tests/test_events.py b/zerver/tests/test_events.py index 043ad03f2d..920627f5e6 100644 --- a/zerver/tests/test_events.py +++ b/zerver/tests/test_events.py @@ -2306,9 +2306,6 @@ class DraftActionTest(BaseAction): def do_enable_drafts_synchronization(self, user_profile: UserProfile) -> None: do_set_user_display_setting(user_profile, "enable_drafts_synchronization", True) - def do_disable_drafts_synchronization(self, user_profile: UserProfile) -> None: - do_set_user_display_setting(user_profile, "enable_drafts_synchronization", False) - def test_draft_create_event(self) -> None: self.do_enable_drafts_synchronization(self.user_profile) dummy_draft = { @@ -2347,13 +2344,3 @@ class DraftActionTest(BaseAction): draft_id = do_create_drafts([dummy_draft], self.user_profile)[0].id action = lambda: do_delete_draft(draft_id, self.user_profile) self.verify_action(action) - - def test_enable_syncing_drafts(self) -> None: - self.do_disable_drafts_synchronization(self.user_profile) - action = lambda: self.do_enable_drafts_synchronization(self.user_profile) - self.verify_action(action) - - def test_disable_syncing_drafts(self) -> None: - self.do_enable_drafts_synchronization(self.user_profile) - action = lambda: self.do_disable_drafts_synchronization(self.user_profile) - self.verify_action(action)