try to fix UI tests

Author: nuest

tests-ui/test_accountdeletion.py

@@ -12,7 +12,7 @@
 
 User = get_user_model()
 
-class AccountDeletionUITest(unittest.TestCase):
+class AccountDeletionUITest(unittest.StaticLiveServerTestCase):
     def setUp(self):
         """Set up the test user and start browser"""
         self.email = "[email protected]"
@@ -36,10 +36,10 @@ def test_delete_account(self):
         write(self.email,  into='email')
         click(S('button[type="submit"]'))
 
-        go_to(f"http://localhost:8000/login/{self.token}")  
+        go_to(ff"{self.live_server_url}/login/{self.token}")  
         sleep(3)
 
-        go_to("http://localhost:8000/usersettings/")
+        go_to(f"{self.live_server_url}/usersettings/")
         sleep(2)
 
         click("Delete account")
@@ -48,7 +48,7 @@ def test_delete_account(self):
         click("Delete")
         sleep(3)
 
-        go_to(f"http://localhost:8000/confirm-delete/{self.delete_token}")
+        go_to(ff"{self.live_server_url}/confirm-delete/{self.delete_token}")
         sleep(3)
 
         click("Permanently Delete Account")

tests-ui/test_admin_block_user.py

@@ -14,7 +14,7 @@
 
 User = get_user_model()
 
-class AdminBlockUserTests(TestCase):
+class AdminBlockUserTests(StaticLiveServerTestCase):
     def setUp(self):
         """Set up a superuser, test user, and start the browser before each test."""
         self.superuser, _ = User.objects.get_or_create(
@@ -35,7 +35,7 @@ def setUp(self):
 
         #self.kill_existing_firefox_processes()
         try:
-            self.browser = start_chrome("http://localhost:8000/admin/", headless=True)
+            self.browser = start_chrome(f"{self.live_server_url}/admin/", headless=True)
         except Exception as e:
             print(f"Error starting browser: {e}")
             self.browser = None
@@ -65,7 +65,7 @@ def block_user_by_action(self, action_name):
             write("admin123", into="Password")
             click("Log in")
 
-            go_to("http://localhost:8000/admin/auth/user/")
+            go_to(f"{self.live_server_url}/admin/auth/user/")
 
             if not User.objects.filter(email="[email protected]").exists():
                 self.test_user = User.objects.create_user(

tests-ui/test_admin_content.py

@@ -7,6 +7,7 @@
 """
 
 from django.test import TestCase
+from django.contrib.staticfiles.testing import StaticLiveServerTestCase
 from django.contrib.auth import get_user_model
 from helium import (
     start_chrome,
@@ -22,13 +23,16 @@
 User = get_user_model()
 
 
-def get_work_from_api():
+def get_work_from_api(base_url):
     """
     Helper function to get a work (id, doi) from the API instead of database.
     Returns the first published work found, preferring works with DOI.
     Returns identifier field that can be used in URLs (either DOI or ID).
+
+    Args:
+        base_url: The base URL of the test server (e.g., self.live_server_url)
     """
-    response = requests.get('http://localhost:8000/api/v1/works/', timeout=5)
+    response = requests.get(f'{base_url}/api/v1/works/', timeout=5)
     if response.status_code == 200:
         data = response.json()
         if data.get('results') and len(data['results']) > 0:
@@ -141,11 +145,20 @@ def test_admin_panel_accessible_to_staff(self):
         self.assertContains(response, 'Site administration')
 
 
-class AdminButtonsBrowserTests(TestCase):
-    """Browser-based tests for admin button visibility."""
+class AdminButtonsBrowserTests(StaticLiveServerTestCase):
+    """Browser-based tests for admin button visibility.
+
+    Uses StaticLiveServerTestCase to automatically start a live test server
+    that serves both the application and static files.
+    """
 
     fixtures = ['test_data_optimap.json']
 
+    @classmethod
+    def setUpClass(cls):
+        """Set up class-level resources including live server."""
+        super().setUpClass()
+
     def setUp(self):
         """Set up test users for each test."""
         # Create admin user
@@ -159,13 +172,13 @@ def test_work_landing_page_anonymous_no_admin_buttons(self):
         """Test that work landing page doesn't show admin buttons to anonymous users."""
 
         # Get work from API instead of database
-        work_data = get_work_from_api()
+        work_data = get_work_from_api(self.live_server_url)
         if not work_data or not work_data.get('identifier'):
             self.skipTest('No works available via API')
 
         try:
             # Use the unified identifier (DOI or ID)
-            start_chrome(f'localhost:8000/work/{work_data["identifier"]}/', headless=True)
+            start_chrome(f'{self.live_server_url}/work/{work_data["identifier"]}/', headless=True)
             driver = get_driver()
 
             # Wait for page to load
@@ -181,7 +194,7 @@ def test_work_landing_page_anonymous_no_admin_buttons(self):
     def test_contribute_page_anonymous_no_publish_buttons(self):
         """Test that contribute page doesn't show publish buttons to anonymous users."""
         try:
-            start_chrome('localhost:8000/contribute/', headless=True)
+            start_chrome(f'{self.live_server_url}/contribute/', headless=True)
             driver = get_driver()
 
             # Wait for page to load

tests-ui/test_contribution_workflow.py

@@ -13,6 +13,7 @@
 """
 
 from django.test import TestCase
+from django.contrib.staticfiles.testing import StaticLiveServerTestCase
 from django.contrib.auth import get_user_model
 from helium import (
     start_chrome,
@@ -34,7 +35,7 @@
 User = get_user_model()
 
 
-class ContributionWorkflowE2ETest(TestCase):
+class ContributionWorkflowE2ETest(StaticLiveServerTestCase):
     """End-to-end test for complete contribution and publication workflow."""
 
     @classmethod
@@ -82,12 +83,12 @@ def test_complete_contribution_workflow(self):
         """
         try:
             # Start browser
-            start_chrome('localhost:8000/', headless=True)
+            start_chrome(f'{self.live_server_url}/', headless=True)
             driver = get_driver()
 
             # Step 1: Login as admin
             # Navigate to home page and open menu
-            go_to('localhost:8000/')
+            go_to(f'{self.live_server_url}/')
             time.sleep(1)
 
             # Find and click the menu dropdown
@@ -108,7 +109,7 @@ def test_complete_contribution_workflow(self):
 
             # Start browser again with authenticated session
             # Set up session cookie
-            start_chrome('localhost:8000/', headless=True)
+            start_chrome(f'{self.live_server_url}/', headless=True)
             driver = get_driver()
 
             # Add session cookie to browser
@@ -122,7 +123,7 @@ def test_complete_contribution_workflow(self):
                 })
 
             # Step 2: Navigate to contribute page
-            go_to('localhost:8000/contribute/')
+            go_to(f'{self.live_server_url}/contribute/')
             time.sleep(2)
 
             # Verify we're on the contribute page
@@ -177,7 +178,7 @@ def test_complete_contribution_workflow(self):
             self.test_work.save()
 
             # Step 6: Navigate to work landing page
-            work_url = f'localhost:8000/work/{self.test_work.id}/'
+            work_url = ff'{self.live_server_url}/work/{self.test_work.id}/'
             go_to(work_url)
             time.sleep(2)
 
@@ -209,7 +210,7 @@ def test_complete_contribution_workflow(self):
                 self.skipTest('Publish button not found - may need to adjust test')
 
             # Step 8: Verify work appears on main map
-            go_to('localhost:8000/')
+            go_to(f'{self.live_server_url}/')
             time.sleep(2)
 
             # Check if map has loaded

tests-ui/test_emailchange.py

@@ -12,7 +12,7 @@
 
 User = get_user_model()
 
-class EmailChangeUITest(unittest.TestCase):
+class EmailChangeUITest(unittest.StaticLiveServerTestCase):
     def setUp(self):
         """Set up the test user and start browser"""
         self.old_email = "[email protected]"
@@ -44,10 +44,10 @@ def test_email_change_process(self):
 
         sleep(1)
 
-        go_to(f"http://localhost:8000/login/{self.token}")  
+        go_to(ff"{self.live_server_url}/login/{self.token}")  
         sleep(3)
 
-        go_to("http://localhost:8000/usersettings/")
+        go_to(f"{self.live_server_url}/usersettings/")
         sleep(2)
 
         click("Change Email")
@@ -61,7 +61,7 @@ def test_email_change_process(self):
 
         if stored_data and "token" in stored_data:
             correct_token = stored_data["token"]
-            confirmation_url = f"http://localhost:8000/confirm-email/{correct_token}/{self.new_email}"
+            confirmation_url = ff"{self.live_server_url}/confirm-email/{correct_token}/{self.new_email}"
             go_to(confirmation_url)
             sleep(5)
         else: