Remove suspend modifier from PartScope.

It leads to a false-positive report in intelliJ complaining about calling suspend functions even though runtime worked.

Author: Jadarma

aockt-test/api/aockt-test.api

@@ -14,17 +14,17 @@ public abstract class io/github/jadarma/aockt/test/AdventSpec : io/kotest/core/s
 	public final fun dispatcherAffinity ()Ljava/lang/Boolean;
 	public final fun getSolution ()Lio/github/jadarma/aockt/core/Solution;
 	public final fun isolationMode ()Lio/kotest/core/spec/IsolationMode;
-	public final fun partOne-51bEbmg (ZZLio/github/jadarma/aockt/test/ExecMode;Lkotlin/time/Duration;Lkotlin/jvm/functions/Function2;)V
-	public static synthetic fun partOne-51bEbmg$default (Lio/github/jadarma/aockt/test/AdventSpec;ZZLio/github/jadarma/aockt/test/ExecMode;Lkotlin/time/Duration;Lkotlin/jvm/functions/Function2;ILjava/lang/Object;)V
-	public final fun partTwo-51bEbmg (ZZLio/github/jadarma/aockt/test/ExecMode;Lkotlin/time/Duration;Lkotlin/jvm/functions/Function2;)V
-	public static synthetic fun partTwo-51bEbmg$default (Lio/github/jadarma/aockt/test/AdventSpec;ZZLio/github/jadarma/aockt/test/ExecMode;Lkotlin/time/Duration;Lkotlin/jvm/functions/Function2;ILjava/lang/Object;)V
+	public final fun partOne-51bEbmg (ZZLio/github/jadarma/aockt/test/ExecMode;Lkotlin/time/Duration;Lkotlin/jvm/functions/Function1;)V
+	public static synthetic fun partOne-51bEbmg$default (Lio/github/jadarma/aockt/test/AdventSpec;ZZLio/github/jadarma/aockt/test/ExecMode;Lkotlin/time/Duration;Lkotlin/jvm/functions/Function1;ILjava/lang/Object;)V
+	public final fun partTwo-51bEbmg (ZZLio/github/jadarma/aockt/test/ExecMode;Lkotlin/time/Duration;Lkotlin/jvm/functions/Function1;)V
+	public static synthetic fun partTwo-51bEbmg$default (Lio/github/jadarma/aockt/test/AdventSpec;ZZLio/github/jadarma/aockt/test/ExecMode;Lkotlin/time/Duration;Lkotlin/jvm/functions/Function1;ILjava/lang/Object;)V
 	public final fun testCaseOrder ()Lio/kotest/core/test/TestCaseOrder;
 	public final fun threads ()Ljava/lang/Integer;
 }
 
 public abstract interface class io/github/jadarma/aockt/test/AdventSpec$PartScope {
-	public abstract fun shouldAllOutput (Ljava/lang/Iterable;Ljava/lang/Object;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
-	public abstract fun shouldOutput (Ljava/lang/String;Ljava/lang/Object;Lkotlin/coroutines/Continuation;)Ljava/lang/Object;
+	public abstract fun shouldAllOutput (Ljava/lang/Iterable;Ljava/lang/Object;)V
+	public abstract fun shouldOutput (Ljava/lang/String;Ljava/lang/Object;)V
 }
 
 public final class io/github/jadarma/aockt/test/AocKtExtension : kotlin/coroutines/AbstractCoroutineContextElement, io/kotest/core/extensions/DisplayNameFormatterExtension, io/kotest/core/extensions/TestCaseExtension {

aockt-test/src/main/kotlin/AdventSpec.kt

@@ -19,6 +19,7 @@ import io.github.jadarma.aockt.test.internal.id
 import io.github.jadarma.aockt.test.internal.partFunction
 import io.github.jadarma.aockt.test.internal.solutionToPart
 import io.kotest.assertions.failure
+import io.kotest.assertions.throwables.shouldNotThrowAny
 import io.kotest.assertions.withClue
 import io.kotest.common.ExperimentalKotest
 import io.kotest.core.concurrency.CoroutineDispatcherFactory
@@ -122,7 +123,7 @@ public abstract class AdventSpec<T : Solution>(
          * @receiver The example puzzle input.
          * @param expected The correct answer to the puzzle for the given input.
          */
-        public suspend infix fun String.shouldOutput(expected: Any)
+        public infix fun String.shouldOutput(expected: Any)
 
         /**
          * For each of the values given creates a new test that asserts that when given as input, it gets the correct
@@ -134,7 +135,7 @@ public abstract class AdventSpec<T : Solution>(
          * @receiver The example puzzle inputs.
          * @param expected The correct answer to the puzzle for all given inputs.
          */
-        public suspend infix fun Iterable<String>.shouldAllOutput(expected: Any)
+        public infix fun Iterable<String>.shouldAllOutput(expected: Any)
     }
 
     /**
@@ -160,7 +161,7 @@ public abstract class AdventSpec<T : Solution>(
         expensive: Boolean,
         executionMode: ExecMode?,
         efficiencyBenchmark: Duration?,
-        examples: (suspend PartScope.() -> Unit)?,
+        examples: (PartScope.() -> Unit)?,
     ) {
         if (efficiencyBenchmark != null && !efficiencyBenchmark.isPositive()) {
             throw ConfigurationException("Efficiency benchmark must be a positive value, but was: $efficiencyBenchmark")
@@ -190,7 +191,16 @@ public abstract class AdventSpec<T : Solution>(
 
             if (examples != null) {
                 context("Validates the examples").config(enabled = execMode != ExecMode.SkipExamples) {
-                    AdventSpecPartScope(partFunction, this).examples()
+                    AdventSpecPartScope().apply(examples).forEachIndexed { index, input, expected ->
+                        test("Example #${index + 1}") {
+                            withClue("Expected answer '$expected' for input: ${input.preview()}") {
+                                val answer = shouldNotThrowAny {
+                                    partFunction(input.toString()).toString()
+                                }
+                                answer shouldBe expected
+                            }
+                        }
+                    }
                 }
             }
 
@@ -267,7 +277,7 @@ public abstract class AdventSpec<T : Solution>(
         expensive: Boolean = false,
         executionMode: ExecMode? = null,
         efficiencyBenchmark: Duration? = null,
-        test: (suspend PartScope.() -> Unit)? = null,
+        test: (PartScope.() -> Unit)? = null,
     ): Unit = partTest(One, enabled, expensive, executionMode, efficiencyBenchmark, test)
 
     /**
@@ -291,6 +301,6 @@ public abstract class AdventSpec<T : Solution>(
         expensive: Boolean = false,
         executionMode: ExecMode? = null,
         efficiencyBenchmark: Duration? = null,
-        test: (suspend PartScope.() -> Unit)? = null,
+        test: (PartScope.() -> Unit)? = null,
     ): Unit = partTest(Two, enabled, expensive, executionMode, efficiencyBenchmark, test)
 }

aockt-test/src/main/kotlin/internal/AdventSpecPartScope.kt

@@ -1,41 +1,21 @@
 package io.github.jadarma.aockt.test.internal
 
 import io.github.jadarma.aockt.test.AdventSpec
-import io.kotest.assertions.throwables.shouldNotThrowAny
-import io.kotest.assertions.withClue
-import io.kotest.core.spec.style.scopes.ContainerScope
-import io.kotest.core.spec.style.scopes.FunSpecContainerScope
-import io.kotest.matchers.shouldBe
 
-/**
- * A [ContainerScope] specialized for testing the outputs of a specific function against predetermined inputs.
- *
- * @param implementation The function to be tested.
- * @param context The parent context in which to append these test cases.
- */
-internal class AdventSpecPartScope(
-    private val implementation: (String) -> Any,
-    private val context: FunSpecContainerScope,
-) : AdventSpec.PartScope, ContainerScope by context {
+/** A simple part scope implementation that builds a list of example inputs. */
+internal class AdventSpecPartScope : AdventSpec.PartScope {
 
-    /** Incrementing counter for the example test cases. */
-    private var exampleNumber: Int = 0
+    private val examples =  mutableListOf<Pair<PuzzleInput, String>>()
 
-    override suspend infix fun String.shouldOutput(expected: Any) {
-        exampleNumber++
-        val input = PuzzleInput(this)
-        val preview = input.preview()
-        context.test("Example #$exampleNumber") {
-            withClue("Expected answer '$expected' for input: $preview") {
-                val answer = shouldNotThrowAny {
-                    this@AdventSpecPartScope.implementation(input.toString()).toString()
-                }
-                answer shouldBe expected.toString()
-            }
-        }
+    override infix fun String.shouldOutput(expected: Any) {
+        examples.add(PuzzleInput(this) to expected.toString())
     }
 
-    override suspend infix fun Iterable<String>.shouldAllOutput(expected: Any) {
+    override infix fun Iterable<String>.shouldAllOutput(expected: Any) {
         forEach { it shouldOutput expected }
     }
+
+    /** Applies the [block] function for each registered example. */
+    inline fun forEachIndexed(block: (Int, PuzzleInput, String) -> Unit) =
+        examples.forEachIndexed { index, example -> block(index, example.first, example.second) }
 }