Temporarily work around marschall/memoryfilesystem/pull/145

Author: ascopes

java-compiler-testing/src/main/java/io/github/ascopes/jct/containers/impl/PackageContainerGroupUrlClassLoader.java

@@ -20,9 +20,13 @@
 import io.github.ascopes.jct.workspaces.PathRoot;
 import java.net.URL;
 import java.net.URLClassLoader;
+import java.util.Locale;
+import java.util.stream.Stream;
 import org.apiguardian.api.API;
 import org.apiguardian.api.API.Status;
 
+import static io.github.ascopes.jct.utils.IoExceptionUtils.uncheckedIo;
+
 /**
  * An extension of the Java {@link URLClassLoader} that wraps around container groups.
  *
@@ -50,6 +54,25 @@ private static URL[] extractUrls(PackageContainerGroup group) {
         .stream()
         .map(Container::getPathRoot)
         .map(PathRoot::getUrl)
+        .map(PackageContainerGroupUrlClassLoader::fixMemoryFileSystemUrls)
         .toArray(URL[]::new);
   }
+
+  // This can be removed once https://github.com/marschall/memoryfilesystem/pull/145/files is
+  // addressed.
+  private static URL fixMemoryFileSystemUrls(URL url) {
+    if (url.getPath().endsWith("/") || isProbablyArchive(url)) {
+      return url;
+    }
+
+    return uncheckedIo(() -> new URL(
+        url.getProtocol(), url.getHost(), url.getPort(), url.getFile() + "/"
+    ));
+  }
+
+  private static boolean isProbablyArchive(URL url) {
+    return Stream
+        .of(".jar", ".war", ".ear", ".zip")
+        .anyMatch(url.getFile().toLowerCase(Locale.ROOT)::endsWith);
+  }
 }