Skip to content
This repository was archived by the owner on May 28, 2018. It is now read-only.

Commit 12a9259

Browse files
japodGerrit Code Review
authored andcommitted
Merge "Fixed org.glassfish.jersey.tests.cdi.resources.CustomInjectionTest.testCustomHk2Injection1 test in tests/integration/cdi-with-jersey-injection-custom-cfg-webapp"
2 parents 90e952a + 0163588 commit 12a9259

File tree

1 file changed

+12
-9
lines changed

1 file changed

+12
-9
lines changed

ext/cdi/jersey-cdi1x/src/main/java/org/glassfish/jersey/ext/cdi1x/internal/CdiComponentProvider.java

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -495,15 +495,18 @@ public boolean isAnnotationPresent(final Class<? extends Annotation> annotationT
495495

496496
@SuppressWarnings("unused")
497497
private void processAnnotatedType(@Observes
498-
@WithAnnotations({
499-
Context.class,
500-
ApplicationPath.class,
501-
HeaderParam.class,
502-
QueryParam.class,
503-
FormParam.class,
504-
MatrixParam.class,
505-
BeanParam.class,
506-
PathParam.class})
498+
// We can not apply the following constraint
499+
// if we want to fully support {@link org.glassfish.jersey.ext.cdi1x.spi.Hk2CustomBoundTypesProvider}.
500+
// Covered by tests/integration/cdi-with-jersey-injection-custom-cfg-webapp test application:
501+
// @WithAnnotations({
502+
// Context.class,
503+
// ApplicationPath.class,
504+
// HeaderParam.class,
505+
// QueryParam.class,
506+
// FormParam.class,
507+
// MatrixParam.class,
508+
// BeanParam.class,
509+
// PathParam.class})
507510
final ProcessAnnotatedType processAnnotatedType) {
508511
final AnnotatedType<?> annotatedType = processAnnotatedType.getAnnotatedType();
509512

0 commit comments

Comments
 (0)