From 1ba9b1b3cdfa946e91564bf3733e2d4b72a3c3d7 Mon Sep 17 00:00:00 2001
From: AJ <aj@alt>
Date: Wed, 17 Apr 2024 09:19:41 -0700
Subject: [PATCH] Remove kotest dependency from compose tests

---
 .../build.gradle.kts                          |  1 -
 .../composecolor/ComposeColorExtensions.kt    |  4 +-
 .../colorint/ComposeColorExtensionsTest.kt    | 74 ++++++++++---------
 3 files changed, 41 insertions(+), 38 deletions(-)

diff --git a/extensions/colormath-ext-jetpack-compose/build.gradle.kts b/extensions/colormath-ext-jetpack-compose/build.gradle.kts
index a7721c7..5138746 100644
--- a/extensions/colormath-ext-jetpack-compose/build.gradle.kts
+++ b/extensions/colormath-ext-jetpack-compose/build.gradle.kts
@@ -34,7 +34,6 @@ kotlin {
         }
         commonTest.dependencies {
             implementation(kotlin("test"))
-            implementation(libs.kotest)
         }
     }
 }
diff --git a/extensions/colormath-ext-jetpack-compose/src/commonMain/kotlin/com/github/ajalt/colormath/extensions/android/composecolor/ComposeColorExtensions.kt b/extensions/colormath-ext-jetpack-compose/src/commonMain/kotlin/com/github/ajalt/colormath/extensions/android/composecolor/ComposeColorExtensions.kt
index 3961e5f..c833558 100644
--- a/extensions/colormath-ext-jetpack-compose/src/commonMain/kotlin/com/github/ajalt/colormath/extensions/android/composecolor/ComposeColorExtensions.kt
+++ b/extensions/colormath-ext-jetpack-compose/src/commonMain/kotlin/com/github/ajalt/colormath/extensions/android/composecolor/ComposeColorExtensions.kt
@@ -72,10 +72,10 @@ fun Color.toComposeColor(): ComposeColor {
     }
 
     return if (s == null) {
-        val (r, g, b, a) = toSRGB()
+        val (r, g, b, a) = toSRGB().clamp()
         ComposeColor(r, g, b, a)
     } else {
-        val (r, g, b, a) = toArray()
+        val (r, g, b, a) = clamp().toArray()
         ComposeColor(r, g, b, a, s)
     }
 }
diff --git a/extensions/colormath-ext-jetpack-compose/src/jvmTest/kotlin/com/github/ajalt/colormath/extensions/android/colorint/ComposeColorExtensionsTest.kt b/extensions/colormath-ext-jetpack-compose/src/jvmTest/kotlin/com/github/ajalt/colormath/extensions/android/colorint/ComposeColorExtensionsTest.kt
index 312652c..58d1358 100644
--- a/extensions/colormath-ext-jetpack-compose/src/jvmTest/kotlin/com/github/ajalt/colormath/extensions/android/colorint/ComposeColorExtensionsTest.kt
+++ b/extensions/colormath-ext-jetpack-compose/src/jvmTest/kotlin/com/github/ajalt/colormath/extensions/android/colorint/ComposeColorExtensionsTest.kt
@@ -6,8 +6,10 @@ import com.github.ajalt.colormath.convertTo
 import com.github.ajalt.colormath.extensions.android.composecolor.toColormathColor
 import com.github.ajalt.colormath.extensions.android.composecolor.toColormathSRGB
 import com.github.ajalt.colormath.extensions.android.composecolor.toComposeColor
-import com.github.ajalt.colormath.model.*
+import com.github.ajalt.colormath.model.JzAzBz
 import com.github.ajalt.colormath.model.LABColorSpaces.LAB50
+import com.github.ajalt.colormath.model.Oklab
+import com.github.ajalt.colormath.model.RGB
 import com.github.ajalt.colormath.model.RGBColorSpaces.ACES
 import com.github.ajalt.colormath.model.RGBColorSpaces.ACEScg
 import com.github.ajalt.colormath.model.RGBColorSpaces.AdobeRGB
@@ -18,58 +20,60 @@ import com.github.ajalt.colormath.model.RGBColorSpaces.DisplayP3
 import com.github.ajalt.colormath.model.RGBColorSpaces.LinearSRGB
 import com.github.ajalt.colormath.model.RGBColorSpaces.ROMM_RGB
 import com.github.ajalt.colormath.model.RGBColorSpaces.SRGB
+import com.github.ajalt.colormath.model.XYZ
 import com.github.ajalt.colormath.model.XYZColorSpaces.XYZ50
-import io.kotest.assertions.throwables.shouldNotThrow
-import io.kotest.assertions.throwables.shouldThrow
-import io.kotest.data.blocking.forAll
-import io.kotest.data.row
-import io.kotest.matchers.shouldBe
 import kotlin.test.Test
+import kotlin.test.assertEquals
+import kotlin.test.fail
 
 class ComposeColorExtensionsTest {
-    private val colormathBlue = RGB(0, 0, 2, 1)
+    private val colormathBlue = RGB(0, 0, 1, 1)
 
+    // TODO(kotest): once kotest is released, go back to using it
     @Test
     fun toColormathColor() {
-        colormathBlue shouldBe Color.Blue.toColormathColor()
+        assertEquals(colormathBlue, Color.Blue.toColormathColor())
     }
 
     @Test
     fun toColormathSRGB() {
-        colormathBlue shouldBe Color.Blue.toColormathSRGB()
+        assertEquals(colormathBlue, Color.Blue.toColormathSRGB())
     }
 
     @Test
     fun toComposeColor() {
-        Color.Blue shouldBe colormathBlue.toComposeColor()
+        assertEquals(Color.Blue, colormathBlue.toComposeColor())
     }
 
     @Test
-    fun outOfGamut() = forAll(
-        row(SRGB),
-        row(ACES),
-        row(ACEScg),
-        row(AdobeRGB),
-        row(BT2020),
-        row(BT709),
-        row(LAB50),
-        row(XYZ50),
-        row(DCI_P3),
-        row(DisplayP3),
-        row(LinearSRGB),
-        row(ROMM_RGB),
-        row(XYZ),
-        row(Oklab),
-        row(JzAzBz),
-    ) { space: ColorSpace<*> ->
+    fun outOfGamut() = listOf<ColorSpace<*>>(
+        SRGB,
+        ACES,
+        ACEScg,
+        AdobeRGB,
+        BT2020,
+        BT709,
+        LAB50,
+        XYZ50,
+        DCI_P3,
+        DisplayP3,
+        LinearSRGB,
+        ROMM_RGB,
+        XYZ,
+        Oklab,
+        JzAzBz,
+    ).forEach { space: ColorSpace<*> ->
         val color = RGB(9, 9, 9, 9).convertTo(space)
-        shouldNotThrow<IllegalArgumentException> { color.clamp().toComposeColor() }
-        shouldThrow<IllegalArgumentException> { color.toComposeColor() }
-    }
-}
 
-interface F {
-    operator fun component1(): String
-}
+        // shouldNotThrow
 
-data class G(val a: String, val b: Int) : F
+        color.clamp().toComposeColor()
+
+        // shouldThrow
+        try {
+            color.toComposeColor()
+        } catch (e: IllegalArgumentException) {
+            // expected
+        }
+    }
+}