Add check for temporary or purely in-memory path when incrementing newConnection

Author: 05nelsonm

sqliter-driver/src/nativeCommonMain/kotlin/co/touchlab/sqliter/native/NativeDatabaseManager.kt

@@ -97,7 +97,18 @@ class NativeDatabaseManager(private val path:String,
                     conn.close()
                     throw e
                 }
-                newConnection.increment()
+
+                // Temporary, or purely in-memory databases live only as long
+                // as the connection. Subsequent connections (even if open at
+                // the same time) are completely separate databases.
+                //
+                // If this is the case, do not increment newConnection so that
+                // this if block executes on every new connection (i.e. ever new
+                // ephemeral database).
+                when (path) {
+                    "", ":memory:" -> {}
+                    else -> newConnection.increment()
+                }
             }
 
             conn

sqliter-driver/src/nativeCommonTest/kotlin/co/touchlab/sqliter/DatabaseManagerTest.kt

@@ -86,6 +86,66 @@ class DatabaseManagerTest : BaseDatabaseTest(){
         assertEquals(1, updateCalled.value)
     }
 
+    @Test
+    fun createCalledEachNewConnectionWhenInMemory() {
+        var creationCount = 0
+
+        val conf = DatabaseConfiguration(
+            name = null,
+            inMemory = true,
+            version = 1, create = { db ->
+                creationCount++
+                db.withStatement(TWO_COL) {
+                    execute()
+                }
+            },
+        )
+
+        val mgr = createDatabaseManager(conf)
+        mgr.withConnection {
+            it.rawExecSql("INSERT INTO test(num, str)values(3,'abc')")
+            assertEquals(1, it.longForQuery("select count(*) from test"))
+
+            mgr.withConnection {
+                assertEquals(0, it.longForQuery("select count(*) from test"))
+            }
+        }
+
+        assertEquals(2, creationCount)
+    }
+
+    @Test
+    fun createCalledEachNewConnectionWhenTemporary() {
+        var creationCount = 0
+
+        val conf = DatabaseConfiguration(
+            name = "",
+            inMemory = true,
+            version = 1,
+            create = { db ->
+                creationCount++
+                db.withStatement(TWO_COL) {
+                    execute()
+                }
+            },
+            extendedConfig = DatabaseConfiguration.Extended(
+                basePath = ""
+            ),
+        )
+
+        val mgr = createDatabaseManager(conf)
+        mgr.withConnection { conn1 ->
+            conn1.rawExecSql("INSERT INTO test(num, str)values(3,'abc')")
+            assertEquals(1, conn1.longForQuery("select count(*) from test"))
+
+            mgr.withConnection { conn2 ->
+                assertEquals(0, conn2.longForQuery("select count(*) from test"))
+            }
+        }
+
+        assertEquals(2, creationCount)
+    }
+
     @Test
     fun downgradeNotAllowed(){
         val upgradeCalled = AtomicInt(0)