From: Daniel Axtens <dja@axtens.net>
Date: Wed, 18 Sep 2019 06:17:27 +0000 (+1000)
Subject: tests: make test suite pass with XML-RPC disabled
X-Git-Tag: v2.2.0-rc1~66
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=be559cbd5acff5dc82f5dbc2b79a452a506eee86;p=thirdparty%2Fpatchwork.git

tests: make test suite pass with XML-RPC disabled

Handy for development purposes.

Signed-off-by: Daniel Axtens <dja@axtens.net>
Reviewed-by: Stephen Finucane <stephen@that.guru>
---

diff --git a/patchwork/tests/test_about.py b/patchwork/tests/test_about.py
index 99892cb0..d8c35b9f 100644
--- a/patchwork/tests/test_about.py
+++ b/patchwork/tests/test_about.py
@@ -3,19 +3,31 @@
 #
 # SPDX-License-Identifier: GPL-2.0-or-later
 
+import unittest
+
+from django.conf import settings
 from django.test import TestCase
 from django.urls import reverse
 
 
 class AboutViewTest(TestCase):
 
-    def test_redirects(self):
-        for view in ['help', 'help-about', 'help-pwclient']:
-            requested_url = reverse(view)
-            redirect_url = reverse('about')
+    def _test_redirect(self, view):
+        requested_url = reverse(view)
+        redirect_url = reverse('about')
 
-            response = self.client.get(requested_url)
-            self.assertRedirects(response, redirect_url, 301)
+        response = self.client.get(requested_url)
+        self.assertRedirects(response, redirect_url, 301)
+
+    def test_redirects(self):
+        for view in ['help', 'help-about']:
+            self._test_redirect(view)
+
+    @unittest.skipUnless(settings.ENABLE_XMLRPC,
+                         'requires xmlrpc interface (use the ENABLE_XMLRPC '
+                         'setting)')
+    def test_redirects_xmlrpc(self):
+        self._test_redirect('help-pwclient')
 
     def test_xmlrpc(self):
         with self.settings(ENABLE_XMLRPC=False):
diff --git a/patchwork/tests/test_xmlrpc.py b/patchwork/tests/test_xmlrpc.py
index 461dfe63..79c6c848 100644
--- a/patchwork/tests/test_xmlrpc.py
+++ b/patchwork/tests/test_xmlrpc.py
@@ -42,6 +42,9 @@ class XMLRPCGenericTest(XMLRPCTest):
             self.rpc.patch_set(0, {})
 
 
+@unittest.skipUnless(settings.ENABLE_XMLRPC,
+                     'requires xmlrpc interface (use the ENABLE_XMLRPC '
+                     'setting)')
 class XMLRPCAuthenticatedTest(LiveServerTestCase):
 
     def setUp(self):