From a6c55afe8c82ead8deb32f90c9324022fd422324 Mon Sep 17 00:00:00 2001
From: Chris Amico <eyeseast@gmail.com>
Date: Mon, 21 Jun 2021 11:57:38 -0400
Subject: [PATCH] Ensure db.path is a string before trying to insert into
 internal database (#1370)

Thanks, @eyeseast
---
 datasette/app.py  |  2 +-
 tests/test_api.py | 14 ++++++++++++++
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/datasette/app.py b/datasette/app.py
index fc5b7d9d..ce59ef54 100644
--- a/datasette/app.py
+++ b/datasette/app.py
@@ -354,7 +354,7 @@ class Datasette:
                 INSERT OR REPLACE INTO databases (database_name, path, is_memory, schema_version)
                 VALUES (?, ?, ?, ?)
             """,
-                [database_name, db.path, db.is_memory, schema_version],
+                [database_name, str(db.path), db.is_memory, schema_version],
                 block=True,
             )
             await populate_schema_tables(internal_db, db)
diff --git a/tests/test_api.py b/tests/test_api.py
index e5e609d6..2d891aae 100644
--- a/tests/test_api.py
+++ b/tests/test_api.py
@@ -25,6 +25,7 @@ from .fixtures import (  # noqa
     METADATA,
 )
 import json
+import pathlib
 import pytest
 import sys
 import urllib
@@ -2123,3 +2124,16 @@ def test_col_nocol_errors(app_client, path, expected_error):
     response = app_client.get(path)
     assert response.status == 400
     assert response.json["error"] == expected_error
+
+
+@pytest.mark.asyncio
+async def test_db_path(app_client):
+    db = app_client.ds.get_database()
+    path = pathlib.Path(db.path)
+
+    assert path.exists()
+
+    datasette = Datasette([path])
+
+    # this will break with a path
+    await datasette.refresh_schemas()