Merge pull request #922 from aSemy/patch-1

value class check - catch KotlinReflectionInternalError

modules/mockk-core/src/jvmMain/kotlin/io/mockk/core/ValueClassSupport.kt

@@ -6,6 +6,7 @@ import kotlin.reflect.full.declaredMemberProperties
 import kotlin.reflect.full.primaryConstructor
 import kotlin.reflect.jvm.isAccessible
 import kotlin.reflect.jvm.javaField
+import kotlin.reflect.jvm.internal.KotlinReflectionInternalError
 
 
 actual object ValueClassSupport {
@@ -61,6 +62,8 @@ actual object ValueClassSupport {
     private val <T : Any> KClass<T>.isValue_safe: Boolean
         get() = try {
             this.isValue
+        } catch (_: KotlinReflectionInternalError) {
+            false
         } catch (_: UnsupportedOperationException) {
             false
         }

test-modules/client-tests/build.gradle.kts

@@ -3,7 +3,9 @@ plugins {
 }
 
 kotlin {
-    jvm()
+    jvm {
+        withJava()
+    }
 
     sourceSets {
         val commonMain by getting {

test-modules/client-tests/src/jvmTest/java/io/mockk/core/JavaEnum.java

@@ -0,0 +1,10 @@
+package io.mockk.core;
+
+/** @see ValueClassSupport */
+public enum JavaEnum {
+    A {
+        public String toString() {
+            return "aaa";
+        }
+    }
+}

test-modules/client-tests/src/jvmTest/kotlin/io/mockk/core/ValueClassSupportTest.kt

@@ -0,0 +1,27 @@
+package io.mockk.core
+
+import io.mockk.every
+import io.mockk.mockk
+import kotlin.test.Test
+import kotlin.test.assertEquals
+
+
+class ValueClassSupportTest {
+
+    /** https://github.com/mockk/mockk/issues/868 */
+    @Test
+    fun `verify Java class does not cause KotlinReflectionInternalError`() {
+        val mock = mockk<MockTarget> {
+            every { func() } returns JavaEnum.A
+        }
+
+        val result = mock.func() // check this doesn't throw KotlinReflectionInternalError
+
+        assertEquals(JavaEnum.A, result)
+    }
+
+}
+
+private class MockTarget {
+    fun func(): JavaEnum = JavaEnum.A
+}