Normalize image names in prefix substitutor

Optionally normalize docker image names to use `library/` prefix when
they don't have any namespace at all. This allows for the
`PrefixingImageNameSubstitutor` to be used to redirect Docker Hub
requests to a private registry without having to apply otherwise
redundant changes to the test code.

Author: findepi

core/src/main/java/org/testcontainers/utility/DockerImageName.java

@@ -26,7 +26,7 @@ public final class DockerImageName {
 
     private static final Pattern REPO_NAME = Pattern.compile(REPO_NAME_PART + "(/" + REPO_NAME_PART + ")*");
 
-    private static final String LIBRARY_PREFIX = "library/";
+    public static final String LIBRARY_PREFIX = "library/";
 
     private final String rawName;
 

core/src/main/java/org/testcontainers/utility/PrefixingImageNameSubstitutor.java

@@ -16,6 +16,9 @@ final class PrefixingImageNameSubstitutor extends ImageNameSubstitutor {
     @VisibleForTesting
     static final String PREFIX_PROPERTY_KEY = "hub.image.name.prefix";
 
+    @VisibleForTesting
+    static final String NORMALIZE_PROPERTY_KEY = "hub.image.name.normalize";
+
     private TestcontainersConfiguration configuration = TestcontainersConfiguration.getInstance();
 
     @VisibleForTesting
@@ -26,6 +29,9 @@ final class PrefixingImageNameSubstitutor extends ImageNameSubstitutor {
     @Override
     public DockerImageName apply(DockerImageName original) {
         final String configuredPrefix = configuration.getEnvVarOrProperty(PREFIX_PROPERTY_KEY, "");
+        final boolean normalize = Boolean.parseBoolean(
+            configuration.getEnvVarOrProperty(NORMALIZE_PROPERTY_KEY, "false")
+        );
 
         if (configuredPrefix.isEmpty()) {
             log.debug("No prefix is configured");
@@ -42,9 +48,18 @@ public DockerImageName apply(DockerImageName original) {
 
         DockerImageName prefixAsImage = DockerImageName.parse(configuredPrefix);
 
-        return original
+        String repository = original.getRepository();
+        if (normalize && !repository.contains("/")) {
+            repository = DockerImageName.LIBRARY_PREFIX + repository;
+        }
+
+        DockerImageName substituted = original
             .withRegistry(prefixAsImage.getRegistry())
-            .withRepository(prefixAsImage.getRepository() + original.getRepository());
+            .withRepository(prefixAsImage.getRepository() + repository);
+        if (normalize) {
+            substituted = substituted.asCompatibleSubstituteFor(original);
+        }
+        return substituted;
     }
 
     @Override

core/src/test/java/org/testcontainers/utility/PrefixingImageNameSubstitutorTest.java

@@ -33,6 +33,21 @@ public void testHappyPath() {
             .isEqualTo("someregistry.com/our-mirror/some/image:tag");
     }
 
+    @Test
+    public void testNormalizeToLibraryPath() {
+        when(mockConfiguration.getEnvVarOrProperty(eq(PrefixingImageNameSubstitutor.PREFIX_PROPERTY_KEY), any()))
+            .thenReturn("someregistry.com/our-mirror/");
+        when(mockConfiguration.getEnvVarOrProperty(eq(PrefixingImageNameSubstitutor.NORMALIZE_PROPERTY_KEY), any()))
+            .thenReturn("true");
+
+        final DockerImageName result = underTest.apply(DockerImageName.parse("image:tag"));
+
+        assertThat(result.asCanonicalNameString())
+            .as("The prefix is applied")
+            .isEqualTo("someregistry.com/our-mirror/library/image:tag");
+        result.assertCompatibleWith(DockerImageName.parse("image:tag"));
+    }
+
     @Test
     public void hubIoRegistryIsNotChanged() {
         when(mockConfiguration.getEnvVarOrProperty(eq(PrefixingImageNameSubstitutor.PREFIX_PROPERTY_KEY), any()))