Support @InjectSpy on javax.inject.Singleton classes

[davidfrickert]

Description

Currently, if I try to @InjectSpy a javax.inject.Singleton class, like javax.transaction.TransactionManager, I get the error:

Invalid use of io.quarkus.test.junit.mockito.InjectSpy - the injected bean does not declare a CDI normal scope but: javax.inject.Singleton

By checking this issue: #8983, it seems that @InjectMock has a param convertScopes that addresses this issue. Could this also work on InjectSpy?

I wanted to do something like this in my tests:

// (...)
verify(transactionManagerSpy).setRollbackOnly();
//

Implementation ideas

No response

[quarkus-bot]

/cc @geoand (testing)

[davidfrickert]

I solved my problem by creating a wrapper bean @ApplicationScoped that injects the TransactionManager. Then I can @InjectSpy it and verify that the method is being called.
But it should still be nice to have a proper way of validating that the transaction was committed or rolled back.

[geoand]

Good idea!

#30623 will take care of it

[davidfrickert]

thanks for the quick fix @geoand !

[geoand]

YW