Ignore container_name in docker-compose files instead of throwing exception

Fixes #2472

Previously, Testcontainers would throw an IllegalStateException when
parsing a docker-compose file that contained the 'container_name'
property. This prevented users from reusing their production
docker-compose files without modification.

This change modifies the behavior to log a warning instead of throwing
an exception, allowing users to use docker-compose files with
container_name properties. The container_name is simply ignored since
Testcontainers manages container naming internally.

Changes:
- ParsedDockerComposeFile: Changed validateNoContainerNameSpecified()
  to log a warning instead of throwing IllegalStateException
- ParsedDockerComposeFileValidationTest: Updated tests to verify that
  container_name is now ignored rather than rejected

Author: Ranjith Muniyappa

core/src/main/java/org/testcontainers/containers/ParsedDockerComposeFile.java

@@ -131,12 +131,11 @@ private void parseAndValidate() {
 
     private void validateNoContainerNameSpecified(String serviceName, Map<String, ?> serviceDefinitionMap) {
         if (serviceDefinitionMap.containsKey("container_name")) {
-            throw new IllegalStateException(
-                String.format(
-                    "Compose file %s has 'container_name' property set for service '%s' but this property is not supported by Testcontainers, consider removing it",
-                    composeFileName,
-                    serviceName
-                )
+            log.warn(
+                "Compose file {} has 'container_name' property set for service '{}'. " +
+                "This property is not supported by Testcontainers and will be ignored.",
+                composeFileName,
+                serviceName
             );
         }
     }

core/src/test/java/org/testcontainers/containers/ParsedDockerComposeFileValidationTest.java

@@ -22,36 +22,35 @@ class ParsedDockerComposeFileValidationTest {
     public Path temporaryFolder;
 
     @Test
-    void shouldValidate() {
+    void shouldIgnoreContainerNameV1() {
         File file = new File("src/test/resources/docker-compose-container-name-v1.yml");
-        assertThatThrownBy(() -> {
-                new ParsedDockerComposeFile(file);
-            })
-            .hasMessageContaining(file.getAbsolutePath())
-            .hasMessageContaining("'container_name' property set for service 'redis'");
+        // container_name should be ignored (with a warning log) instead of throwing an exception
+        assertThatNoException().isThrownBy(() -> new ParsedDockerComposeFile(file));
     }
 
     @Test
-    void shouldRejectContainerNameV1() {
-        assertThatThrownBy(() -> {
-                new ParsedDockerComposeFile(ImmutableMap.of("redis", ImmutableMap.of("container_name", "redis")));
-            })
-            .hasMessageContaining("'container_name' property set for service 'redis'");
+    void shouldIgnoreContainerNameInMapV1() {
+        // container_name should be ignored (with a warning log) instead of throwing an exception
+        assertThatNoException()
+            .isThrownBy(() ->
+                new ParsedDockerComposeFile(ImmutableMap.of("redis", ImmutableMap.of("container_name", "redis")))
+            );
     }
 
     @Test
-    void shouldRejectContainerNameV2() {
-        assertThatThrownBy(() -> {
+    void shouldIgnoreContainerNameV2() {
+        // container_name should be ignored (with a warning log) instead of throwing an exception
+        assertThatNoException()
+            .isThrownBy(() ->
                 new ParsedDockerComposeFile(
                     ImmutableMap.of(
                         "version",
                         "2",
                         "services",
                         ImmutableMap.of("redis", ImmutableMap.of("container_name", "redis"))
                     )
-                );
-            })
-            .hasMessageContaining("'container_name' property set for service 'redis'");
+                )
+            );
     }
 
     @Test