Issue oshai#484 - update KLoggerNameResolver tests for jvm

Author: scrat98

src/javaMain/kotlin/io/github/oshai/kotlinlogging/internal/KLoggerNameResolver.kt

@@ -1,13 +1,17 @@
 package io.github.oshai.kotlinlogging.internal
 
-import java.lang.reflect.Modifier
+import kotlin.reflect.KClass
 
 /** Resolves name of java classes */
 internal actual object KLoggerNameResolver {
 
   /** get class name for function by the package of the function */
   internal actual fun name(func: () -> Unit): String {
-    return func::class.java.name.toCleanClassName()
+    return name(func::class)
+  }
+
+  internal fun name(clazz: KClass<*>): String {
+    return clazz.java.name.toCleanClassName()
   }
 
   private val classNameEndings = listOf("Kt$", "$")
@@ -21,26 +25,4 @@ internal actual object KLoggerNameResolver {
     }
     return this
   }
-
-  /** get class name for java class (that usually represents kotlin class) */
-  internal fun <T : Any> name(forClass: Class<T>): String = unwrapCompanionClass(forClass).name
-
-  /** unwrap companion class to enclosing class given a Java Class */
-  private fun <T : Any> unwrapCompanionClass(clazz: Class<T>): Class<*> {
-    return clazz.enclosingClass?.let { enclosingClass ->
-      try {
-        enclosingClass.declaredFields
-          .find { field ->
-            field.name == clazz.simpleName &&
-              Modifier.isStatic(field.modifiers) &&
-              field.type == clazz
-          }
-          ?.run { enclosingClass }
-      } catch (se: SecurityException) {
-        // The security manager isn't properly set up, so it won't be possible
-        // to search for the target declared field.
-        null
-      }
-    } ?: clazz
-  }
 }

src/jvmTest/kotlin/io/github/oshai/kotlinlogging/internal/KLoggerNameResolverTest.kt

@@ -6,49 +6,46 @@ import org.junit.jupiter.api.TestInstance
 import org.junit.jupiter.params.ParameterizedTest
 import org.junit.jupiter.params.provider.Arguments
 import org.junit.jupiter.params.provider.MethodSource
+import kotlin.reflect.KClass
 
 @TestInstance(TestInstance.Lifecycle.PER_CLASS)
 class KLoggerNameResolverTest {
 
   @ParameterizedTest
   @MethodSource("testNames")
-  fun testNames(expectedName: String, clazz: Class<*>) {
+  fun testNames(expectedName: String, clazz: KClass<*>) {
     assertEquals(expectedName, KLoggerNameResolver.name(clazz))
   }
 
   private fun testNames(): Stream<Arguments> =
     Stream.of(
-      Arguments.of("io.github.oshai.kotlinlogging.internal.BaseClass", BaseClass::class.java),
-      Arguments.of("io.github.oshai.kotlinlogging.internal.ChildClass", ChildClass::class.java),
+      Arguments.of("io.github.oshai.kotlinlogging.internal.BaseClass", BaseClass::class),
+      Arguments.of("io.github.oshai.kotlinlogging.internal.ChildClass", ChildClass::class),
       Arguments.of(
         "io.github.oshai.kotlinlogging.internal.BaseClass",
-        BaseClass.Companion::class.java,
+        BaseClass.Companion::class,
       ),
       Arguments.of(
         "io.github.oshai.kotlinlogging.internal.ChildClass",
-        ChildClass.Companion::class.java,
+        ChildClass.Companion::class,
       ),
-      Arguments.of("io.github.oshai.kotlinlogging.internal.Singleton", Singleton::class.java),
-      Arguments.of("io.github.oshai.kotlinlogging.internal.MyInterface", MyInterface::class.java),
-      Arguments.of("java.lang.Object", Any().javaClass),
+      Arguments.of("io.github.oshai.kotlinlogging.internal.Singleton", Singleton::class),
+      Arguments.of("io.github.oshai.kotlinlogging.internal.MyInterface", MyInterface::class),
+      Arguments.of("java.lang.Object", Any()::class),
       Arguments.of(
-        "io.github.oshai.kotlinlogging.internal.KLoggerNameResolverTest\$testNames$1",
-        object {}.javaClass,
+        "io.github.oshai.kotlinlogging.internal.KLoggerNameResolverTest",
+        object {}::class,
       ),
       Arguments.of(
-        "io.github.oshai.kotlinlogging.internal.BaseClass\$InnerClass\$Obj",
-        BaseClass.InnerClass.Obj::class.java,
-      ),
-      Arguments.of(
-        "io.github.oshai.kotlinlogging.internal.BaseClass\$InnerClass\$Obj",
-        BaseClass.InnerClass.Obj.javaClass,
+        "io.github.oshai.kotlinlogging.internal.BaseClass",
+        BaseClass.InnerClass.Obj::class,
       ),
       Arguments.of(
-        "io.github.oshai.kotlinlogging.internal.BaseClass\$InnerClass",
-        BaseClass.InnerClass.CmpObj::class.java,
+        "io.github.oshai.kotlinlogging.internal.BaseClass",
+        BaseClass.InnerClass.CmpObj::class,
       ),
-      Arguments.of("io.github.oshai.kotlinlogging.internal.Foo\$Bar", Foo.Bar::class.java),
-      Arguments.of("io.github.oshai.kotlinlogging.internal.Foo\$Bar2", Foo.Bar3.javaClass),
+      Arguments.of("io.github.oshai.kotlinlogging.internal.Foo", Foo.Bar::class),
+      Arguments.of("io.github.oshai.kotlinlogging.internal.Foo", Foo.Bar3::class),
       Arguments.of(
         "io.github.oshai.kotlinlogging.internal.PrivateCompanion",
         PrivateCompanion().companionClass,
@@ -88,7 +85,7 @@ class Foo {
 }
 
 class PrivateCompanion {
-  val companionClass: Class<*> = Companion::class.java
+  val companionClass: KClass<*> = Companion::class
 
   private companion object
 }