cleanup

Author: stevensJourney

Sources/PowerSync/PowerSyncDatabase.swift

@@ -12,12 +12,12 @@ public let DEFAULT_DB_FILENAME = "powersync.db"
 public func PowerSyncDatabase(
     schema: Schema,
     dbFilename: String = DEFAULT_DB_FILENAME,
-    logger: (any LoggerProtocol)? = nil
+    logger: (any LoggerProtocol) = DefaultLogger()
 ) -> PowerSyncDatabaseProtocol {
 
     return KotlinPowerSyncDatabaseImpl(
         schema: schema,
         dbFilename: dbFilename,
-        logger: logger != nil ? DatabaseLogger(logger!) : nil
+        logger: DatabaseLogger(logger)
     )
 }

Tests/PowerSyncTests/Kotlin/KotlinPowerSyncDatabaseImplTests.swift

@@ -27,7 +27,7 @@ final class KotlinPowerSyncDatabaseImplTests: XCTestCase {
         // Tests currently fail if this is called.
         // The watched query tests try and read from the DB while it's closing. 
         // This causes a PowerSyncException to be thrown in the Kotlin flow.
-        // Custom exceptions are not supported by SKIEE. This causes a crash.
+        // Custom exceptions in flows are not supported by SKIEE. This causes a crash.
         // FIXME: Reapply once watched query errors are handled better.
         // try await database.close()
         database = nil