5 files changed, 42 insertions, 5 deletions
diff --git a/tests/playwright/conftest.py b/tests/playwright/conftest.py
index 4e7f5a4..adf5ef3 100644
--- a/tests/playwright/conftest.py
+++ b/tests/playwright/conftest.py
@@ -7,10 +7,12 @@ from wsgiref import simple_server
 from pyramid.authentication import AuthTktCookieHelper
 from pyramid.testing import DummyRequest
 
+import fietsboek.config
 from testutils import load_gpx_asset
 from fietsboek import models, util, actions
 from fietsboek.models.track import Visibility, TrackType
 from fietsboek.config import Config
+from fietsboek.views.tileproxy import TileRequester
 
 import pytest
 
@@ -55,7 +57,11 @@ def dbaccess(app):
     through and the running WSGI app cannot read them.
     """
     session_factory = app.registry["dbsession_factory"]
-    return session_factory()
+    factory = session_factory()
+
+    yield factory
+
+    factory.close()
 
 
 class Helper:
@@ -112,11 +118,14 @@ class Helper:
         """
         if user is None:
             user = self.john_doe()
+        config = fietsboek.config.parse(self.app_settings)
         with self.dbaccess:
             user = self.dbaccess.merge(user)
             track = actions.add_track(
                 self.dbaccess,
                 self.data_manager,
+                TileRequester(None),
+                config.public_tile_layers()[0],
                 owner=user,
                 title="Another awesome track",
                 visibility=Visibility.PRIVATE,
diff --git a/tests/playwright/test_basic.py b/tests/playwright/test_basic.py
index 231962e..3ae0f58 100644
--- a/tests/playwright/test_basic.py
+++ b/tests/playwright/test_basic.py
@@ -101,6 +101,33 @@ def test_edit(page: Page, playwright_helper, dbaccess):
     assert track.description == "Not so descriptive anymore"
 
 
+def test_edit_change_gpx(page: Page, playwright_helper, tmp_path, dbaccess):
+    playwright_helper.login()
+    track_id = playwright_helper.add_track().id
+
+    track = dbaccess.execute(select(models.Track).filter_by(id=track_id)).scalar_one()
+    old_cache = track.cache.length, track.cache.uphill, track.cache.downhill
+
+    page.goto(f"/track/{track_id}")
+    page.locator(".btn", has_text="Edit").click()
+
+    gpx_data = load_gpx_asset("Synthetic_BRouter_1.gpx.gz")
+    gpx_path = tmp_path / "NewGPX.gpx"
+    with open(gpx_path, "wb") as gpx_fobj:
+        gpx_fobj.write(gpx_data)
+
+    page.get_by_label("New file for this track").set_input_files(gpx_path)
+    page.locator(".btn", has_text="Save").click()
+
+    track = dbaccess.execute(select(models.Track).filter_by(id=track_id)).scalar_one()
+    new_cache = track.cache
+    dbaccess.refresh(new_cache)
+
+    assert old_cache[0] != new_cache.length
+    assert old_cache[1] != new_cache.uphill
+    assert old_cache[2] != new_cache.downhill
+
+
 def test_browse(page: Page, playwright_helper, dbaccess):
     playwright_helper.login()
     track = playwright_helper.add_track()
diff --git a/tests/playwright/test_profiles.py b/tests/playwright/test_profiles.py
index 7e5fb3c..ffbaab0 100644
--- a/tests/playwright/test_profiles.py
+++ b/tests/playwright/test_profiles.py
@@ -5,7 +5,7 @@ def test_forbidden(page: Page, playwright_helper):
     john = playwright_helper.john_doe()
 
     with page.expect_response(lambda resp: resp.status == 403):
-        page.goto(f"/user/{john.id}")
+        page.goto(f"/user/{john.id}/")
 
 
 def test_profile(page: Page, playwright_helper):
diff --git a/tests/playwright/test_share.py b/tests/playwright/test_share.py
index de288a0..dcba899 100644
--- a/tests/playwright/test_share.py
+++ b/tests/playwright/test_share.py
@@ -29,7 +29,8 @@ def test_view_wrong_link(page: Page, playwright_helper, dbaccess):
     with page.expect_response(lambda resp: resp.status == 403):
         page.goto(f"/track/{track.id}?secret=foobar")
 
-    assert "Forbidden" in page.content()
+    assert "No entry" in page.content()
+    assert "not allowed to access" in page.content()
 
 
 def test_change_link(page: Page, playwright_helper, dbaccess):
diff --git a/tests/playwright/test_tileproxy.py b/tests/playwright/test_tileproxy.py
index d4d3389..2a2bdc0 100644
--- a/tests/playwright/test_tileproxy.py
+++ b/tests/playwright/test_tileproxy.py
@@ -14,7 +14,7 @@ def test_tileproxy(page: Page, playwright_helper, caplog):
 
     # If we're too fast, the log entry might not be there yet, wait 2 more
     # seconds
-    if "Skipping tile proxy request for testing URL" not in caplog.messages:
+    if "Skipping tile request for testing URL" not in caplog.messages:
         time.sleep(2)
 
-    assert "Skipping tile proxy request for testing URL" in caplog.messages
+    assert "Skipping tile request for testing URL" in caplog.messages