[change:tests] Switch to ChannelsLiveServerTestCase #904

Replaced StaticLiveServerTestCase with ChannelsLiveServerTestCase to resolve JS errors caused by failed websocket connections.

Fixes #904

Author: dee077

.github/workflows/ci.yml

@@ -29,15 +29,15 @@ jobs:
           - django~=4.2.0
           - django~=5.0.0
           - django~=5.1.0
-          - django~=5.2rc0
+          - django~=5.2.0
         exclude:
           # Django 5.0+ requires Python >=3.10
           - python-version: "3.9"
             django-version: django~=5.0.0
           - python-version: "3.9"
             django-version: django~=5.1.0
           - python-version: "3.9"
-            django-version: django~=5.2rc0
+            django-version: django~=5.2.0
           # Python 3.13 supported only in Django >=5.1.3
           - python-version: "3.13"
             django-version: django~=4.2.0
@@ -86,7 +86,7 @@ jobs:
     - name: Tests
       if: ${{ !cancelled() && steps.deps.conclusion == 'success' }}
       run: |
-        coverage run runtests.py --parallel
+        coverage run runtests.py
         coverage combine
         coverage xml
       env:

tests/openwisp2/asgi.py

@@ -0,0 +1,6 @@
+import os
+
+from django.core.asgi import get_asgi_application
+
+os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'openwisp2.settings')
+application = get_asgi_application()

tests/openwisp2/settings.py

@@ -77,7 +77,13 @@
 ]
 
 DATABASES = {
-    'default': {'ENGINE': 'django.db.backends.sqlite3', 'NAME': 'openwisp_utils.db'}
+    'default': {
+        'ENGINE': 'django.db.backends.sqlite3',
+        'NAME': 'openwisp_utils.db',
+        'TEST': {
+            'NAME': 'openwisp_utils_test.db',
+        },
+    }
 }
 TEST_RUNNER = 'openwisp_utils.metric_collection.tests.runner.MockRequestPostRunner'
 OPENWISP_ADMIN_SITE_CLASS = 'test_project.site.CustomAdminSite'
@@ -125,7 +131,7 @@
 CELERY_TASK_ALWAYS_EAGER = True
 CELERY_TASK_EAGER_PROPAGATES = True
 CELERY_BROKER_URL = 'memory://'
-
+ASGI_APPLICATION = 'openwisp2.asgi.application'
 # local settings must be imported before test runner otherwise they'll be ignored
 try:
     from local_settings import *  # noqa

tests/test_project/tests/test_selenium.py

@@ -1,15 +1,14 @@
-from django.contrib.staticfiles.testing import StaticLiveServerTestCase
 from django.urls import reverse
 from selenium.common.exceptions import JavascriptException, NoSuchElementException
 from selenium.webdriver import ActionChains
 from selenium.webdriver.common.by import By
 
 from ..models import Book, Operator, Shelf
 from . import CreateMixin
-from .utils import SeleniumTestMixin
+from .utils import ChannelsLiveServerTestCase, SeleniumTestMixin
 
 
-class TestMenu(SeleniumTestMixin, StaticLiveServerTestCase):
+class TestMenu(SeleniumTestMixin, ChannelsLiveServerTestCase):
     def tearDown(self):
         super().tearDown()
         # Clear local storage
@@ -358,7 +357,7 @@ def test_menu_on_narrow_screen(self):
         self.web_driver.set_window_size(1366, 768)
 
 
-class TestBasicFilter(SeleniumTestMixin, StaticLiveServerTestCase, CreateMixin):
+class TestBasicFilter(SeleniumTestMixin, ChannelsLiveServerTestCase, CreateMixin):
     shelf_model = Shelf
     book_model = Book
 
@@ -503,7 +502,7 @@ def test_book_filter(self):
             self.assertEqual(paginator.get_attribute('innerText'), '1 book')
 
 
-class TestInputFilters(SeleniumTestMixin, CreateMixin, StaticLiveServerTestCase):
+class TestInputFilters(SeleniumTestMixin, CreateMixin, ChannelsLiveServerTestCase):
     shelf_model = Shelf
 
     def test_input_filters(self):
@@ -593,7 +592,7 @@ def test_input_filters(self):
             self.find_element(By.XPATH, user_xpath)
 
 
-class TestDashboardCharts(SeleniumTestMixin, CreateMixin, StaticLiveServerTestCase):
+class TestDashboardCharts(SeleniumTestMixin, CreateMixin, ChannelsLiveServerTestCase):
     def setUp(self):
         super().setUp()
         self.web_driver.set_window_size(1600, 768)
@@ -611,7 +610,9 @@ def test_pie_chart_zero_annotation(self):
         self.assertEqual(annotation_text.text, '0')
 
 
-class TestAutocompleteFilter(SeleniumTestMixin, CreateMixin, StaticLiveServerTestCase):
+class TestAutocompleteFilter(
+    SeleniumTestMixin, CreateMixin, ChannelsLiveServerTestCase
+):
     shelf_model = Shelf
     book_model = Book
 
@@ -708,7 +709,7 @@ def test_autocomplete_owner_filter(self):
             )
 
 
-class TestFirefoxSeleniumHelpers(SeleniumTestMixin, StaticLiveServerTestCase):
+class TestFirefoxSeleniumHelpers(SeleniumTestMixin, ChannelsLiveServerTestCase):
     def setUp(self):
         super().setUp()
         self.login()
@@ -746,7 +747,7 @@ def test_find_elements(self):
         self.assertTrue(len(divs) > 1)
 
 
-class TestChromeSeleniumHelpers(SeleniumTestMixin, StaticLiveServerTestCase):
+class TestChromeSeleniumHelpers(SeleniumTestMixin, ChannelsLiveServerTestCase):
     browser = 'chrome'
 
     def test_get_browser_logs(self):

tests/test_project/tests/utils.py

@@ -2,6 +2,9 @@
 import os
 import uuid
 
+from channels.testing import (
+    ChannelsLiveServerTestCase as BaseChannelsLiveServerTestCase,
+)
 from django.contrib.auth import get_user_model
 from django.contrib.auth.models import AnonymousUser
 from openwisp_utils.tests import SeleniumTestMixin as BaseSeleniumTestMixin
@@ -194,6 +197,10 @@ def wait_for_dropdown(self, filter_class):
         self.wait_for_visibility(By.CSS_SELECTOR, f'.{filter_class} .filter-options')
 
 
+class ChannelsLiveServerTestCase(BaseChannelsLiveServerTestCase):
+    server_static = True
+
+
 class MockUser:
     def __init__(self, is_superuser=False):
         self.is_superuser = is_superuser