From 8790acfbf156ced2979c78352b559c0fbd776065 Mon Sep 17 00:00:00 2001
From: Simon Willison <swillison@gmail.com>
Date: Fri, 18 Oct 2019 12:22:23 -0700
Subject: [PATCH] Failing test for #597

---
 tests/fixtures.py |  7 +++++++
 tests/test_api.py | 18 ++++++++++++++++++
 2 files changed, 25 insertions(+)

diff --git a/tests/fixtures.py b/tests/fixtures.py
index dac28dc0..a4c32f36 100644
--- a/tests/fixtures.py
+++ b/tests/fixtures.py
@@ -178,6 +178,13 @@ def app_client_two_attached_databases():
     )
 
 
+@pytest.fixture(scope="session")
+def app_client_conflicting_database_names():
+    yield from make_app_client(
+        extra_databases={"foo.db": EXTRA_DATABASE_SQL, "foo-bar.db": EXTRA_DATABASE_SQL}
+    )
+
+
 @pytest.fixture(scope="session")
 def app_client_two_attached_databases_one_immutable():
     yield from make_app_client(
diff --git a/tests/test_api.py b/tests/test_api.py
index cc00b780..71bb5a5e 100644
--- a/tests/test_api.py
+++ b/tests/test_api.py
@@ -7,6 +7,7 @@ from .fixtures import (  # noqa
     app_client_larger_cache_size,
     app_client_returned_rows_matches_page_size,
     app_client_two_attached_databases_one_immutable,
+    app_client_conflicting_database_names,
     app_client_with_cors,
     app_client_with_dot,
     generate_compound_rows,
@@ -1652,3 +1653,20 @@ def test_cors(app_client_with_cors, path, status_code):
     response = app_client_with_cors.get(path)
     assert response.status == status_code
     assert "*" == response.headers["Access-Control-Allow-Origin"]
+
+
+def test_common_prefix_database_names(app_client_conflicting_database_names):
+    # https://github.com/simonw/datasette/issues/597
+    assert ["fixtures", "foo", "foo-bar"] == [
+        d["name"]
+        for d in json.loads(
+            app_client_conflicting_database_names.get("/-/databases.json").body
+        )
+    ]
+    for db_name, path in (("foo", "/foo.json"), ("foo-bar", "/foo-bar.json")):
+        assert (
+            db_name
+            == json.loads(app_client_conflicting_database_names.get(path).body)[
+                "database"
+            ]
+        )