fix: default port expose in containerized_testing_database.py

Author: atstaeff

tests/shared/docker/container.py

@@ -1,5 +1,7 @@
 from dataclasses import dataclass
 import subprocess
+import json
+import time
 
 from .docker_command_failed_error import DockerCommandFailedError
 from ..util.remove_whitespace import remove_whitespace
@@ -24,10 +26,7 @@ def get_exposed_port(self, internal_port: int = 3_000) -> int:
         docker_command = [
             'docker',
             'inspect',
-            # this is a string that cannot be break apart in a useful way
-            # pylint: disable=line-too-long
-            f'--format=\'{{{{(index (index .NetworkSettings.Ports "{internal_port}/tcp") 0).HostPort}}}}\'',
-            # pylint: enable=line-too-long
+            f'--format={{{{json .NetworkSettings.Ports}}}}',
             self.id
         ]
 
@@ -41,14 +40,35 @@ def get_exposed_port(self, internal_port: int = 3_000) -> int:
             if process.returncode != 0:
                 raise DockerCommandFailedError(
                     f'Inspect failed with output: {stderr.decode("utf-8")}')
+            
+            # Parse the JSON output to get the port mapping
+            try:
+                port_mappings = json.loads(stdout.decode('utf-8').strip())
+                port_key = f"{internal_port}/tcp"
+                
+                if port_key not in port_mappings or not port_mappings[port_key]:
+                    # Wait briefly and try to restart/refresh the container
+                    time.sleep(1)
+                    self.restart()
+                    time.sleep(2)  # Wait for container to be ready
+                    
+                    # Try one more time after restart
+                    return self.get_exposed_port(internal_port)
+                    
+                return int(port_mappings[port_key][0]['HostPort'])
+            except (json.JSONDecodeError, KeyError, IndexError, TypeError) as e:
+                raise DockerCommandFailedError(
+                    f'Failed to parse port mapping: {stdout.decode("utf-8")}. Error: {str(e)}')
 
-            exposed_port_as_string = remove_whitespace(
-                stdout.decode('utf-8')
-            ).replace('\'', '').replace('|', '')
-
-        try:
-            return int(exposed_port_as_string)
-        except ValueError as value_error:
-            raise ValueError(
-                f'Could not parse port from: {exposed_port_as_string}'
-            ) from value_error
+    def restart(self):
+        """Restart the container"""
+        docker_command = ['docker', 'restart', self.id]
+        with subprocess.Popen(
+            docker_command,
+            stdout=subprocess.PIPE,
+            stderr=subprocess.PIPE
+        ) as process:
+            process.communicate()
+            
+            if process.returncode != 0:
+                raise DockerCommandFailedError('Failed to restart container')

tests/shared/docker/image.py

@@ -17,6 +17,8 @@ def run(self, command: list[str], is_detached: bool, do_expose_ports: bool) -> C
             docker_command.append('-d')
 
         if do_expose_ports:
+            # Expose the default HTTP port (3000)
+            docker_command.extend(['-p', '3000'])
             docker_command.append('-P')
 
         docker_command.append(self.__get_full_name())