-
Notifications
You must be signed in to change notification settings - Fork 179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
"Packages and file facades are not yet supported in Kotlin reflection. Meanwhile please use Java reflection to inspect this class" #682
Comments
Interesting, the exception happens here: https://github.com/spekframework/spek/blob/2.x/spek-runner/junit5/src/main/kotlin/org/spekframework/spek2/junit/SpekTestEngine.kt#L52. What kotlin and JVM version are you using? |
Kotlin version is 1.3.31. Java version:
|
Most probably the JVM version, I'm using JDK10 at home. Is JDK 8 a requirement for you? Is it possible to use JDK 10 but set the compilation target to Java 8? |
Just tried with JDK 11 and got the same error. Anything else you'd like me to try? |
All good, thanks for the help! I'll try to replicate it. |
@charleskorn can you try dev build: |
Tried that with https://github.com/charleskorn/kaml and now I'm getting:
Can I suggest adding some unit tests around this code? It seems to be a recurring source of issues. |
I added some tests before but it does not catch all cases. Can you show me which class is causing the problem? |
It looks like it's the companion to |
I'll post in another build once #687 is merged. So sorry for the trouble! |
Build finished, can you try |
Looks like it's working, thanks! |
Tests are failed to run because of spekframework/spek#682
I've just tried to update https://github.com/charleskorn/kaml to use Spek 2.0.3 and I'm getting this error when running the tests through Gradle:
AssertionsKt is a reference to this file which doesn't contain a class.
The text was updated successfully, but these errors were encountered: