From 87a256c9c581242a53a25c9e31ec576422855d25 Mon Sep 17 00:00:00 2001
From: ineanto <pantoine.rochas@gmail.com>
Date: Fri, 14 Jul 2023 13:50:59 +0200
Subject: [PATCH] fix: make tests run in order

---
 .../test/storage/redis/RedisCacheTest.java     | 18 +++++++++++++++---
 1 file changed, 15 insertions(+), 3 deletions(-)

diff --git a/src/test/java/xyz/atnrch/nicko/test/storage/redis/RedisCacheTest.java b/src/test/java/xyz/atnrch/nicko/test/storage/redis/RedisCacheTest.java
index b8bf8b7..18eb313 100644
--- a/src/test/java/xyz/atnrch/nicko/test/storage/redis/RedisCacheTest.java
+++ b/src/test/java/xyz/atnrch/nicko/test/storage/redis/RedisCacheTest.java
@@ -5,6 +5,7 @@ import be.seeseemelk.mockbukkit.ServerMock;
 import be.seeseemelk.mockbukkit.entity.PlayerMock;
 import org.junit.jupiter.api.*;
 import xyz.atnrch.nicko.NickoBukkit;
+import xyz.atnrch.nicko.appearance.ActionResult;
 import xyz.atnrch.nicko.appearance.AppearanceManager;
 import xyz.atnrch.nicko.config.Configuration;
 import xyz.atnrch.nicko.config.DataSourceConfiguration;
@@ -13,9 +14,9 @@ import xyz.atnrch.nicko.storage.PlayerDataStore;
 
 import java.util.Optional;
 
-import static org.junit.jupiter.api.Assertions.assertEquals;
-import static org.junit.jupiter.api.Assertions.assertTrue;
+import static org.junit.jupiter.api.Assertions.*;
 
+@TestMethodOrder(MethodOrderer.OrderAnnotation.class)
 public class RedisCacheTest {
     private static ServerMock server;
     private static NickoBukkit plugin;
@@ -35,6 +36,7 @@ public class RedisCacheTest {
 
     @Test
     @DisplayName("Cache Profile")
+    @Order(1)
     public void cacheProfile() {
         final Optional<NickoProfile> optionalProfile = plugin.getDataStore().getData(player.getUniqueId());
         assertTrue(optionalProfile.isPresent());
@@ -43,7 +45,8 @@ public class RedisCacheTest {
 
     @Test
     @DisplayName("Update Cache Profile")
-    public void updatePlayerCache() {
+    @Order(2)
+    public void updateCache() {
         final PlayerDataStore dataStore = plugin.getDataStore();
         final AppearanceManager appearanceManager = AppearanceManager.get(player);
         appearanceManager.setName("Notch");
@@ -54,6 +57,15 @@ public class RedisCacheTest {
         assertEquals(retrieved.getName(), "Notch");
     }
 
+    @Test
+    @DisplayName("Delete Cache Profile")
+    @Order(3)
+    public void deleteCache() {
+        final PlayerDataStore dataStore = plugin.getDataStore();
+        final ActionResult cacheDelete = dataStore.getCache().delete(player.getUniqueId());
+        assertFalse(cacheDelete.isError());
+    }
+
     @AfterAll
     public static void shutdown() {
         MockBukkit.unmock();