refactor(common-utils): Use forAll in tests to simplify code

Move the stand-alone assertion to the begging of the block for visibility while at it. Signed-off-by: Sebastian Schuberth <[email protected]>
oss-review-toolkit · May 24, 2024 · 779fbc9 · 779fbc9
1 parent f0ff84f
commit 779fbc9
Showing 1 changed file with 12 additions and 14 deletions.
diff --git a/utils/common/src/test/kotlin/ExtensionsTest.kt b/utils/common/src/test/kotlin/ExtensionsTest.kt
@@ -374,26 +374,24 @@ class ExtensionsTest : WordSpec({
             val special = listOf('-', '.', '_', '~')
             val unreserved = alpha + digit + special
 
-            assertSoftly {
-                reserved.forEach {
-                    val decoded = it.toString()
+            " ".percentEncode() shouldBe "%20"
 
-                    val encoded = decoded.percentEncode()
+            reserved.forAll {
+                val decoded = it.toString()
 
-                    encoded shouldBe String.format(Locale.ROOT, "%%%02X", it.code)
-                    URLDecoder.decode(encoded, Charsets.UTF_8) shouldBe decoded
-                }
+                val encoded = decoded.percentEncode()
 
-                unreserved.forEach {
-                    val decoded = it.toString()
+                encoded shouldBe String.format(Locale.ROOT, "%%%02X", it.code)
+                URLDecoder.decode(encoded, Charsets.UTF_8) shouldBe decoded
+            }
 
-                    val encoded = decoded.percentEncode()
+            unreserved.asList().forAll {
+                val decoded = it.toString()
 
-                    encoded shouldBe decoded
-                    URLDecoder.decode(encoded, Charsets.UTF_8) shouldBe decoded
-                }
+                val encoded = decoded.percentEncode()
 
-                " ".percentEncode() shouldBe "%20"
+                encoded shouldBe decoded
+                URLDecoder.decode(encoded, Charsets.UTF_8) shouldBe decoded
             }
         }
     }