Implement tests

Author: AzureDoom

.github/workflows/gradle.yml

@@ -19,4 +19,6 @@ jobs:
     - name: Setup Gradle
       uses: gradle/actions/setup-gradle@af1da67850ed9a4cedd57bfd976089dd991e2582 # v4.0.0
     - name: Verify build is not broken
-      run: ./gradlew build
+      run: ./gradlew build
+    - name: Verify tests are passing
+      run: ./gradlew test

build.gradle

@@ -26,6 +26,14 @@ dependencies {
     implementation 'com.mysql:mysql-connector-j:9.5.0'
     implementation 'org.mariadb.jdbc:mariadb-java-client:3.5.7'
     implementation 'org.postgresql:postgresql:42.7.8'
+    // Testing
+    testImplementation platform("org.junit:junit-bom:6.0.1")
+    testImplementation "org.junit.jupiter:junit-jupiter"
+    testRuntimeOnly "org.junit.platform:junit-platform-launcher"
+}
+
+test {
+    useJUnitPlatform()
 }
 
 spotless {

src/main/java/com/azuredoom/levelingcore/Main.java

@@ -1,14 +1,7 @@
 package com.azuredoom.levelingcore;
 
-import com.azuredoom.levelingcore.config.ConfigManager;
-import com.azuredoom.levelingcore.config.LevelFormulaFactory;
-import com.azuredoom.levelingcore.database.DataSourceFactory;
-import com.azuredoom.levelingcore.database.JdbcLevelRepository;
-import com.azuredoom.levelingcore.level.LevelServiceImpl;
-
 import java.nio.file.Path;
 import java.nio.file.Paths;
-import java.util.UUID;
 
 public class Main {
 
@@ -18,28 +11,28 @@ public class Main {
 
     static void main() {
         // TODO: Move to server startup so registration are auto handled
-        var config = ConfigManager.loadOrCreate(configPath);
-        var desc = LevelFormulaFactory.descriptorFromConfig(config);
-        var formula = LevelFormulaFactory.fromConfig(config);
-        var jdbcUrl = config.database.jdbcUrl;
-        var user = config.database.username;
-        var pass = config.database.password;
-        var maxPoolSize = config.database.maxPoolSize;
-        var dataSource = DataSourceFactory.create(jdbcUrl, user, pass, maxPoolSize);
-        var repository = new JdbcLevelRepository(dataSource);
-        if (config.formula.migrateXP) {
-            repository.migrateFormulaIfNeeded(formula, desc);
-        }
-        var levelService = new LevelServiceImpl(formula, repository);
+        // var config = ConfigManager.loadOrCreate(configPath);
+        // var desc = LevelFormulaFactory.descriptorFromConfig(config);
+        // var formula = LevelFormulaFactory.fromConfig(config);
+        // var jdbcUrl = config.database.jdbcUrl;
+        // var user = config.database.username;
+        // var pass = config.database.password;
+        // var maxPoolSize = config.database.maxPoolSize;
+        // var dataSource = DataSourceFactory.create(jdbcUrl, user, pass, maxPoolSize);
+        // var repository = new JdbcLevelRepository(dataSource);
+        // if (config.formula.migrateXP) {
+        // repository.migrateFormulaIfNeeded(formula, desc);
+        // }
+        // var levelService = new LevelServiceImpl(formula, repository);
 
         // TODO: Remove once hooks into the player/mob kill events are found and integrable.
-        var testId = UUID.fromString("d3804858-4bb8-4026-ae21-386255ed467d");
-        levelService.addXp(testId, 500);
+        // var testId = UUID.fromString("d3804858-4bb8-4026-ae21-386255ed467d");
+        // levelService.addXp(testId, 500);
         // TODO: Move to chat based logging instead of System loggers
-        LOGGER.log(System.Logger.Level.INFO, String.format("XP: %d", levelService.getXp(testId)));
-        LOGGER.log(System.Logger.Level.INFO, String.format("Level: %d", levelService.getLevel(testId)));
+        // LOGGER.log(System.Logger.Level.INFO, String.format("XP: %d", levelService.getXp(testId)));
+        // LOGGER.log(System.Logger.Level.INFO, String.format("Level: %d", levelService.getLevel(testId)));
 
         // TODO: Move to server shutdown so JDBC resources are properly closed
-        Runtime.getRuntime().addShutdownHook(new Thread(repository::close));
+        // Runtime.getRuntime().addShutdownHook(new Thread(repository::close));
     }
 }

src/test/java/com/azuredoom/levelingcoretesting/AppBootstrap.java

@@ -0,0 +1,35 @@
+package com.azuredoom.levelingcoretesting;
+
+import com.azuredoom.levelingcore.config.ConfigManager;
+import com.azuredoom.levelingcore.config.LevelFormulaFactory;
+import com.azuredoom.levelingcore.database.DataSourceFactory;
+import com.azuredoom.levelingcore.database.JdbcLevelRepository;
+import com.azuredoom.levelingcore.level.LevelServiceImpl;
+
+import java.nio.file.Path;
+
+public final class AppBootstrap {
+
+    private AppBootstrap() {}
+
+    public static LevelServiceImpl createLevelService(Path configPath) {
+        var config = ConfigManager.loadOrCreate(configPath);
+        var desc = LevelFormulaFactory.descriptorFromConfig(config);
+        var formula = LevelFormulaFactory.fromConfig(config);
+
+        var ds = DataSourceFactory.create(
+            config.database.jdbcUrl,
+            config.database.username,
+            config.database.password,
+            config.database.maxPoolSize
+        );
+
+        var repository = new JdbcLevelRepository(ds);
+
+        if (config.formula.migrateXP) {
+            repository.migrateFormulaIfNeeded(formula, desc);
+        }
+
+        return new LevelServiceImpl(formula, repository);
+    }
+}

src/test/java/com/azuredoom/levelingcoretesting/LevelServiceTest.java

@@ -0,0 +1,30 @@
+package com.azuredoom.levelingcoretesting;
+
+import com.azuredoom.levelingcore.Main;
+import org.junit.jupiter.api.Test;
+
+import java.util.UUID;
+
+import static org.junit.jupiter.api.Assertions.assertEquals;
+import static org.junit.jupiter.api.Assertions.assertTrue;
+
+class LevelServiceTest {
+
+    public static final System.Logger LOGGER = System.getLogger(Main.class.getName());
+
+    @Test
+    void addXp_increasesXpAndLevel() {
+        var service = AppBootstrap.createLevelService(Main.configPath);
+        var id = UUID.fromString("d3804858-4bb8-4026-ae21-386255ed467d");
+        var amount = 500;
+        var beforeXp = service.getXp(id);
+        var beforeLevel = service.getLevel(id);
+
+        service.addXp(id, amount);
+        LOGGER.log(System.Logger.Level.INFO, String.format("XP: %d", service.getXp(id)));
+        LOGGER.log(System.Logger.Level.INFO, String.format("Level: %d", service.getLevel(id)));
+
+        assertEquals(beforeXp + amount, service.getXp(id));
+        assertTrue(service.getLevel(id) >= beforeLevel);
+    }
+}