From 1c6c6d2e6897c1173ed6e209c8b7133688e75c58 Mon Sep 17 00:00:00 2001
From: Simon Willison <swillison@gmail.com>
Date: Tue, 17 Feb 2026 13:30:46 -0800
Subject: [PATCH] Fix test_write_wrapper_set_authorizer: use permissive
 callback instead of None

conn.set_authorizer(None) does not clear the authorizer - SQLite treats
None as an invalid callback. The denied state persists on the shared
write connection, causing subsequent non-deny test cases to fail.

Fixes test added in 8a315f3d.
---
 tests/test_write_wrapper.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/test_write_wrapper.py b/tests/test_write_wrapper.py
index cb320c06..55e0461e 100644
--- a/tests/test_write_wrapper.py
+++ b/tests/test_write_wrapper.py
@@ -445,7 +445,7 @@ async def test_write_wrapper_set_authorizer(datasette, actor, table, should_deny
                 try:
                     yield
                 finally:
-                    conn.set_authorizer(None)
+                    conn.set_authorizer(lambda *args: sqlite3.SQLITE_OK)
 
             return wrapper