Test cases

Author: humitos

readthedocs/rtd_tests/tests/test_api.py

@@ -862,6 +862,22 @@ def setUp(self):
             },
         }
 
+    def test_webhook_skipped_project(self, trigger_build):
+        client = APIClient()
+        self.project.skip = True
+        self.project.save()
+
+        response = client.post(
+            '/api/v2/webhook/github/{0}/'.format(
+                self.project.slug,
+            ),
+            self.github_payload,
+            format='json',
+        )
+        self.assertDictEqual(response.data, {'detail': 'This project is currently disabled'})
+        self.assertEqual(response.status_code, status.HTTP_406_NOT_ACCEPTABLE)
+        self.assertFalse(trigger_build.called)
+
     def test_github_webhook_for_branches(self, trigger_build):
         """GitHub webhook API."""
         client = APIClient()

readthedocs/rtd_tests/tests/test_core_utils.py

@@ -18,6 +18,18 @@ def setUp(self):
         self.project = get(Project, container_time_limit=None)
         self.version = get(Version, project=self.project)
 
+    @mock.patch('readthedocs.projects.tasks.update_docs_task')
+    def test_trigger_skipped_project(self, update_docs_task):
+        self.project.skip = True
+        self.project.save()
+        result = trigger_build(
+            project=self.project,
+            version=self.version,
+        )
+        self.assertEqual(result, (None, None))
+        self.assertFalse(update_docs_task.signature.called)
+        self.assertFalse(update_docs_task.signature().apply_async.called)
+
     @mock.patch('readthedocs.projects.tasks.update_docs_task')
     def test_trigger_custom_queue(self, update_docs):
         """Use a custom queue when routing the task"""

readthedocs/rtd_tests/tests/test_project_querysets.py

@@ -30,6 +30,13 @@ def test_subproject_queryset_as_manager_gets_correct_class(self):
             'ManagerFromParentRelatedProjectQuerySetBase'
         )
 
+    def test_is_active(self):
+        project = fixture.get(Project, skip=False)
+        self.assertTrue(Project.objects.is_active(project))
+
+        project = fixture.get(Project, skip=True)
+        self.assertFalse(Project.objects.is_active(project))
+
 
 class FeatureQuerySetTests(TestCase):