You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tomee.apache.org by "Romain Manni-Bucau (JIRA)" <ji...@apache.org> on 2016/10/13 11:34:20 UTC
[jira] [Resolved] (TOMEE-1958) Mockito doesn't @MockInjector to be
set as container properties
[ https://issues.apache.org/jira/browse/TOMEE-1958?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Romain Manni-Bucau resolved TOMEE-1958.
---------------------------------------
Resolution: Fixed
Assignee: Romain Manni-Bucau
Fix Version/s: 7.0.2
I understand now,
issue was not about @MockInjector but mockito CDI integration which was executed before the mock injector was used which was causing not yet created references.
Fixed mockito integration to support that.
> Mockito doesn't @MockInjector to be set as container properties
> ---------------------------------------------------------------
>
> Key: TOMEE-1958
> URL: https://issues.apache.org/jira/browse/TOMEE-1958
> Project: TomEE
> Issue Type: Improvement
> Reporter: Xavier Dury
> Assignee: Romain Manni-Bucau
> Priority: Trivial
> Fix For: 7.0.2
>
>
> Currently, when using {{ApplicationComposer}}, {{@MockInjector}} can be declared in two ways:
> {code}
> public @MockInjector FallbackPropertyInjector injector() {
> return new MockitoInjector();
> }
> public @MockInjector Class<? extends FallbackPropertyInjector> injectorClass() {
> return MockitoInjector.class;
> }
> {code}
> As almost everything can now be declared with annotations on the test class ({{@ContainerProperties}} = {{@Configuration}}, {{@Classes}} ~ {{@Module}}), it would be nice if {{@MockInjector}} could be too:
> {code}
> @RunWith(ApplicationComposer.class)
> @MockInjector(MockitoInjector.class)
> @Classes(...)
> public class MyTest {...}
> {code}
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)