spyk broken for class hierarchies about mockk

[oleksiyp]

Thanks for reporting. Let me check this example.

[oleksiyp]

Okay, so this to be fixed in 1.5.2 soon.

There is a few points mixed here:

1. The correct way to create a spy is just spyk(SpyTest()) to copy everything from an object or spyk<SpyTest>() to call a default constructor (specifying different than object type type parameter looks like error prone. will check how to make it better)
2. just Runs.apply{println("Intercepted")} just does not work. just Runs pretty synthetic construct which actually do nothing on response. println is executed but at declaration time, not at interception time. Is it confusing?
3. There is a bug in spy initialization code that doesn't take into account base classes (fixed, thanks for pointing out)

Besides fixing a bug I will check spyk type parameter and introduce way to intercept spy calls(this actually a bug as well):

        every {
            spyObj.doLog()
        } answers {
            println("Intercepted");
            callOriginal();
            nothing
        }

Documentation update would be nice to do as well.

[oleksiyp]

@dan-van can you check 1.5.2 version?

Something alike this should work:

        open class BaseTest(val someReference:String) {
            private var logger: org.slf4j.Logger = LoggerFactory.getLogger(BaseTest::class.java)
            open fun doLog() {
                logger.info("Base $someReference")
            }
        }

        class SpyTest() : BaseTest("A spy"){
            private var logger: org.slf4j.Logger = LoggerFactory.getLogger(SpyTest::class.java)
            override fun doLog() {
                logger.info("spyTest")
            }
        }

        val spyObj = spyk(SpyTest())
        every {
            spyObj.doLog()
        } answers {
            println("Intercepted");
            callOriginal();
        }

        assert(spyObj.someReference != null)
        spyObj.doLog()

[oleksiyp]

Looks like issue is resolved. @dan-van Thanks for reporting again

[dan-van]

Many thanks - sorry I was "away from project" for a few days. Yes 1.5.3 works for my usecase.