refactor: Simplify reflection logic in ComposeTestNodeProvider

- The `reflect` function has been refactored to simplify the process of accessing the internal `ComposeUiTest` from a `ComposeContentTestRule`.
- Removed the separate `findField` and `getOrNull` helper functions.
- The reflection logic is now consolidated directly within the `reflect` function, using a `try-catch` block to handle potential `NoSuchFieldException` and `IllegalAccessException`.

Author: softartdev

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

@@ -5,33 +5,19 @@ package com.softartdev.notedelight
 import androidx.compose.ui.test.ComposeUiTest
 import androidx.compose.ui.test.ExperimentalTestApi
 import androidx.compose.ui.test.junit4.ComposeContentTestRule
-import java.lang.reflect.Field
 
 fun reflect(composeTestRule: ComposeContentTestRule): ComposeUiTest {
-    findField(composeTestRule, "composeTest")?.let { field ->
-        field.getOrNull(composeTestRule)?.let { value ->
-            if (value is ComposeUiTest) return value
-        }
-    }
-    throw IllegalStateException(
-        "Failed to reflect ComposeUiTest from ${composeTestRule.javaClass.name}. " +
-                "Expected AndroidComposeTestRule/DesktopComposeTestRule internals to contain it."
-    )
-}
-
-@Suppress("SameParameterValue")
-private fun findField(target: Any, name: String): Field? {
-    var c: Class<*>? = target.javaClass
+    var c: Class<*>? = composeTestRule.javaClass
     while (c != null && c != Any::class.java) {
-        runCatching { c.getDeclaredField(name) }
-            .onSuccess { return it }
+        try {
+            val f = c.getDeclaredField("composeTest")
+            f.isAccessible = true
+            val v = f.get(composeTestRule)
+            if (v is ComposeUiTest) return v
+        } catch (_: NoSuchFieldException) {
+        } catch (_: IllegalAccessException) {
+        }
         c = c.superclass
     }
-    return null
+    throw IllegalStateException("ComposeUiTest not found")
 }
-
-private fun Field.getOrNull(instance: Any): Any? =
-    runCatching {
-        isAccessible = true
-        get(instance)
-    }.getOrNull()