Mattia Tommasone comments

Results 145 comments of


                                            Mattia Tommasone

Fatal exception in coroutines machinery

This really looks like an issue with kotlin coroutines, not with mockk.

getLabel is deprecated. Use isRequired or isRepeated instead.

Thanks for submitting this: it will be fixed in the next release of the client library.

getLabel is deprecated. Use isRequired or isRepeated instead.

V20 of the Google Ads API will be released on June 6th, so we will be releasing a new version of the client library along with it.

Updating from 1.14.0 to 1.14.2 introduces problem mocking a Kotlin class

@sgerke-1L @marcelstoer this looks like a regression introduced by the switch from `decorate` to `redefine`.

Updating from 1.14.0 to 1.14.2 introduces problem mocking a Kotlin class

Yep, I agree with @marcelstoer, so you have official maintainer approval to add an arrow dependency in the client-tests module :)

Improve Unecessary Stubbings error reporting to map it to a specific test

Are you using the `@MockKExtension` to check for unnecessary stubbing? How is your test lifecycle configured? By default, in JUnit 5 the lifecycle is configured as `TestInstance.Lifecycle.PER_METHOD`, which calls `checkUnnecessaryStub`...

Improve Unecessary Stubbings error reporting to map it to a specific test

Can you please try explicitly configuring your JUnit lifecycle to use `TestInstance.Lifecycle.PER_METHOD`? Thanks!

add test reports to GitHub actions

@aSemy, can you please resolve the conflicts?

fix: prevent confirmVerified(vararg mocks) from clearing unrelated mocks

Wild guess: could it be that `verify` with more than one call, such as the one in the `confirmVerifiedAll` test, don't add the right values to the list of verified...

fix: prevent confirmVerified(vararg mocks) from clearing unrelated mocks

That's interesting - I'd do more investigation to understand who's clearing the verifiedCalls list, when and why :)