4.x: Enable disabled TCKs

dependencies/pom.xml

@@ -89,7 +89,7 @@
         <version.lib.jboss.classfilewriter>1.3.0.Final</version.lib.jboss.classfilewriter>
         <version.lib.jboss.logging>3.5.3.Final</version.lib.jboss.logging>
         <version.lib.jaxb-runtime>4.0.3</version.lib.jaxb-runtime>
-        <version.lib.jersey>3.1.3</version.lib.jersey>
+        <version.lib.jersey>3.1.4</version.lib.jersey>
         <version.lib.jgit>6.7.0.202309050840-r</version.lib.jgit>
         <version.lib.junit>5.9.3</version.lib.junit>
         <version.lib.kafka>3.6.0</version.lib.kafka>

http/http/src/main/java/io/helidon/http/PathMatchers.java

@@ -24,6 +24,7 @@
 import java.util.regex.Pattern;
 
 import io.helidon.common.parameters.Parameters;
+import io.helidon.common.uri.UriEncoding;
 import io.helidon.common.uri.UriPath;
 
 /**
@@ -276,8 +277,9 @@ static final class ExactPathMatcher implements PathMatcher {
         private final String pathWithTrailingSlash;
 
         ExactPathMatcher(String path) {
-            this.path = path;
-            this.pathWithTrailingSlash = path + "/";
+            // We work with decoded URIs
+            this.path = UriEncoding.decodeUri(path);
+            this.pathWithTrailingSlash = this.path + "/";
         }
 
         @Override

integrations/micrometer/cdi/src/test/java/io/helidon/integrations/micrometer/cdi/CatchAllExceptionMapper.java

@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) 2023 Oracle and/or its affiliates.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package io.helidon.integrations.micrometer.cdi;
+
+import java.lang.System.Logger.Level;
+
+import io.helidon.webserver.http.ServerRequest;
+
+import jakarta.annotation.Priority;
+import jakarta.ws.rs.WebApplicationException;
+import jakarta.ws.rs.core.Context;
+import jakarta.ws.rs.core.Response;
+import jakarta.ws.rs.ext.ExceptionMapper;
+import jakarta.ws.rs.ext.Provider;
+
+/**
+ * Default mapper for exceptions that are not caught.
+ */
+@Provider
+@Priority(5000)
+public class CatchAllExceptionMapper implements ExceptionMapper<Exception> {
+
+    private static final System.Logger LOGGER = System.getLogger(CatchAllExceptionMapper.class.getName());
+    @Context
+    private ServerRequest serverRequest;
+
+    @Override
+    public Response toResponse(Exception exception) {
+        serverRequest.context().register("unmappedException", exception);
+        if (exception instanceof WebApplicationException wae) {
+            return wae.getResponse();
+        } else {
+            LOGGER.log(Level.WARNING, () -> "Internal server error", exception);
+            return Response.serverError().build();
+        }
+    }
+}

integrations/micrometer/cdi/src/test/java/io/helidon/integrations/micrometer/cdi/HelloWorldTest.java

@@ -41,6 +41,7 @@
  */
 @HelidonTest
 @AddBean(HelloWorldResource.class)
+@AddBean(CatchAllExceptionMapper.class)
 public class HelloWorldTest {
 
     @Inject

microprofile/cdi/src/main/java/io/helidon/microprofile/cdi/HelidonContainerImpl.java

@@ -24,7 +24,6 @@
 import java.util.List;
 import java.util.Map;
 import java.util.ServiceLoader;
-import java.util.UUID;
 import java.util.concurrent.TimeUnit;
 import java.util.concurrent.atomic.AtomicBoolean;
 import java.util.logging.Handler;
@@ -46,6 +45,8 @@
 import jakarta.enterprise.context.BeforeDestroyed;
 import jakarta.enterprise.context.Destroyed;
 import jakarta.enterprise.context.Initialized;
+import jakarta.enterprise.event.Startup;
+import jakarta.enterprise.inject.Any;
 import jakarta.enterprise.inject.se.SeContainer;
 import jakarta.enterprise.inject.spi.BeanManager;
 import jakarta.enterprise.inject.spi.CDI;
@@ -96,6 +97,8 @@ final class HelidonContainerImpl extends Weld implements HelidonContainer {
     private static final Context ROOT_CONTEXT;
     // whether the current shutdown was invoked by the shutdown hook
     private static final AtomicBoolean FROM_SHUTDOWN_HOOK = new AtomicBoolean();
+    // Default Weld container id. TCKs assumes this value.
+    private static final String STATIC_INSTANCE = "STATIC_INSTANCE";
 
     static {
         HelidonFeatures.flavor(HelidonFlavor.MP);
@@ -119,7 +122,7 @@ final class HelidonContainerImpl extends Weld implements HelidonContainer {
 
     HelidonContainerImpl() {
         this.bootstrap = new WeldBootstrap();
-        id = UUID.randomUUID().toString();
+        this.id = STATIC_INSTANCE;
     }
 
     /**
@@ -205,6 +208,7 @@ public Collection<URL> getResources(String name) {
         }
         deployment.getServices().add(ExternalConfiguration.class, configurationBuilder.build());
 
+        // CDI TCK tests expects SE to be excluded, which means Helidon may require to do things that Weld is supposed to do.
         bootstrap.startContainer(id, Environments.SE, deployment);
 
         bootstrap.startInitialization();
@@ -315,6 +319,7 @@ private HelidonContainerImpl doStart() {
         keepLoggingActive(shutdownHook);
 
         bm.getEvent().select(Initialized.Literal.APPLICATION).fire(new ContainerInitialized(id));
+        bm.getEvent().select(Any.Literal.INSTANCE).fire(new Startup());
 
         now = System.currentTimeMillis() - now;
         LOGGER.fine("Container started in " + now + " millis (this excludes the initialization time)");

microprofile/config/src/main/java/io/helidon/microprofile/config/ConfigCdiExtension.java

@@ -46,6 +46,7 @@
 import jakarta.enterprise.context.Dependent;
 import jakarta.enterprise.context.spi.CreationalContext;
 import jakarta.enterprise.event.Observes;
+import jakarta.enterprise.inject.Vetoed;
 import jakarta.enterprise.inject.spi.AfterBeanDiscovery;
 import jakarta.enterprise.inject.spi.AfterDeploymentValidation;
 import jakarta.enterprise.inject.spi.Annotated;
@@ -136,7 +137,7 @@ private void processAnnotatedType(@Observes @WithAnnotations(ConfigProperties.cl
     private <X> void harvestConfigPropertyInjectionPointsFromEnabledObserverMethod(@Observes ProcessObserverMethod<?, X> event,
                                                                                    BeanManager beanManager) {
         AnnotatedMethod<X> annotatedMethod = event.getAnnotatedMethod();
-        if (annotatedMethod != null) {
+        if (annotatedMethod != null && !annotatedMethod.getDeclaringType().isAnnotationPresent(Vetoed.class)) {
             List<AnnotatedParameter<X>> annotatedParameters = annotatedMethod.getParameters();
             if (annotatedParameters != null) {
                 for (AnnotatedParameter<?> annotatedParameter : annotatedParameters) {

microprofile/lra/jax-rs/src/test/java/io/helidon/microprofile/lra/LoadBalancedCoordinatorTest.java

@@ -33,6 +33,7 @@
 import io.helidon.lra.coordinator.client.CoordinatorClient;
 import io.helidon.lra.coordinator.client.PropagatedHeaders;
 import io.helidon.microprofile.config.ConfigCdiExtension;
+import io.helidon.microprofile.lra.resources.CatchAllExceptionMapper;
 import io.helidon.microprofile.lra.resources.CdiNestedCompleteOrCompensate;
 import io.helidon.microprofile.lra.resources.CommonAfter;
 import io.helidon.microprofile.lra.resources.DontEnd;
@@ -95,6 +96,7 @@
 // LRA client
 @AddExtension(LraCdiExtension.class)
 // Test resources
+@AddBean(CatchAllExceptionMapper.class)
 @AddBean(JaxRsCompleteOrCompensate.class)
 @AddBean(NonJaxRsCompleteOrCompensate.class)
 @AddBean(NonJaxRsCompleteOrCompensateCS.class)

microprofile/lra/jax-rs/src/test/java/io/helidon/microprofile/lra/resources/CatchAllExceptionMapper.java

@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) 2023 Oracle and/or its affiliates.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package io.helidon.microprofile.lra.resources;
+
+import java.lang.System.Logger.Level;
+
+import io.helidon.webserver.http.ServerRequest;
+
+import jakarta.annotation.Priority;
+import jakarta.ws.rs.WebApplicationException;
+import jakarta.ws.rs.core.Context;
+import jakarta.ws.rs.core.Response;
+import jakarta.ws.rs.ext.ExceptionMapper;
+import jakarta.ws.rs.ext.Provider;
+
+/**
+ * Default mapper for exceptions that are not caught.
+ */
+@Provider
+@Priority(5000)
+public class CatchAllExceptionMapper implements ExceptionMapper<Exception> {
+
+    private static final System.Logger LOGGER = System.getLogger(CatchAllExceptionMapper.class.getName());
+    @Context
+    private ServerRequest serverRequest;
+
+    @Override
+    public Response toResponse(Exception exception) {
+        serverRequest.context().register("unmappedException", exception);
+        if (exception instanceof WebApplicationException wae) {
+            return wae.getResponse();
+        } else {
+            LOGGER.log(Level.WARNING, () -> "Internal server error", exception);
+            return Response.serverError().build();
+        }
+    }
+}

microprofile/metrics/src/test/java/io/helidon/microprofile/metrics/CatchAllExceptionMapper.java

@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) 2023 Oracle and/or its affiliates.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+package io.helidon.microprofile.metrics;
+
+import java.lang.System.Logger.Level;
+
+import io.helidon.webserver.http.ServerRequest;
+
+import jakarta.annotation.Priority;
+import jakarta.ws.rs.WebApplicationException;
+import jakarta.ws.rs.core.Context;
+import jakarta.ws.rs.core.Response;
+import jakarta.ws.rs.ext.ExceptionMapper;
+import jakarta.ws.rs.ext.Provider;
+
+/**
+ * Default mapper for exceptions that are not caught.
+ */
+@Provider
+@Priority(5000)
+public class CatchAllExceptionMapper implements ExceptionMapper<Exception> {
+
+    private static final System.Logger LOGGER = System.getLogger(CatchAllExceptionMapper.class.getName());
+    @Context
+    private ServerRequest serverRequest;
+
+    @Override
+    public Response toResponse(Exception exception) {
+        serverRequest.context().register("unmappedException", exception);
+        if (exception instanceof WebApplicationException wae) {
+            return wae.getResponse();
+        } else {
+            LOGGER.log(Level.WARNING, () -> "Internal server error", exception);
+            return Response.serverError().build();
+        }
+    }
+}

microprofile/metrics/src/test/java/io/helidon/microprofile/metrics/HelloWorldTest.java

@@ -24,6 +24,7 @@
 import java.util.regex.Pattern;
 import java.util.stream.IntStream;
 
+import io.helidon.microprofile.testing.junit5.AddBean;
 import io.helidon.microprofile.testing.junit5.AddConfig;
 import io.helidon.microprofile.testing.junit5.HelidonTest;
 
@@ -55,6 +56,7 @@
 @HelidonTest
 @AddConfig(key = "metrics." + MetricsCdiExtension.REST_ENDPOINTS_METRIC_ENABLED_PROPERTY_NAME, value = "true")
 @AddConfig(key = "metrics.permit-all", value = "true")
+@AddBean(CatchAllExceptionMapper.class)
 class HelloWorldTest {
 
     @Inject

microprofile/server/pom.xml

@@ -31,6 +31,11 @@
     <name>Helidon Microprofile Server</name>
     <description>Server of the microprofile implementation</description>
 
+    <properties>
+        <surefire.argLine>-Xmx1024m -Dfile.encoding=UTF-8 -Dorg.jboss.weld.construction.relaxed=false</surefire.argLine>
+        <failsafe.argLine>-Xmx1024m -Dfile.encoding=UTF-8 -Dorg.jboss.weld.construction.relaxed=false</failsafe.argLine>
+    </properties>
+
     <dependencies>
         <dependency>
             <groupId>io.helidon.config</groupId>

microprofile/server/src/main/java/io/helidon/microprofile/server/JaxRsCdiExtension.java

@@ -26,20 +26,14 @@
 import java.util.Set;
 import java.util.concurrent.atomic.AtomicBoolean;
 
-import io.helidon.webserver.http.ServerRequest;
-
 import jakarta.annotation.Priority;
 import jakarta.enterprise.context.ApplicationScoped;
 import jakarta.enterprise.context.Initialized;
 import jakarta.enterprise.event.Observes;
 import jakarta.enterprise.inject.spi.Extension;
 import jakarta.enterprise.inject.spi.ProcessManagedBean;
 import jakarta.ws.rs.Path;
-import jakarta.ws.rs.WebApplicationException;
 import jakarta.ws.rs.core.Application;
-import jakarta.ws.rs.core.Context;
-import jakarta.ws.rs.core.Response;
-import jakarta.ws.rs.ext.ExceptionMapper;
 import jakarta.ws.rs.ext.Provider;
 import org.eclipse.microprofile.config.Config;
 import org.eclipse.microprofile.config.ConfigProvider;
@@ -277,29 +271,11 @@ JaxRsService toJerseySupport(JaxRsApplication jaxRsApplication,
                                  InjectionManager injectionManager) {
 
         ResourceConfig resourceConfig = jaxRsApplication.resourceConfig();
-        resourceConfig.register(new CatchAllExceptionMapper());
 
         return JaxRsService.create(resourceConfig,
                                    injectionManager);
     }
 
-    @Provider
-    private static class CatchAllExceptionMapper implements ExceptionMapper<Exception> {
-        @Context
-        private ServerRequest serverRequest;
-
-        @Override
-        public Response toResponse(Exception exception) {
-            serverRequest.context().register("unmappedException", exception);
-            if (exception instanceof WebApplicationException wae) {
-                return wae.getResponse();
-            } else {
-                LOGGER.log(Level.WARNING, () -> "Internal server error", exception);
-                return Response.serverError().build();
-            }
-        }
-    }
-
     Optional<String> findContextRoot(io.helidon.config.Config config, JaxRsApplication jaxRsApplication) {
         return config.get(jaxRsApplication.appClassName() + "." + RoutingPath.CONFIG_KEY_PATH)
                 .asString()