Update Module provided recipes to use the provided scope (#725)

Co-authored-by: cjobinab <[email protected]>

Author: cjobinabo

src/main/resources/META-INF/rewrite/ibm-java.yml

@@ -117,30 +117,33 @@ recipeList:
 ---
 type: specs.openrewrite.org/v1beta/recipe
 name: org.openrewrite.java.migrate.RemovedJavaXMLWSModuleProvided
-displayName: Do not use `java.xml.ws` module in WebSphere Liberty
+displayName: Do not package `java.xml.ws` module in WebSphere Liberty applications
 description:
   The `java.xml.ws` module was removed in Java11. Websphere Liberty provides its own implementation of the module, which can be used by specifying the `jaxws-2.2` feature in the server.xml file.
-  This recipe removes the `javax.xml.ws` module from the application's build dependency in favor of the Websphere Liberty implementation to avoid class loading issues.
+  This recipe updates the `javax.xml.ws` dependency to use the `provided` scope to avoid class loading issues.
 tags:
   - java11
 recipeList:
-  - org.openrewrite.java.dependencies.RemoveDependency:
+  - org.openrewrite.maven.ChangeDependencyScope:
       groupId: javax.xml.ws
       artifactId: jaxws-api
+      newScope: provided
 ---
 type: specs.openrewrite.org/v1beta/recipe
 name: org.openrewrite.java.migrate.RemovedJaxBModuleProvided
-displayName: Do not use `java.xml.bind` and `java.activation` modules in WebSphere Liberty
+displayName: Do not package `java.xml.bind` and `java.activation` modules in WebSphere Liberty applications
 description:
   The `java.xml.bind` and `java.activation` modules were removed in Java11.
   Websphere Liberty provides its own implementation of the modules, which can be used by specifying the `jaxb-2.2` feature in the server.xml file.
-  This recipe removes the `javax.xml.bind` and `javax.activation` modules from the application's build dependency in favor of the Websphere Liberty implementation to avoid class loading issues.
+  This recipe updates the `javax.xml.bind` and `javax.activation` dependencies to use the `provided` scope to avoid class loading issues.
 tags:
   - java11
 recipeList:
-  - org.openrewrite.java.dependencies.RemoveDependency:
+  - org.openrewrite.maven.ChangeDependencyScope:
       groupId: javax.xml.bind
       artifactId: jaxb-api
-  - org.openrewrite.java.dependencies.RemoveDependency:
+      newScope: provided
+  - org.openrewrite.maven.ChangeDependencyScope:
       groupId: javax.activation
       artifactId: activation
+      newScope: provided

src/test/java/org/openrewrite/java/migrate/IBMSemeruTest.java

@@ -382,6 +382,12 @@ void removeMavenXMLWSModuleDependency() {
                     <artifactId>gson</artifactId>
                     <version>2.10.1</version>
                   </dependency>
+                  <dependency>
+                        <groupId>javax.xml.ws</groupId>
+                        <artifactId>jaxws-api</artifactId>
+                        <version>2.2</version>
+                    <scope>provided</scope>
+                  </dependency>
                 </dependencies>
                 <build>
                   <plugins>
@@ -425,16 +431,6 @@ void gradleDependencyXMLWSModuleExclusion() {
               dependencies {
                   implementation("javax.xml.ws:jaxws-api:2.0")
               }
-              """,
-            """
-              plugins {
-                  id 'java-library'
-              }
-              repositories {
-                  mavenCentral()
-              }
-              dependencies {
-              }
               """
           )
         );
@@ -500,6 +496,18 @@ void removeMavenXMLJaxBModuleDependency() {
                     <artifactId>gson</artifactId>
                     <version>2.10.1</version>
                   </dependency>
+                  <dependency>
+                    <groupId>javax.xml.bind</groupId>
+                    <artifactId>jaxb-api</artifactId>
+                     <version>2.3.1</version>
+                    <scope>provided</scope>
+                  </dependency>
+                  <dependency>
+                    <groupId>javax.activation</groupId>
+                    <artifactId>activation</artifactId>
+                    <version>1.1.1</version>
+                    <scope>provided</scope>
+                  </dependency>
                 </dependencies>
               </project>
               """
@@ -525,16 +533,6 @@ void gradleDependencyXMLJaxBModuleExclusion() {
                   implementation("javax.xml.bind:jaxb-api:2.3.1")
                   implementation("javax.activation:activation:1.1.1")
               }
-              """,
-            """
-              plugins {
-                  id 'java-library'
-              }
-              repositories {
-                  mavenCentral()
-              }
-              dependencies {
-              }
               """
           )
         );