Browse Source

SQLDatabaseManager refactor pass

nossr50 2 weeks ago
parent
commit
43502b255c

+ 6 - 0
Changelog.txt

@@ -3,6 +3,12 @@ Version 2.2.046
     Added permissions related to Spears
     Added permissions related to Spears
     Added /spears skill command
     Added /spears skill command
     Fixed bug where converting from SQL to FlatFile would not copy data for tridents, crossbows, maces, or spears
     Fixed bug where converting from SQL to FlatFile would not copy data for tridents, crossbows, maces, or spears
+    (Codebase) Added dockerized unit tests for SQL databases
+    (Codebase) Large refactoring to SQLDatabaseManager to bring it up to modern standards and improve maintainability
+
+    NOTES:
+    If you manually compile mcMMO you will need docker to run the unit tests, if you'd rather not install docker simply just add -DskipTests to your maven instructions
+
 
 
 Version 2.2.045
 Version 2.2.045
     Green Thumb now replants some crops it was failing to replant before (see notes)
     Green Thumb now replants some crops it was failing to replant before (see notes)

File diff suppressed because it is too large
+ 408 - 311
src/main/java/com/gmail/nossr50/database/SQLDatabaseManager.java


+ 0 - 2
src/test/java/com/gmail/nossr50/database/SQLDatabaseManagerTest.java

@@ -474,8 +474,6 @@ class SQLDatabaseManagerTest {
         when(upgradeManager.shouldUpgrade(any(UpgradeType.class))).thenReturn(false);
         when(upgradeManager.shouldUpgrade(any(UpgradeType.class))).thenReturn(false);
     }
     }
 
 
-
-
     // ------------------------------------------------------------------------
     // ------------------------------------------------------------------------
     // New user -> rows in all core tables
     // New user -> rows in all core tables
     // ------------------------------------------------------------------------
     // ------------------------------------------------------------------------

Some files were not shown because too many files changed in this diff