Optimized database initialization (#47)

Signed-off-by: Arnau Mora <[email protected]>

Author: ArnyminerZ

src/main/kotlin/Main.kt

@@ -23,7 +23,7 @@ fun main() {
     ServerDatabase.configureFromEnvironment()
 
     Logger.info("Connecting to the database, and creating tables...")
-    ServerDatabase.instance
+    ServerDatabase.instance.initialize()
 
     Logger.info("Initializing Crowdin connection...")
     Localization.init()

src/main/kotlin/com/arnyminerz/escalaralcoiaicomtat/backend/ServerDatabase.kt

@@ -14,6 +14,7 @@ import org.jetbrains.exposed.sql.SqlLogger
 import org.jetbrains.exposed.sql.Transaction
 import org.jetbrains.exposed.sql.addLogger
 import org.jetbrains.exposed.sql.transactions.experimental.newSuspendedTransaction
+import org.jetbrains.exposed.sql.transactions.transaction
 
 /**
  * A utility class for interacting with the database.
@@ -75,9 +76,15 @@ class ServerDatabase private constructor() {
         Database.connect(url, driver, username, password)
     }
 
-    suspend fun <T> query(block: suspend Transaction.() -> T): T = newSuspendedTransaction(Dispatchers.IO, database) {
+    /**
+     * Creates all the missing tables and columns for the database.
+     * Should be run as soon as possible in the program's lifecycle.
+     */
+    fun initialize() = transaction(database) {
         SchemaUtils.createMissingTablesAndColumns(Areas, Zones, Sectors, Paths, BlockingTable, InfoTable)
+    }
 
+    suspend fun <T> query(block: suspend Transaction.() -> T): T = newSuspendedTransaction(Dispatchers.IO, database) {
         logger?.let { addLogger(it) }
 
         block()

src/test/kotlin/com/arnyminerz/escalaralcoiaicomtat/backend/server/base/ApplicationTestBase.kt

@@ -35,7 +35,7 @@ abstract class ApplicationTestBase {
         File("testing.db").takeIf { it.exists() }?.delete()
 
         // Access the database once to initialize
-        ServerDatabase.instance
+        ServerDatabase.instance.initialize()
 
         // Configure storage
         Storage.BaseDir = File(System.getProperty("user.home"), ".EAIC-Testing")