Move muted tests file and allow for additional files to be configured (…

…elastic#107916)

Some refactoring to the muted tests plugin to better support usage in a
composite build configuration.

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/test/MutedTestPlugin.java

@@ -8,23 +8,34 @@
 
 package org.elasticsearch.gradle.internal.test;
 
-import org.elasticsearch.gradle.internal.conventions.util.Util;
 import org.elasticsearch.gradle.internal.info.BuildParams;
 import org.gradle.api.Plugin;
 import org.gradle.api.Project;
+import org.gradle.api.file.RegularFile;
 import org.gradle.api.provider.Provider;
 import org.gradle.api.tasks.testing.Test;
 
-import java.io.File;
+import java.util.Arrays;
+import java.util.List;
 
 public class MutedTestPlugin implements Plugin<Project> {
+    private static final String ADDITIONAL_FILES_PROPERTY = "org.elasticsearch.additional.muted.tests";
+
     @Override
     public void apply(Project project) {
-        File infoPath = new File(Util.locateElasticsearchWorkspace(project.getGradle()), "build-tools-internal");
+        String additionalFilePaths = project.hasProperty(ADDITIONAL_FILES_PROPERTY)
+            ? project.property(ADDITIONAL_FILES_PROPERTY).toString()
+            : "";
+        List<RegularFile> additionalFiles = Arrays.stream(additionalFilePaths.split(","))
+            .filter(p -> p.isEmpty() == false)
+            .map(p -> project.getRootProject().getLayout().getProjectDirectory().file(p))
+            .toList();
+
         Provider<MutedTestsBuildService> mutedTestsProvider = project.getGradle()
             .getSharedServices()
             .registerIfAbsent("mutedTests", MutedTestsBuildService.class, spec -> {
-                spec.getParameters().getInfoPath().set(infoPath);
+                spec.getParameters().getInfoPath().set(project.getRootProject().getProjectDir());
+                spec.getParameters().getAdditionalFiles().set(additionalFiles);
             });
 
         project.getTasks().withType(Test.class).configureEach(test -> {

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/test/MutedTestsBuildService.java

@@ -13,7 +13,9 @@
 import com.fasterxml.jackson.databind.ObjectMapper;
 import com.fasterxml.jackson.dataformat.yaml.YAMLFactory;
 
+import org.gradle.api.file.RegularFile;
 import org.gradle.api.file.RegularFileProperty;
+import org.gradle.api.provider.ListProperty;
 import org.gradle.api.services.BuildService;
 import org.gradle.api.services.BuildServiceParameters;
 
@@ -28,25 +30,34 @@
 import java.util.List;
 
 public abstract class MutedTestsBuildService implements BuildService<MutedTestsBuildService.Params> {
-    private final List<String> excludePatterns;
+    private final List<String> excludePatterns = new ArrayList<>();
+    private final ObjectMapper objectMapper = new ObjectMapper(new YAMLFactory());
 
     public MutedTestsBuildService() {
         File infoPath = getParameters().getInfoPath().get().getAsFile();
         File mutedTestsFile = new File(infoPath, "muted-tests.yml");
-        try (InputStream is = new BufferedInputStream(new FileInputStream(mutedTestsFile))) {
-            ObjectMapper objectMapper = new ObjectMapper(new YAMLFactory());
-            List<MutedTest> mutedTests = objectMapper.readValue(is, MutedTests.class).getTests();
-            excludePatterns = buildExcludePatterns(mutedTests == null ? Collections.emptyList() : mutedTests);
-        } catch (IOException e) {
-            throw new UncheckedIOException(e);
+        excludePatterns.addAll(buildExcludePatterns(mutedTestsFile));
+        for (RegularFile regularFile : getParameters().getAdditionalFiles().get()) {
+            excludePatterns.addAll(buildExcludePatterns(regularFile.getAsFile()));
         }
     }
 
     public List<String> getExcludePatterns() {
         return excludePatterns;
     }
 
-    private static List<String> buildExcludePatterns(List<MutedTest> mutedTests) {
+    private List<String> buildExcludePatterns(File file) {
+        List<MutedTest> mutedTests;
+
+        try (InputStream is = new BufferedInputStream(new FileInputStream(file))) {
+            mutedTests = objectMapper.readValue(is, MutedTests.class).getTests();
+            if (mutedTests == null) {
+                return Collections.emptyList();
+            }
+        } catch (IOException e) {
+            throw new UncheckedIOException(e);
+        }
+
         List<String> excludes = new ArrayList<>();
         if (mutedTests.isEmpty() == false) {
             for (MutedTestsBuildService.MutedTest mutedTest : mutedTests) {
@@ -84,6 +95,8 @@ private static List<String> buildExcludePatterns(List<MutedTest> mutedTests) {
 
     public interface Params extends BuildServiceParameters {
         RegularFileProperty getInfoPath();
+
+        ListProperty<RegularFile> getAdditionalFiles();
     }
 
     public static class MutedTest {

build-tools/src/testFixtures/groovy/org/elasticsearch/gradle/fixtures/AbstractGradleFuncTest.groovy

@@ -59,7 +59,7 @@ abstract class AbstractGradleFuncTest extends Specification {
                 id 'base'
             }
         """
-        def mutedTestsFile = Files.createFile(Path.of(testProjectDir.newFolder("build-tools-internal").path, "muted-tests.yml"))
+        def mutedTestsFile = testProjectDir.newFile("muted-tests.yml")
         mutedTestsFile << """
             tests: []
         """