Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WFLY-18702] Ensure the wildfly-preview feature pack's jaxrs-server layer can provision MicroProfile REST Client #17349

Merged
merged 3 commits into from Oct 30, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Expand Up @@ -4,7 +4,10 @@
~ SPDX-License-Identifier: Apache-2.0
-->

<layer-spec xmlns="urn:jboss:galleon:layer-spec:1.0" name="jaxrs-server">
<layer-spec xmlns="urn:jboss:galleon:layer-spec:2.0" name="jaxrs-server">
<props>
<prop name="org.wildfly.rule.kind" value="base-layer"/>
</props>
<dependencies>
<layer name="datasources-web-server"/>
<!-- cloud-server depends on jaxrs-server, jaxrs could be excluded from cloud-server,
Expand All @@ -13,5 +16,12 @@
<layer name="bean-validation" optional="true"/>
<layer name="cdi" optional="true"/>
<layer name="jpa" optional="true"/>
<!-- The above content is copied from the wildfly-ee feature pack variant of this file.
In standard WildFly the wildfly feature pack also declares this layer, but since that FP
depends on wildfly-ee, its minimal content gets applied 'on top' of what wildfly-ee
does. But the wildfly-preview feature pack does not depend on wildfly-ee, so we need
to declare here both the 'wildfly-ee' content and the 'wildfly' content. This next
bit is the 'wildfly' content. -->
<layer name="microprofile-rest-client" optional="true"/>
</dependencies>
</layer-spec>
11 changes: 0 additions & 11 deletions testsuite/integration/microprofile-tck/rest-client/pom.xml
Expand Up @@ -595,17 +595,6 @@
</execution>
</executions>
</plugin>
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
<executions>
<!--Disable the default surefire execution until WFLY-14136 is sorted. -->
<execution>
<id>default-test</id>
<phase>none</phase>
</execution>
</executions>
</plugin>
</plugins>
</build>
</profile>
Expand Down
7 changes: 4 additions & 3 deletions testsuite/integration/microprofile/pom.xml
Expand Up @@ -575,14 +575,14 @@
</build>
</profile>

<!-- Test against the ee 9 feature pack -->
<!-- Test against WildFly Preview dist -->
<!-- TODO convert this to a bootable jar setup once a release
of the wildfly jar plugin with transformation support is available -->
<profile>
<id>preview.test.profile</id>
<activation>
<property>
<name>DISABLED.ts.preview</name>
<name>ts.preview</name>
</property>
</activation>
<properties>
Expand Down Expand Up @@ -661,6 +661,7 @@
<layer>cloud-server</layer>
<layer>microprofile-platform</layer>
<layer>ejb</layer>
<layer>micrometer</layer>
<!-- Dependencies for reactive specs -->
<layer>microprofile-reactive-messaging-kafka</layer>
<layer>h2-default-datasource</layer>
Expand Down Expand Up @@ -888,7 +889,7 @@
<id>bootablejar.preview.profile</id>
<activation>
<property>
<name>DISABLED.ts.bootable.preview</name>
<name>ts.bootable.preview</name>
</property>
</activation>
<dependencies>
Expand Down
4 changes: 2 additions & 2 deletions testsuite/integration/pom.xml
Expand Up @@ -320,7 +320,7 @@
<modules>
<!--<module>basic</module>-->
<!--<module>clustering</module>-->
<!--<module>microprofile</module>-->
<module>microprofile</module>
<module>microprofile-tck</module>
<module>elytron-oidc-client</module>
</modules>
Expand All @@ -339,7 +339,7 @@
<module>legacy</module>
<module>ws</module>
<!--<module>clustering</module>-->
<!--<module>microprofile</module>-->
<module>microprofile</module>
<module>microprofile-tck</module>
<module>secman</module>
<module>elytron</module>
Expand Down