Resource injection problem about jersey2-guice

[rkapsi]

Thank you. I see the problem. Do you mind if I incorporate your Test into the fix?

[abrandimarti]

No problem.

I have done a little debugging and it seems that Guice annotation type gets lost in GuiceBinder line 81

[rkapsi]

ServiceLocatorImpl lines 532-535

Guice's BindingAnnotation is not a JSR-330 Qualifier and HK2 ignores it. There is a Descriptor cache and it ends up using the GuiceBinder for the plain HelloService.

[rkapsi]

And you're correct as well. The at-Other annotation gets lost in GuiceBinder. But that's because at-Other isn't a JSR-330 Qualifier.

[abrandimarti]

I have added a new commit on my pull request. It's only an attempt that tries to fix the at-Other testcase.

Unfortunately it uses Guice internal static method Annotations.generateAnnotation(). This method has a precondition that the annotation must have methods with a default value. I wonder if Guice check Annotations.isAllDefaultMethods() should return true also in the case that the annotations has no methods?

By the way, it would be nice to have Travis to pass Gradle --continue to see all failures

Thanks
Alessandro

[abrandimarti]

Sorry, I have understood now: "at-Other isn't a JSR-330 Qualifier."
Please ignore my last comment and commit

[rkapsi]

Adding this fixes the problem:

3eb788d

Basically, you want to instantiate your HelloResource resource with Guice and not with HK2.

[abrandimarti]

Thanks Roger. I have tested 0.5-SNAPSHOT on my project and it works correctly.
Any chance to release 0.5 soon?

[rkapsi]

Done