fix some UI tests and update changelog

Author: nuest

CHANGELOG.md

@@ -32,6 +32,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   - Support for complex geometries (points, lines, polygons, multigeometry)
   - Export extreme points (northernmost, southernmost, easternmost, westernmost) and geometric center
   - Configurable via `WIKIBASE_*` environment variables
+- **Global regions layer** - Interactive map overlay showing continent and ocean boundaries:
+  - Toggle-able layer control to show/hide global regions on the main map
+  - Simplified ocean geometries for efficient rendering
+  - Color-coded regions (brown for continents, blue for oceans)
+  - Dashed line styling for clear visual distinction
+  - Click to view region details and navigate to regional feeds
+  - Integrated with feed landing pages showing region outlines
 - **Geocoding/gazetteer search** - Map search functionality allowing users to search for locations by name:
   - Nominatim geocoder integration (default)
   - Optional GeoNames support (requires username configuration)
@@ -97,6 +104,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - **Unified admin control panel** - Consolidated admin status display, publish/unpublish buttons, provenance information, and "Edit in Admin" link into single highlighted box at top of work landing page. Provenance is collapsible.
 - **Improved text wrapping** - Page titles and abstract text now properly wrap on narrow windows instead of overflowing.
 - **Unified URL structure** - Changed ID-based URLs from `/publication/<id>/` to `/work/<id>/` for consistency with DOI-based URLs.
+- **Consolidated work identifier logic** - Centralized logic for determining work identifiers (DOI or internal ID) in a `get_identifier()` method on the `Work` model:
+  - Ensures consistent identifier usage across permalinks, sitemaps, and API responses
+  - Prioritizes DOI when available, falls back to internal ID
+  - Reduces code duplication across views and serializers
 - **Refactored views_geometry.py** - Eliminated code duplication by making DOI-based functions wrap ID-based functions. Reduced from 375 to 240 lines (~36% reduction).
 - **Renamed "Locate" to "Contribute"** - URL, page title, and navigation updated for clarity about crowdsourcing purpose.
 - **Contribute page layout refactored** - Fixed text overflow issues with proper CSS containment strategy.

tests-ui/test_feeds_and_work_landing.py

@@ -43,7 +43,7 @@ def setUpClass(cls):
     def test_europe_feed_page_loads(self):
         """Test that the Europe feed page loads and displays works."""
         try:
-            start_chrome(f'{self.live_server_url}/feeds/continent/europe/', headless=False)
+            start_chrome(f'{self.live_server_url}/feeds/continent/europe/', headless=True)
             driver = get_driver()
 
             # Check page loaded
@@ -94,39 +94,38 @@ def test_work_landing_page_from_fixture(self):
         try:
             # Use the work's identifier (DOI or ID)
             identifier = work.get_identifier()
-            start_chrome(f'{self.live_server_url}/work/{identifier}/', headless=False)
+            start_chrome(f'{self.live_server_url}/work/{identifier}/', headless=True)
             driver = get_driver()
 
             # Check page loaded
             self.assertIn("OPTIMAP", driver.title)
 
-            # Check for work title
+            # Check for work details on the page
+            page_text = driver.page_source
             self.assertTrue(
-                Text(work.title).exists() and work.title in driver.page_source,
-                f"Page should contain work title: {work.title}"
+                work.title in page_text,
+                "Work landing page shows the work title"
+            )
+            self.assertTrue(
+                work.abstract in page_text,
+                "Work landing page shows the work abstract"
+            )
+            self.assertTrue(
+                work.doi in page_text and f"href=\"https://doi.org/{work.doi}\"",
+                "Work landing page shows the work doi as a link"
             )
-
-            page_text = driver.page_source.lower()
             self.assertTrue(
-                'work' in page_text and work.title in page_text,
-                "Page should be a work landing page showing the work title"
+                work.source.name in page_text and f"href=\"{work.source.homepage_url}\"" in page_text,
+                "Work landing page shows the work source as a link"
+            )
+            self.assertTrue(
+                f"href=\"{work.openalex_id}\"" in page_text,
+                "Work landing page shows the OpenAlex ID as a link"
             )
 
             leaflet_paths = find_all(S('path.leaflet-interactive'))
-            self.assertEqual(len(leaflet_paths), 3) # has geometries on the map from test_data_optimap.json
-            for path in leaflet_paths:
-                self.assertEqual(path.web_element.get_attribute('stroke'), '#158F9B')
-
-            click(leaflet_paths[0])
-        
-            wait_until(lambda: Text('View work details').exists())
-
-            # the last paper is the first in the paths
-            self.assertIn('Visit work', S('div.leaflet-popup-content').web_element.text)
-            self.assertIn('Paper 3', S('div.leaflet-popup-content').web_element.text)
-            self.assertIn('OPTIMAP Test Journal', S('div.leaflet-popup-content').web_element.text)
-            self.assertIn('better? Dresden!', S('div.leaflet-popup-content').web_element.text)
-
+            self.assertEqual(len(leaflet_paths), 1) # has one on the map
+ 
             # Take screenshot
             screenshot_path = os.path.join(os.getcwd(), 'tests-ui', 'screenshots', 'work_landing.png')
             driver.save_screenshot(screenshot_path)