test: Switch to kotest & test mixinsHEADmaster

10 files changed, 150 insertions, 105 deletions

diff --git a/src/test/kotlin/MixinTest.kt b/src/test/kotlin/MixinTest.kt
new file mode 100644
index 0000000..55aa7c2
--- /dev/null
+++ b/src/test/kotlin/MixinTest.kt
@@ -0,0 +1,34 @@
+package moe.nea.firmament.test
+
+import org.junit.jupiter.api.Assertions
+import org.junit.jupiter.api.Test
+import org.spongepowered.asm.mixin.MixinEnvironment
+import org.spongepowered.asm.mixin.transformer.IMixinTransformer
+import moe.nea.firmament.init.MixinPlugin
+
+class MixinTest {
+	@Test
+	fun mixinAudit() {
+		FirmTestBootstrap.bootstrapMinecraft()
+		MixinEnvironment.getCurrentEnvironment().audit()
+		val mp = MixinPlugin.instances.single()
+		Assertions.assertEquals(
+			mp.expectedFullPathMixins,
+			mp.appliedFullPathMixins,
+		)
+		Assertions.assertNotEquals(
+			0,
+			mp.mixins.size
+		)
+
+	}
+
+	@Test
+	fun hasInstalledMixinTransformer() {
+		Assertions.assertInstanceOf(
+			IMixinTransformer::class.java,
+			MixinEnvironment.getCurrentEnvironment().activeTransformer
+		)
+	}
+}
+
diff --git a/src/test/kotlin/root.kt b/src/test/kotlin/root.kt
index 045fdd5..000ddda 100644
--- a/src/test/kotlin/root.kt
+++ b/src/test/kotlin/root.kt
@@ -24,6 +24,7 @@ object FirmTestBootstrap {
 		println("Bootstrap completed at $loadEnd after $loadDuration")
 	}
 
+	@JvmStatic
 	fun bootstrapMinecraft() {
 	}
 }
diff --git a/src/test/kotlin/testutil/AutoBootstrapExtension.kt b/src/test/kotlin/testutil/AutoBootstrapExtension.kt
new file mode 100644
index 0000000..6f225a0
--- /dev/null
+++ b/src/test/kotlin/testutil/AutoBootstrapExtension.kt
@@ -0,0 +1,14 @@
+package moe.nea.firmament.test.testutil
+
+import com.google.auto.service.AutoService
+import org.junit.jupiter.api.extension.BeforeAllCallback
+import org.junit.jupiter.api.extension.Extension
+import org.junit.jupiter.api.extension.ExtensionContext
+import moe.nea.firmament.test.FirmTestBootstrap
+
+@AutoService(Extension::class)
+class AutoBootstrapExtension : Extension, BeforeAllCallback {
+	override fun beforeAll(p0: ExtensionContext) {
+		FirmTestBootstrap.bootstrapMinecraft()
+	}
+}
diff --git a/src/test/kotlin/testutil/KotestPlugin.kt b/src/test/kotlin/testutil/KotestPlugin.kt
deleted file mode 100644
index 6db50fb..0000000
--- a/src/test/kotlin/testutil/KotestPlugin.kt
+++ /dev/null
@@ -1,16 +0,0 @@
-package moe.nea.firmament.test.testutil
-
-import io.kotest.core.config.AbstractProjectConfig
-import io.kotest.core.extensions.Extension
-import moe.nea.firmament.test.FirmTestBootstrap
-
-class KotestPlugin : AbstractProjectConfig() {
-	override fun extensions(): List<Extension> {
-		return listOf()
-	}
-
-	override suspend fun beforeProject() {
-		FirmTestBootstrap.bootstrapMinecraft()
-		super.beforeProject()
-	}
-}
diff --git a/src/test/kotlin/util/ColorCodeTest.kt b/src/test/kotlin/util/ColorCodeTest.kt
index 949749e..7c581c5 100644
--- a/src/test/kotlin/util/ColorCodeTest.kt
+++ b/src/test/kotlin/util/ColorCodeTest.kt
@@ -1,57 +1,57 @@
 package moe.nea.firmament.test.util
 
-import io.kotest.core.spec.style.AnnotationSpec
 import org.junit.jupiter.api.Assertions
+import org.junit.jupiter.api.Test
 import moe.nea.firmament.util.removeColorCodes
 
 
-class ColorCodeTest : AnnotationSpec() {
-    @Test
-    fun testWhatever() {
-        Assertions.assertEquals("", "".removeColorCodes())
-        Assertions.assertEquals("", "§".removeColorCodes())
-        Assertions.assertEquals("", "§a".removeColorCodes())
-        Assertions.assertEquals("ab", "a§ab".removeColorCodes())
-        Assertions.assertEquals("ab", "a§ab§§".removeColorCodes())
-        Assertions.assertEquals("abc", "a§ab§§c".removeColorCodes())
-        Assertions.assertEquals("bc", "§ab§§c".removeColorCodes())
-        Assertions.assertEquals("b§lc", "§ab§l§§c".removeColorCodes(true))
-        Assertions.assertEquals("b§lc§l", "§ab§l§§c§l".removeColorCodes(true))
-        Assertions.assertEquals("§lb§lc", "§l§ab§l§§c".removeColorCodes(true))
-    }
-
-    @Test
-    fun testEdging() {
-        Assertions.assertEquals("", "§".removeColorCodes())
-        Assertions.assertEquals("a", "a§".removeColorCodes())
-        Assertions.assertEquals("b", "§ab§".removeColorCodes())
-    }
-
-    @Test
-    fun `testDouble§`() {
-        Assertions.assertEquals("1", "§§1".removeColorCodes())
-    }
-
-    @Test
-    fun testKeepNonColor() {
-        Assertions.assertEquals("§k§l§m§n§o§r", "§k§l§m§f§n§o§r".removeColorCodes(true))
-    }
-
-    @Test
-    fun testPlainString() {
-        Assertions.assertEquals("bcdefgp", "bcdefgp".removeColorCodes())
-        Assertions.assertEquals("", "".removeColorCodes())
-    }
-
-    @Test
-    fun testSomeNormalTestCases() {
-        Assertions.assertEquals(
-            "You are not currently in a party.",
-            "§r§cYou are not currently in a party.§r".removeColorCodes()
-        )
-        Assertions.assertEquals(
-            "Ancient Necron's Chestplate ✪✪✪✪",
-            "§dAncient Necron's Chestplate §6✪§6✪§6✪§6✪".removeColorCodes()
-        )
-    }
+class ColorCodeTest {
+	@Test
+	fun testWhatever() {
+		Assertions.assertEquals("", "".removeColorCodes())
+		Assertions.assertEquals("", "§".removeColorCodes())
+		Assertions.assertEquals("", "§a".removeColorCodes())
+		Assertions.assertEquals("ab", "a§ab".removeColorCodes())
+		Assertions.assertEquals("ab", "a§ab§§".removeColorCodes())
+		Assertions.assertEquals("abc", "a§ab§§c".removeColorCodes())
+		Assertions.assertEquals("bc", "§ab§§c".removeColorCodes())
+		Assertions.assertEquals("b§lc", "§ab§l§§c".removeColorCodes(true))
+		Assertions.assertEquals("b§lc§l", "§ab§l§§c§l".removeColorCodes(true))
+		Assertions.assertEquals("§lb§lc", "§l§ab§l§§c".removeColorCodes(true))
+	}
+
+	@Test
+	fun testEdging() {
+		Assertions.assertEquals("", "§".removeColorCodes())
+		Assertions.assertEquals("a", "a§".removeColorCodes())
+		Assertions.assertEquals("b", "§ab§".removeColorCodes())
+	}
+
+	@Test
+	fun `testDouble§`() {
+		Assertions.assertEquals("1", "§§1".removeColorCodes())
+	}
+
+	@Test
+	fun testKeepNonColor() {
+		Assertions.assertEquals("§k§l§m§n§o§r", "§k§l§m§f§n§o§r".removeColorCodes(true))
+	}
+
+	@Test
+	fun testPlainString() {
+		Assertions.assertEquals("bcdefgp", "bcdefgp".removeColorCodes())
+		Assertions.assertEquals("", "".removeColorCodes())
+	}
+
+	@Test
+	fun testSomeNormalTestCases() {
+		Assertions.assertEquals(
+			"You are not currently in a party.",
+			"§r§cYou are not currently in a party.§r".removeColorCodes()
+		)
+		Assertions.assertEquals(
+			"Ancient Necron's Chestplate ✪✪✪✪",
+			"§dAncient Necron's Chestplate §6✪§6✪§6✪§6✪".removeColorCodes()
+		)
+	}
 }
diff --git a/src/test/kotlin/util/TextUtilText.kt b/src/test/kotlin/util/TextUtilText.kt
index 46ed3b4..94ab222 100644
--- a/src/test/kotlin/util/TextUtilText.kt
+++ b/src/test/kotlin/util/TextUtilText.kt
@@ -1,16 +1,18 @@
 package moe.nea.firmament.test.util
 
-import io.kotest.core.spec.style.AnnotationSpec
 import org.junit.jupiter.api.Assertions
+import org.junit.jupiter.api.Test
 import moe.nea.firmament.test.testutil.ItemResources
 import moe.nea.firmament.util.getLegacyFormatString
 
-class TextUtilText : AnnotationSpec() {
+class TextUtilText {
 	@Test
 	fun testThing() {
 		// TODO: add more tests that are directly validated with 1.8.9 code
 		val text = ItemResources.loadText("all-chat")
-		Assertions.assertEquals("§r§r§8[§r§9302§r§8] §r§6♫ §r§b[MVP§r§d+§r§b] lrg89§r§f: test§r",
-		                        text.getLegacyFormatString())
+		Assertions.assertEquals(
+			"§r§r§8[§r§9302§r§8] §r§6♫ §r§b[MVP§r§d+§r§b] lrg89§r§f: test§r",
+			text.getLegacyFormatString()
+		)
 	}
 }
diff --git a/src/test/kotlin/util/math/GChainReconciliationTest.kt b/src/test/kotlin/util/math/GChainReconciliationTest.kt
index 502bd9e..380ea5c 100644
--- a/src/test/kotlin/util/math/GChainReconciliationTest.kt
+++ b/src/test/kotlin/util/math/GChainReconciliationTest.kt
@@ -1,12 +1,12 @@
 package moe.nea.firmament.test.util.math
 
-import io.kotest.core.spec.style.AnnotationSpec
 import org.junit.jupiter.api.Assertions
 import org.junit.jupiter.api.Assertions.*
+import org.junit.jupiter.api.Test
 import moe.nea.firmament.util.math.GChainReconciliation
 import moe.nea.firmament.util.math.GChainReconciliation.rotated
 
-class GChainReconciliationTest : AnnotationSpec() {
+class GChainReconciliationTest  {
 
 	fun <T> assertEqualCycles(
 		expected: List<T>,
diff --git a/src/test/kotlin/util/skyblock/AbilityUtilsTest.kt b/src/test/kotlin/util/skyblock/AbilityUtilsTest.kt
index 206a357..9d25aad 100644
--- a/src/test/kotlin/util/skyblock/AbilityUtilsTest.kt
+++ b/src/test/kotlin/util/skyblock/AbilityUtilsTest.kt
@@ -1,7 +1,7 @@
 package moe.nea.firmament.test.util.skyblock
 
-import io.kotest.core.spec.style.AnnotationSpec
 import org.junit.jupiter.api.Assertions
+import org.junit.jupiter.api.Test
 import kotlin.time.Duration.Companion.minutes
 import kotlin.time.Duration.Companion.seconds
 import net.minecraft.text.Text
@@ -9,7 +9,7 @@ import moe.nea.firmament.test.testutil.ItemResources
 import moe.nea.firmament.util.skyblock.AbilityUtils
 import moe.nea.firmament.util.unformattedString
 
-class AbilityUtilsTest : AnnotationSpec() {
+class AbilityUtilsTest {
 
 	fun List<AbilityUtils.ItemAbility>.stripDescriptions() = map {
 		it.copy(descriptionLines = it.descriptionLines.map { Text.literal(it.unformattedString) })
@@ -24,9 +24,11 @@ class AbilityUtilsTest : AnnotationSpec() {
 					false,
 					AbilityUtils.AbilityActivation.RIGHT_CLICK,
 					null,
-					listOf("Throw your pickaxe to create an",
-					       "explosion mining all ores in a 3 block",
-					       "radius.").map(Text::literal),
+					listOf(
+						"Throw your pickaxe to create an",
+						"explosion mining all ores in a 3 block",
+						"radius."
+					).map(Text::literal),
 					48.seconds
 				)
 			),
@@ -43,8 +45,10 @@ class AbilityUtilsTest : AnnotationSpec() {
 					true,
 					AbilityUtils.AbilityActivation.RIGHT_CLICK,
 					null,
-					listOf("Grants +200% ⸕ Mining Speed for",
-					       "10s.").map(Text::literal),
+					listOf(
+						"Grants +200% ⸕ Mining Speed for",
+						"10s."
+					).map(Text::literal),
 					2.minutes
 				)
 			),
@@ -58,8 +62,10 @@ class AbilityUtilsTest : AnnotationSpec() {
 			listOf(
 				AbilityUtils.ItemAbility(
 					"Instant Transmission", true, AbilityUtils.AbilityActivation.RIGHT_CLICK, 23,
-					listOf("Teleport 12 blocks ahead of you and",
-					       "gain +50 ✦ Speed for 3 seconds.").map(Text::literal),
+					listOf(
+						"Teleport 12 blocks ahead of you and",
+						"gain +50 ✦ Speed for 3 seconds."
+					).map(Text::literal),
 					null
 				),
 				AbilityUtils.ItemAbility(
@@ -67,9 +73,11 @@ class AbilityUtilsTest : AnnotationSpec() {
 					false,
 					AbilityUtils.AbilityActivation.SNEAK_RIGHT_CLICK,
 					90,
-					listOf("Teleport to your targeted block up",
-					       "to 61 blocks away.",
-					       "Soulflow Cost: 1").map(Text::literal),
+					listOf(
+						"Teleport to your targeted block up",
+						"to 61 blocks away.",
+						"Soulflow Cost: 1"
+					).map(Text::literal),
 					null
 				)
 			),
diff --git a/src/test/kotlin/util/skyblock/ItemTypeTest.kt b/src/test/kotlin/util/skyblock/ItemTypeTest.kt
index cca3d13..c0ef2a3 100644
--- a/src/test/kotlin/util/skyblock/ItemTypeTest.kt
+++ b/src/test/kotlin/util/skyblock/ItemTypeTest.kt
@@ -1,26 +1,28 @@
 package moe.nea.firmament.test.util.skyblock
 
-import io.kotest.core.spec.style.ShouldSpec
-import io.kotest.matchers.shouldBe
+import org.junit.jupiter.api.Assertions
+import org.junit.jupiter.api.DynamicTest
+import org.junit.jupiter.api.TestFactory
 import moe.nea.firmament.test.testutil.ItemResources
 import moe.nea.firmament.util.skyblock.ItemType
 
-class ItemTypeTest
-	: ShouldSpec(
-	{
-		context("ItemType.fromItemstack") {
-			listOf(
-				"pets/lion-item" to ItemType.PET,
-				"pets/rabbit-selected" to ItemType.PET,
-				"pets/mithril-golem-not-selected" to ItemType.PET,
-				"aspect-of-the-void" to ItemType.SWORD,
-				"titanium-drill" to ItemType.DRILL,
-				"diamond-pickaxe" to ItemType.PICKAXE,
-				"gemstone-gauntlet" to ItemType.GAUNTLET,
-			).forEach { (name, typ) ->
-				should("return $typ for $name") {
-					ItemType.fromItemStack(ItemResources.loadItem(name)) shouldBe typ
-				}
+class ItemTypeTest {
+	@TestFactory
+	fun fromItemstack() =
+		listOf(
+			"pets/lion-item" to ItemType.PET,
+			"pets/rabbit-selected" to ItemType.PET,
+			"pets/mithril-golem-not-selected" to ItemType.PET,
+			"aspect-of-the-void" to ItemType.SWORD,
+			"titanium-drill" to ItemType.DRILL,
+			"diamond-pickaxe" to ItemType.PICKAXE,
+			"gemstone-gauntlet" to ItemType.GAUNTLET,
+		).map { (name, typ) ->
+			DynamicTest.dynamicTest("return $typ for $name") {
+				Assertions.assertEquals(
+					typ,
+					ItemType.fromItemStack(ItemResources.loadItem(name))
+				)
 			}
 		}
-	})
+}
diff --git a/src/test/kotlin/util/skyblock/SackUtilTest.kt b/src/test/kotlin/util/skyblock/SackUtilTest.kt
index f93cd2b..e0e3e63 100644
--- a/src/test/kotlin/util/skyblock/SackUtilTest.kt
+++ b/src/test/kotlin/util/skyblock/SackUtilTest.kt
@@ -1,12 +1,12 @@
 package moe.nea.firmament.test.util.skyblock
 
-import io.kotest.core.spec.style.AnnotationSpec
 import org.junit.jupiter.api.Assertions
+import org.junit.jupiter.api.Test
 import moe.nea.firmament.test.testutil.ItemResources
 import moe.nea.firmament.util.skyblock.SackUtil
 import moe.nea.firmament.util.skyblock.SkyBlockItems
 
-class SackUtilTest : AnnotationSpec() {
+class SackUtilTest {
 	@Test
 	fun testOneRottenFlesh() {
 		Assertions.assertEquals(