try fixing tests by adding enqueue decorators

Johnetordoff · Mar 18, 2024 · 025bcf7 · 025bcf7
1 parent 1454e57
commit 025bcf7
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/osf/models/user.py b/osf/models/user.py
@@ -986,6 +986,7 @@ def deactivate_account(self):
         if isinstance(req, FlaskRequest):
             logout()
         remove_sessions_for_user(self)
+        signals.user_account_deactivated.send(self)
 
     def reactivate_account(self):
         """

diff --git a/osf_tests/test_notable_domains.py b/osf_tests/test_notable_domains.py
@@ -320,6 +320,7 @@ def ignored_notable_domain(self):
             note=NotableDomain.Note.IGNORED,
         )
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_spam_to_unknown_one_spam_domain(self, factory, spam_notable_domain_one, spam_notable_domain_two, unknown_notable_domain, ignored_notable_domain):
         obj_one = factory()
@@ -338,6 +339,7 @@ def test_from_spam_to_unknown_one_spam_domain(self, factory, spam_notable_domain
         assert obj_one.spam_status == SpamStatus.UNKNOWN
         assert len(obj_one.spam_data['domains']) == 0
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_spam_to_unknown_two_spam_domains(self, factory, spam_notable_domain_one, spam_notable_domain_two, unknown_notable_domain, ignored_notable_domain):
         obj_two = factory()
@@ -356,6 +358,7 @@ def test_from_spam_to_unknown_two_spam_domains(self, factory, spam_notable_domai
         assert obj_two.spam_status == SpamStatus.SPAM
         assert set(obj_two.spam_data['domains']) == set([self.spam_domain_two.netloc])
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_spam_to_unknown_marked_by_external(self, factory, spam_notable_domain_one, spam_notable_domain_two, unknown_notable_domain, ignored_notable_domain):
         obj_three = factory()
@@ -376,6 +379,7 @@ def test_from_spam_to_unknown_marked_by_external(self, factory, spam_notable_dom
         assert obj_three.spam_status == SpamStatus.SPAM
         assert len(obj_three.spam_data['domains']) == 0
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_spam_to_ignored_one_spam_domain(self, factory, spam_notable_domain_one, spam_notable_domain_two, unknown_notable_domain, ignored_notable_domain):
         obj_one = factory()
@@ -394,6 +398,7 @@ def test_from_spam_to_ignored_one_spam_domain(self, factory, spam_notable_domain
         assert obj_one.spam_status == SpamStatus.UNKNOWN
         assert len(obj_one.spam_data['domains']) == 0
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_spam_to_ignored_two_spam_domains(self, factory, spam_notable_domain_one, spam_notable_domain_two, unknown_notable_domain, ignored_notable_domain):
         obj_two = factory()
@@ -412,6 +417,7 @@ def test_from_spam_to_ignored_two_spam_domains(self, factory, spam_notable_domai
         assert obj_two.spam_status == SpamStatus.SPAM
         assert set(obj_two.spam_data['domains']) == set([self.spam_domain_two.netloc])
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_spam_to_ignored_makred_by_external(self, factory, spam_notable_domain_one, spam_notable_domain_two, unknown_notable_domain, ignored_notable_domain):
         obj_three = factory()
@@ -432,6 +438,7 @@ def test_from_spam_to_ignored_makred_by_external(self, factory, spam_notable_dom
         assert obj_three.spam_status == SpamStatus.SPAM
         assert len(obj_three.spam_data['domains']) == 0
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_unknown_to_spam_unknown_plus_ignored(self, factory, unknown_notable_domain, ignored_notable_domain):
         obj_one = factory()
@@ -450,6 +457,7 @@ def test_from_unknown_to_spam_unknown_plus_ignored(self, factory, unknown_notabl
         assert obj_one.spam_status == SpamStatus.SPAM
         assert set(obj_one.spam_data['domains']) == set([self.unknown_domain.netloc])
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_unknown_to_spam_unknown_only(self, factory, unknown_notable_domain, ignored_notable_domain):
         obj_two = factory()
@@ -468,6 +476,7 @@ def test_from_unknown_to_spam_unknown_only(self, factory, unknown_notable_domain
         assert obj_two.spam_status == SpamStatus.SPAM
         assert set(obj_two.spam_data['domains']) == set([self.unknown_domain.netloc])
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_ignored_to_spam_unknown_plus_ignored(self, factory, unknown_notable_domain, ignored_notable_domain):
         obj_one = factory()
@@ -486,6 +495,7 @@ def test_from_ignored_to_spam_unknown_plus_ignored(self, factory, unknown_notabl
         assert obj_one.spam_status == SpamStatus.SPAM
         assert set(obj_one.spam_data['domains']) == set([self.ignored_domain.netloc])
 
+    @pytest.mark.enable_enqueue_task
     @pytest.mark.parametrize('factory', [NodeFactory, CommentFactory, PreprintFactory, RegistrationFactory, UserFactory])
     def test_from_ignored_to_spam_ignored_only(self, factory, unknown_notable_domain, ignored_notable_domain):
         obj_two = factory()