Address review: document Windows test workaround

Erlend E. Aasland · Erlend E. Aasland · commit 8d1c13804ba7 · 2021-05-26T00:17:27.000+02:00
diff --git a/Lib/sqlite3/test/dbapi.py b/Lib/sqlite3/test/dbapi.py
@@ -181,6 +181,9 @@ def __fspath__(self):
         path = Path()
         with sqlite.connect(path) as cx:
             cx.execute('create table test(id integer)')
+
+        # bpo-42862: addCleanup fails to unlink TESTFN on Windows unless we
+        # explicitly close the connection and run the GC.
         cx.close()
         gc_collect()
 
@@ -193,6 +196,9 @@ def test_open_uri(self):
         with sqlite.connect('file:' + TESTFN + '?mode=ro', uri=True) as cx:
             with self.assertRaises(sqlite.OperationalError):
                 cx.execute('insert into test(id) values(1)')
+
+        # bpo-42862: addCleanup fails to unlink TESTFN on Windows unless we
+        # explicitly close the connection and run the GC.
         cx.close()
         gc_collect()
 
diff --git a/Lib/sqlite3/test/hooks.py b/Lib/sqlite3/test/hooks.py
@@ -261,6 +261,9 @@ def trace(statement):
         con2.execute("create table bar(x)")
         cur.execute(queries[1])
         self.assertEqual(traced_statements, queries)
+
+        # bpo-42862: addCleanup fails to unlink TESTFN on Windows unless we
+        # explicitly close the connections and run the GC.
         con1.close()
         con2.close()
         gc_collect()
