Use a lenient artifact view for resolving javaModulesMergeJars

Fixes #36

Author: jjohannes

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Extra Java Module Info Gradle Plugin - Changelog
 
+## Version 1.1
+* [Fixed] [#36](https://github.com/jjohannes/extra-java-module-info/issues/36) - mergeJar can lead to unnecessary build failures (Thanks [nieqian1230](https://github.com/nieqian1230) for reporting!)
+
 ## Version 1.0
 * Moved project to [GradleX](https://gradlex.org) - new plugin ID: `org.gradlex.extra-java-module-info`
 

src/main/java/org/gradlex/javamodule/moduleinfo/ExtraJavaModuleInfoPlugin.java

@@ -105,7 +105,8 @@ private void configureTransform(Project project, ExtraJavaModuleInfoPluginExtens
                 p.getFailOnMissingModuleInfo().set(extension.getFailOnMissingModuleInfo());
 
                 // See: https://github.com/adammurdoch/dependency-graph-as-task-inputs/blob/main/plugins/src/main/java/TestPlugin.java
-                Provider<Set<ResolvedArtifactResult>> artifacts = project.provider(() -> javaModulesMergeJars.getIncoming().getArtifacts().getArtifacts());
+                Provider<Set<ResolvedArtifactResult>> artifacts = project.provider(() ->
+                        javaModulesMergeJars.getIncoming().artifactView(v -> v.lenient(true)).getArtifacts().getArtifacts());
                 p.getMergeJarIds().set(artifacts.map(new IdExtractor()));
                 p.getMergeJars().set(artifacts.map(new FileExtractor(project.getLayout())));
             });

src/test/groovy/org/gradlex/javamodule/moduleinfo/test/EdgeCasesFunctionalTest.groovy

@@ -0,0 +1,63 @@
+package org.gradlex.javamodule.moduleinfo.test
+
+import org.gradlex.javamodule.moduleinfo.test.fixture.GradleBuild
+import org.gradlex.javamodule.moduleinfo.test.fixture.LegacyLibraries
+import spock.lang.Specification
+
+class EdgeCasesFunctionalTest extends Specification {
+
+    @Delegate
+    GradleBuild build = new GradleBuild()
+
+    LegacyLibraries libs = new LegacyLibraries(false)
+
+    def setup() {
+        settingsFile << 'rootProject.name = "test-project"'
+        buildFile << '''
+            plugins {
+                id("application")
+                id("org.gradlex.extra-java-module-info")
+            }
+            application {
+                mainModule.set("org.gradle.sample.app")
+                mainClass.set("org.gradle.sample.app.Main")
+            }
+        '''
+    }
+
+    def "doe not fail if an unused Jar on the merge path cannot be resolved"() {
+        given:
+        file("src/main/java/org/gradle/sample/app/Main.java") << """
+            package org.gradle.sample.app;
+            
+            public class Main {
+                public static void main(String[] args) {
+                }
+            }
+        """
+        file("src/main/java/module-info.java") << """
+            module org.gradle.sample.app {
+                requires org.slf4j;
+            }
+        """
+        buildFile << """                  
+            dependencies {
+                implementation("org.slf4j:slf4j-api:2.0.3")
+            }
+            
+            extraJavaModuleInfo {
+                failOnMissingModuleInfo.set(false)
+                module("${libs.zookeeper}", "org.apache.zookeeper") {
+                    mergeJar("${libs.zookeeperJute}")
+
+                    exports("org.apache.jute")
+                    exports("org.apache.zookeeper")
+                    exports("org.apache.zookeeper.server.persistence")
+                }
+            }
+        """
+
+        expect:
+        run()
+    }
+}