Merge pull request #23111 from famod/graphql-tcks-noise

Fix noise about missing `src/main/resources/overrides` in GraphQL TCK module
quarkusio · Jan 24, 2022 · 5d51dfe · 5d51dfe
2 parents 8297512 + 9ce62cd
commit 5d51dfe
Showing 1 changed file with 10 additions and 6 deletions.
diff --git a/tcks/microprofile-graphql/src/main/java/io/quarkus/tck/graphql/TestInterceptor.java b/tcks/microprofile-graphql/src/main/java/io/quarkus/tck/graphql/TestInterceptor.java
@@ -60,10 +60,12 @@ public TestInterceptor() {
     }
 
     private void loadSchemaOverride() {
+        Path folderPath = Paths.get(OVERRIDES);
+        if (!Files.isDirectory(folderPath)) {
+            return;
+        }
         // Get all csv files
-        try {
-            Path folderPath = Paths.get(OVERRIDES);
-            DirectoryStream<Path> overrides = Files.newDirectoryStream(folderPath);
+        try (DirectoryStream<Path> overrides = Files.newDirectoryStream(folderPath)) {
             List<Path> overrideFiles = toListOfPathsForSchema(overrides);
             List<org.eclipse.microprofile.graphql.tck.dynamic.schema.TestData> testDataList = toListOfTestDataForSchema(
                     overrideFiles);
@@ -78,9 +80,11 @@ private void loadSchemaOverride() {
     }
 
     private void loadExecutionOverride() {
-        try {
-            Path folderPath = Paths.get(OVERRIDES);
-            DirectoryStream<Path> overrides = Files.newDirectoryStream(folderPath);
+        Path folderPath = Paths.get(OVERRIDES);
+        if (!Files.isDirectory(folderPath)) {
+            return;
+        }
+        try (DirectoryStream<Path> overrides = Files.newDirectoryStream(folderPath)) {
             Set<Path> overrideFolders = toListOfPathsForExecution(overrides);
             executionTestDataMap.putAll(toMapOfTestDataForExecution(overrideFolders));
         } catch (IOException ex) {