Macro expansion failure with HigherKinds

I use helpers in my tests to keep things clean such as:

def whenGetById[F[_]](implicit algebra: MyAlgebra[F]): OngoingStubbing[F[Either[MyError, MyResult]]] =
    when(algebra.get(anyVal[My.Id])).delegate

// or on later versions
def whenGetById[F[_]](implicit algebra: MyAlgebra[F]): OngoingStubbing[F[Either[MyError, MyResult]]] =
    when(algebra.get(any[My.Id])).delegate

This worked fine from 0.4.4 (obviously without the delegate call) to 1.0.2. However upon upgrading to any version past 1.0.3 my tests now fail to compile:

[error] /***/MockServices.scala:15:46: exception during macro expansion: 
[error] scala.ScalaReflectionException: type F is not a class
[error]         at scala.reflect.api.Symbols$SymbolApi.asClass(Symbols.scala:272)
[error]         at scala.reflect.api.Symbols$SymbolApi.asClass$(Symbols.scala:272)
[error]         at scala.reflect.internal.Symbols$SymbolContextApiImpl.asClass(Symbols.scala:94)
[error]         at org.mockito.internal.ValueClassExtractor$.materialise(ValueClassExtractor.scala:36)
[error]     when[F[Either[MyError, MyResult]]](merchants.get(anyVal[My.Id])).delegate

Currently I have to work around this with the following method, which allows me to use 1.0.5, but I lost any benefits of using the ScalaOngoingStubbing

private def hkWhen[T](methodCall: T): OngoingStubbing[T] = Mockito.when(methodCall)

I presume this is related to the ValueClassExtractor requirement added to when, which does not work for the F HigherKind.

Moreover, since version 1.0.2? my tests have started spitting out dead code errors on when conditions with value classes – despite stating the types.

[bbonanno]

@RawToast Thanks for reporting this, v1.0.6 should fix it, please let me know otherwise

Regarding the dead code errors, I don’t have the flag enabled as most of the code throws it so it’s too much noise for me, but I can take a look as in theory it shouldn’t happen if you specify the types, could you please create a separate issue with a full example that I can copy and paste to reproduce the problem?

Thanks

[bbonanno]

@RawToast I see, I got confused cause in my understanding, an algebra is a group of case classes and/or instances of sealed traits, which don’t make a great candidate for mocking, but in your case, you’re mocking services, which is what mocks are for 😃

I’ll have a look to see if this could be built in a more generic way, as here everything but the Either seems configurable