From cbb2d328a1613bdb56172f59855f77b83c6ad49b Mon Sep 17 00:00:00 2001 From: Risto Lahtela <24460436+AuroraLS3@users.noreply.github.com> Date: Sun, 21 Mar 2021 08:26:25 +0200 Subject: [PATCH] Removed some unused code --- .../java/com/djrapitops/plan/DataSvc.java | 6 +-- .../delivery/rendering/html/Contributors.java | 4 +- .../gathering/ShutdownDataPreservation.java | 2 + .../events/CookieChangeTransaction.java | 2 +- .../ShutdownDataPreservationTest.java | 33 +++++++------- .../plan/gathering/ShutdownSaveTest.java | 21 ++------- .../test/java/extension/PrintExtension.java | 43 ------------------- .../utilities/mocks/PluginMockComponent.java | 10 ++++- 8 files changed, 36 insertions(+), 85 deletions(-) delete mode 100644 Plan/common/src/test/java/extension/PrintExtension.java diff --git a/Plan/common/src/main/java/com/djrapitops/plan/DataSvc.java b/Plan/common/src/main/java/com/djrapitops/plan/DataSvc.java index cc27379f7..b83404cb2 100644 --- a/Plan/common/src/main/java/com/djrapitops/plan/DataSvc.java +++ b/Plan/common/src/main/java/com/djrapitops/plan/DataSvc.java @@ -181,9 +181,9 @@ public class DataSvc implements DataService { } private static class Mapper { - Class typeA; - Class typeB; - Function func; + final Class typeA; + final Class typeB; + final Function func; public Mapper(Class typeA, Class typeB, Function func) { this.typeA = typeA; diff --git a/Plan/common/src/main/java/com/djrapitops/plan/delivery/rendering/html/Contributors.java b/Plan/common/src/main/java/com/djrapitops/plan/delivery/rendering/html/Contributors.java index 5b79796af..2d6d1a32b 100644 --- a/Plan/common/src/main/java/com/djrapitops/plan/delivery/rendering/html/Contributors.java +++ b/Plan/common/src/main/java/com/djrapitops/plan/delivery/rendering/html/Contributors.java @@ -103,8 +103,8 @@ public class Contributors { } private static class Contributor implements Comparable { - String name; - For[] contributed; + final String name; + final For[] contributed; Contributor(String name, For... contributed) { this.name = name; diff --git a/Plan/common/src/main/java/com/djrapitops/plan/gathering/ShutdownDataPreservation.java b/Plan/common/src/main/java/com/djrapitops/plan/gathering/ShutdownDataPreservation.java index 027aef12b..2504db494 100644 --- a/Plan/common/src/main/java/com/djrapitops/plan/gathering/ShutdownDataPreservation.java +++ b/Plan/common/src/main/java/com/djrapitops/plan/gathering/ShutdownDataPreservation.java @@ -38,6 +38,7 @@ import java.nio.charset.StandardCharsets; import java.nio.file.Files; import java.nio.file.Path; import java.nio.file.StandardOpenOption; +import java.util.Collections; import java.util.List; import java.util.Optional; import java.util.concurrent.ExecutionException; @@ -119,6 +120,7 @@ public class ShutdownDataPreservation extends TaskSystem.Task { } List loadFinishedSessions() { + if (!Files.exists(storeLocation)) return Collections.emptyList(); try (Stream lines = Files.lines(storeLocation)) { return lines.map(FinishedSession::deserializeCSV) .filter(Optional::isPresent) diff --git a/Plan/common/src/main/java/com/djrapitops/plan/storage/database/transactions/events/CookieChangeTransaction.java b/Plan/common/src/main/java/com/djrapitops/plan/storage/database/transactions/events/CookieChangeTransaction.java index 827f55c53..f3416ec45 100644 --- a/Plan/common/src/main/java/com/djrapitops/plan/storage/database/transactions/events/CookieChangeTransaction.java +++ b/Plan/common/src/main/java/com/djrapitops/plan/storage/database/transactions/events/CookieChangeTransaction.java @@ -52,7 +52,7 @@ public class CookieChangeTransaction extends Transaction { if (username == null) { execute(new ExecStatement(CookieTable.DELETE_ALL_STATEMENT) { @Override - public void prepare(PreparedStatement statement) throws SQLException { + public void prepare(PreparedStatement statement) { // No parameters } }); diff --git a/Plan/common/src/test/java/com/djrapitops/plan/gathering/ShutdownDataPreservationTest.java b/Plan/common/src/test/java/com/djrapitops/plan/gathering/ShutdownDataPreservationTest.java index ade0bae9b..5d79d8db8 100644 --- a/Plan/common/src/test/java/com/djrapitops/plan/gathering/ShutdownDataPreservationTest.java +++ b/Plan/common/src/test/java/com/djrapitops/plan/gathering/ShutdownDataPreservationTest.java @@ -16,19 +16,17 @@ */ package com.djrapitops.plan.gathering; -import com.djrapitops.plan.PlanPlugin; import com.djrapitops.plan.PlanSystem; import com.djrapitops.plan.gathering.domain.FinishedSession; +import net.playeranalytics.plugin.PlatformAbstractionLayer; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; import org.junit.jupiter.api.io.TempDir; import utilities.RandomData; -import utilities.dagger.DaggerPlanPluginComponent; -import utilities.dagger.PlanPluginComponent; -import utilities.mocks.PlanPluginMocker; -import utilities.mocks.TestPlatformAbstractionLayer; +import utilities.mocks.PluginMockComponent; import java.nio.file.Path; +import java.util.Collections; import java.util.List; import static org.junit.jupiter.api.Assertions.assertEquals; @@ -38,18 +36,10 @@ class ShutdownDataPreservationTest { private ShutdownDataPreservation underTest; @BeforeEach - void setupPreservation(@TempDir Path temporaryFolder) { - PlanPlugin planMock = PlanPluginMocker.setUp() - .withDataFolder(temporaryFolder.resolve("ShutdownSaveTest").toFile()) - .withLogging() - .getPlanMock(); - TestPlatformAbstractionLayer abstractionLayer = new TestPlatformAbstractionLayer(planMock); - PlanPluginComponent pluginComponent = DaggerPlanPluginComponent.builder() - .bindTemporaryDirectory(temporaryFolder) - .plan(planMock) - .abstractionLayer(abstractionLayer) - .build(); - PlanSystem system = pluginComponent.system(); + void setupPreservation(@TempDir Path temporaryFolder) throws Exception { + PluginMockComponent pluginMockComponent = new PluginMockComponent(temporaryFolder); + PlanSystem system = pluginMockComponent.getPlanSystem(); + PlatformAbstractionLayer abstractionLayer = pluginMockComponent.getAbstractionLayer(); underTest = new ShutdownDataPreservation( system.getPlanFiles(), @@ -69,4 +59,13 @@ class ShutdownDataPreservationTest { assertEquals(expected, result); } + @Test + void dataIsSameAfterStorageWhenNoSessions() { + List expected = Collections.emptyList(); + underTest.storeFinishedSessions(expected); + + List result = underTest.loadFinishedSessions(); + assertEquals(expected, result); + } + } \ No newline at end of file diff --git a/Plan/common/src/test/java/com/djrapitops/plan/gathering/ShutdownSaveTest.java b/Plan/common/src/test/java/com/djrapitops/plan/gathering/ShutdownSaveTest.java index c7b98f19b..ebb835c59 100644 --- a/Plan/common/src/test/java/com/djrapitops/plan/gathering/ShutdownSaveTest.java +++ b/Plan/common/src/test/java/com/djrapitops/plan/gathering/ShutdownSaveTest.java @@ -16,7 +16,6 @@ */ package com.djrapitops.plan.gathering; -import com.djrapitops.plan.PlanPlugin; import com.djrapitops.plan.PlanSystem; import com.djrapitops.plan.gathering.cache.SessionCache; import com.djrapitops.plan.gathering.domain.ActiveSession; @@ -32,19 +31,14 @@ import com.djrapitops.plan.storage.database.transactions.StoreServerInformationT import com.djrapitops.plan.storage.database.transactions.commands.RemoveEverythingTransaction; import com.djrapitops.plan.storage.database.transactions.events.PlayerRegisterTransaction; import com.djrapitops.plan.storage.database.transactions.events.WorldNameStoreTransaction; -import extension.PrintExtension; import org.junit.jupiter.api.AfterEach; import org.junit.jupiter.api.BeforeEach; import org.junit.jupiter.api.Test; -import org.junit.jupiter.api.extension.ExtendWith; import org.junit.jupiter.api.io.TempDir; import utilities.RandomData; import utilities.TestConstants; import utilities.TestPluginLogger; -import utilities.dagger.DaggerPlanPluginComponent; -import utilities.dagger.PlanPluginComponent; -import utilities.mocks.PlanPluginMocker; -import utilities.mocks.TestPlatformAbstractionLayer; +import utilities.mocks.PluginMockComponent; import java.nio.file.Path; import java.util.Collection; @@ -62,7 +56,6 @@ import static org.mockito.Mockito.when; * * @author AuroraLS3 */ -@ExtendWith(PrintExtension.class) class ShutdownSaveTest { private boolean shutdownStatus; @@ -72,16 +65,8 @@ class ShutdownSaveTest { @BeforeEach void setupShutdownSaveObject(@TempDir Path temporaryFolder) throws Exception { - PlanPlugin planMock = PlanPluginMocker.setUp() - .withDataFolder(temporaryFolder.resolve("ShutdownSaveTest").toFile()) - .withLogging() - .getPlanMock(); - PlanPluginComponent pluginComponent = DaggerPlanPluginComponent.builder() - .bindTemporaryDirectory(temporaryFolder) - .plan(planMock) - .abstractionLayer(new TestPlatformAbstractionLayer(planMock)) - .build(); - PlanSystem system = pluginComponent.system(); + PluginMockComponent pluginMockComponent = new PluginMockComponent(temporaryFolder); + PlanSystem system = pluginMockComponent.getPlanSystem(); database = system.getDatabaseSystem().getSqLiteFactory().usingFileCalled("test"); database.init(); diff --git a/Plan/common/src/test/java/extension/PrintExtension.java b/Plan/common/src/test/java/extension/PrintExtension.java deleted file mode 100644 index 14e0e4174..000000000 --- a/Plan/common/src/test/java/extension/PrintExtension.java +++ /dev/null @@ -1,43 +0,0 @@ -/* - * This file is part of Player Analytics (Plan). - * - * Plan is free software: you can redistribute it and/or modify - * it under the terms of the GNU Lesser General Public License v3 as published by - * the Free Software Foundation, either version 3 of the License, or - * (at your option) any later version. - * - * Plan is distributed in the hope that it will be useful, - * but WITHOUT ANY WARRANTY; without even the implied warranty of - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the - * GNU Lesser General Public License for more details. - * - * You should have received a copy of the GNU Lesser General Public License - * along with Plan. If not, see . - */ -package extension; - -import org.junit.jupiter.api.extension.AfterTestExecutionCallback; -import org.junit.jupiter.api.extension.BeforeTestExecutionCallback; -import org.junit.jupiter.api.extension.ExtensionContext; - -import java.lang.reflect.Method; - -/** - * JUnit 5 Extension that prints what test is being run before each test. - * - * @author AuroraLS3 - */ -public class PrintExtension implements BeforeTestExecutionCallback, AfterTestExecutionCallback { - - @Override - public void beforeTestExecution(ExtensionContext context) { - String testName = context.getTestClass().map(Class::getSimpleName).orElse("?"); - String testMethodName = context.getTestMethod().map(Method::getName).orElse("?"); - System.out.println(">> " + testName + " - " + testMethodName); - } - - @Override - public void afterTestExecution(ExtensionContext context) { - System.out.println(); - } -} \ No newline at end of file diff --git a/Plan/common/src/test/java/utilities/mocks/PluginMockComponent.java b/Plan/common/src/test/java/utilities/mocks/PluginMockComponent.java index b063c7eb3..38146aa88 100644 --- a/Plan/common/src/test/java/utilities/mocks/PluginMockComponent.java +++ b/Plan/common/src/test/java/utilities/mocks/PluginMockComponent.java @@ -19,6 +19,7 @@ package utilities.mocks; import com.djrapitops.plan.PlanPlugin; import com.djrapitops.plan.PlanSystem; import com.djrapitops.plan.utilities.logging.PluginErrorLogger; +import net.playeranalytics.plugin.PlatformAbstractionLayer; import utilities.dagger.DaggerPlanPluginComponent; import utilities.dagger.PlanPluginComponent; @@ -35,6 +36,7 @@ public class PluginMockComponent { private PlanPlugin planMock; private PlanPluginComponent component; + private TestPlatformAbstractionLayer abstractionLayer; public PluginMockComponent(Path tempDir) { this.tempDir = tempDir; @@ -57,10 +59,11 @@ public class PluginMockComponent { private void initComponent() throws Exception { if (component == null) { PlanPlugin planMock = getPlanMock(); + abstractionLayer = new TestPlatformAbstractionLayer(planMock); component = DaggerPlanPluginComponent.builder() .bindTemporaryDirectory(tempDir) .plan(planMock) - .abstractionLayer(new TestPlatformAbstractionLayer(planMock)) + .abstractionLayer(abstractionLayer) .build(); } } @@ -74,4 +77,9 @@ public class PluginMockComponent { initComponent(); return component; } + + public PlatformAbstractionLayer getAbstractionLayer() throws Exception { + initComponent(); + return abstractionLayer; + } } \ No newline at end of file