From 8f782b06f7e1fd71140790de324d34cae6460040 Mon Sep 17 00:00:00 2001
From: Rusty Russell <rusty@rustcorp.com.au>
Date: Tue, 29 Jun 2021 07:03:16 +0930
Subject: [PATCH] pytest: don't copy db to query it.

Should be safe to do in-place.

Signed-off-by: Rusty Russell <rusty@rustcorp.com.au>
---
 tests/db.py | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/tests/db.py b/tests/db.py
index bb153221d..905b8d423 100644
--- a/tests/db.py
+++ b/tests/db.py
@@ -6,7 +6,6 @@ import os
 import psycopg2
 import random
 import re
-import shutil
 import signal
 import sqlite3
 import string
@@ -24,13 +23,12 @@ class Sqlite3Db(object):
         return None
 
     def query(self, query):
-        orig = os.path.join(self.path)
-        copy = self.path + ".copy"
-        shutil.copyfile(orig, copy)
-        db = sqlite3.connect(copy)
+        db = sqlite3.connect(self.path)
 
         db.row_factory = sqlite3.Row
         c = db.cursor()
+        # Don't get upset by concurrent writes; wait for up to 5 seconds!
+        c.execute("PRAGMA busy_timeout = 5000")
         c.execute(query)
         rows = c.fetchall()