diff --git a/nicko-core/dependency-reduced-pom.xml b/nicko-core/dependency-reduced-pom.xml index dbcbfbf..817f08a 100644 --- a/nicko-core/dependency-reduced-pom.xml +++ b/nicko-core/dependency-reduced-pom.xml @@ -19,9 +19,6 @@ maven-surefire-plugin 3.0.0-M7 - - true - maven-shade-plugin diff --git a/nicko-core/src/main/java/net/artelnatif/nicko/storage/sql/SQLStorage.java b/nicko-core/src/main/java/net/artelnatif/nicko/storage/sql/SQLStorage.java index a40673a..c755461 100644 --- a/nicko-core/src/main/java/net/artelnatif/nicko/storage/sql/SQLStorage.java +++ b/nicko-core/src/main/java/net/artelnatif/nicko/storage/sql/SQLStorage.java @@ -34,6 +34,8 @@ public class SQLStorage extends Storage { @Override public ActionResult store(UUID uuid, NickoProfile profile) { final Connection connection = getProvider().getConnection(); + if (connection == null) return new ActionResult<>(I18NDict.Error.SQL_ERROR); + try { final String sql = """ INSERT IGNORE INTO nicko.DATA diff --git a/nicko-core/src/test/java/net/artelnatif/nicko/test/storage/BrokenSQLTest.java b/nicko-core/src/test/java/net/artelnatif/nicko/test/storage/BrokenSQLTest.java index c2e5486..ce89297 100644 --- a/nicko-core/src/test/java/net/artelnatif/nicko/test/storage/BrokenSQLTest.java +++ b/nicko-core/src/test/java/net/artelnatif/nicko/test/storage/BrokenSQLTest.java @@ -34,12 +34,12 @@ public class BrokenSQLTest { } @Test - @DisplayName("Store Player Via SQL") + @DisplayName("Fail to Store Player Via SQL") public void storePlayer() { final PlayerMock playerMock = server.addPlayer(); final NickoProfile profile = new NickoProfile("Notch", "Notch", Locale.ENGLISH, true); final ActionResult storeAction = plugin.getDataStore().getStorage().store(playerMock.getUniqueId(), profile); - Assertions.assertFalse(storeAction.isError()); + Assertions.assertTrue(storeAction.isError()); } @AfterAll