<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Thu, May 9, 2013 at 11:10 PM, Mircea Markus <span dir="ltr"><<a href="mailto:mmarkus@redhat.com" target="_blank">mmarkus@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
On 9 May 2013, at 20:56, Dan Berindei wrote:<br>
<br>
> > Another alternative that come to my mind was to add a new method in<br>
> > AdvancedCache that returns the TimeService (and this I can mock it in<br>
> > the test suite)<br>
><br>
> +1<br>
><br>
><br>
> Couldn't you change CacheLoaderManager to call ComponentRegistry.wireDependencies(cacheStore)?<br>
><br>
> That way, each cache store could have a separate @Inject method, and it could depend on any cache-scoped or global-scoped component.<br>
> It may require an infinispan-module.properties file in each cache store module, but it then it could be used for any other component.<br>
</div>if you do ComponentRegistry.wireDependencies(cacheStore) any annotated method would get invoked, just curious why would it require an module.properties...<br>
</blockquote></div><br></div><div class="gmail_extra">Well, the component registry looks at the component metadata in the jar, so it needs a way to load the metadata for all the modules. But it looks like it doesn't use module.properties, it needs a file called META-INF/services/org.infinispan.factories.components.ModuleMetadataFileFinder instead. <br>
</div></div>