stuebingerb
stuebingerb
I have disabled caching completely in https://github.com/stuebingerb/mockk/actions/runs/3014589035 and the tests still run fine. I believe it might be an issue with the jvm target not being set properly.
I'm not sure if I have already fully understood the build, but to me it seems that the [JVM toolchain](https://github.com/mockk/mockk/blob/master/buildSrc/src/main/kotlin/buildsrc/convention/toolchain-jvm.gradle.kts#L12-L22) uses it's values from `Deps.Versions`: ``` tasks.withType().configureEach { options.encoding =...
There's also a `javaToolchainMainVersion` and a `javaToolchainTestVersion` but at first glance I only see the test version change in the matrix (which might be the reason for [this error](https://github.com/stuebingerb/mockk/runs/8248401003?check_suite_focus=true#step:5:298)): ```...
Currently looking into a potential open issue: although the mockk tests succeed, our own actual tests fail.
With the changes from https://github.com/mockk/mockk/pull/916/commits/5f70333ca4b5f1220727254d98ba12493f0f7b2b our own tests are now running successfully. Unfortunately, my knowledge of mockk is too limited to make much sense out of it so I would...
> Would you be able to add a test using the example that failed? (A sealed class with an abstract child?) Tried, but that test also succeeded without my modifications....
This is the relevant stack trace from our code (slightly anonymized), if it helps (we use [Koin](https://insert-koin.io/)): ``` Caused by: org.koin.core.error.InstanceCreationException: Could not create instance for [Singleton:'org.example.SomeProvider'] at org.koin.core.instance.InstanceFactory.create(InstanceFactory.kt:61) at...
> Yes the stacktrace helps, thanks, although I can't figure out exactly what's wrong. At least I think it shows that we're on the right path! I don't see any...
Sorry for commit spam. The current implementation here now seems to fix both, mockk's tests and our own. Please kindly review to spot anything I might have missed. (I've also...
So... I tried to add a new test case based on our code and I'm claiming that something in the CI test matrix is currently broken. Have a look at...