refactor: Simplify reflection in ComposeTestNodeProvider

- Remove robust and last-resort reflection paths for finding `ComposeUiTest`.
- The code now only attempts to find the `ComposeUiTest` instance in a field named "composeTest".
- Delete unused private helper functions: `allFields` and `allNoArgMethods`.

Author: softartdev

ui/test-jvm/src/main/kotlin/com/softartdev/notedelight/ComposeTestNodeProvider.kt

@@ -6,35 +6,13 @@ import androidx.compose.ui.test.ComposeUiTest
 import androidx.compose.ui.test.ExperimentalTestApi
 import androidx.compose.ui.test.junit4.ComposeContentTestRule
 import java.lang.reflect.Field
-import java.lang.reflect.Method
 
 fun reflect(composeTestRule: ComposeContentTestRule): ComposeUiTest {
-    // Just in case some wrapper directly implements ComposeUiTest.
-    (composeTestRule as? ComposeUiTest)?.let { return it }
-
-    // 1) Fast path: both AndroidComposeTestRule and DesktopComposeTestRule store it in "composeTest".
     findField(composeTestRule, "composeTest")?.let { field ->
         field.getOrNull(composeTestRule)?.let { value ->
             if (value is ComposeUiTest) return value
         }
     }
-
-    // 2) Robust path: scan all fields in the hierarchy and return the first ComposeUiTest found.
-    for (field in allFields(composeTestRule.javaClass)) {
-        field.getOrNull(composeTestRule)?.let { value ->
-            if (value is ComposeUiTest) return value
-        }
-    }
-
-    // 3) Last resort: scan no-arg methods returning ComposeUiTest.
-    for (method in allNoArgMethods(composeTestRule.javaClass)) {
-        if (ComposeUiTest::class.java.isAssignableFrom(method.returnType)) {
-            method.isAccessible = true
-            val value = runCatching { method.invoke(composeTestRule) }.getOrNull()
-            if (value is ComposeUiTest) return value
-        }
-    }
-
     throw IllegalStateException(
         "Failed to reflect ComposeUiTest from ${composeTestRule.javaClass.name}. " +
                 "Expected AndroidComposeTestRule/DesktopComposeTestRule internals to contain it."
@@ -52,17 +30,6 @@ private fun findField(target: Any, name: String): Field? {
     return null
 }
 
-private fun allFields(clazz: Class<*>): Sequence<Field> =
-    generateSequence(clazz) { it.superclass }
-        .takeWhile { it != Any::class.java }
-        .flatMap { it.declaredFields.asSequence() }
-
-private fun allNoArgMethods(clazz: Class<*>): Sequence<Method> =
-    generateSequence(clazz) { it.superclass }
-        .takeWhile { it != Any::class.java }
-        .flatMap { it.declaredMethods.asSequence() }
-        .filter { it.parameterTypes.isEmpty() }
-
 private fun Field.getOrNull(instance: Any): Any? =
     runCatching {
         isAccessible = true