feat: update CalculationInstanceDockerAdapter to expose port

Author: andrei2699

src/Adapters/Driven/Analysis.Instance.Docker/CalculationInstanceDockerAdapter.cs

@@ -27,12 +27,30 @@ public async Task<string> Allocate(
 
         await PullImageIfNeeded(client, image.ImageName, cancellationToken);
 
+        var hostPort = freePortProvider.GetFreeTcpPort();
+
+        var portBindings = new Dictionary<string, IList<PortBinding>>
+        {
+            {
+                $"{calculationDockerConfig.Port}/tcp",
+                new List<PortBinding> { new() { HostPort = hostPort.ToString() } }
+            },
+        };
+
         var response = await client.Containers.CreateContainerAsync(
             new CreateContainerParameters
             {
                 Name = $"scriptbee-calculation-{instanceId}",
                 Image = image.ImageName,
-                HostConfig = new HostConfig { NetworkMode = calculationDockerConfig.Network },
+                HostConfig = new HostConfig
+                {
+                    NetworkMode = calculationDockerConfig.Network,
+                    PortBindings = portBindings,
+                },
+                ExposedPorts = new Dictionary<string, EmptyStruct>
+                {
+                    { $"{calculationDockerConfig.Port}/tcp", new EmptyStruct() },
+                },
             },
             cancellationToken
         );
@@ -50,7 +68,7 @@ await client.Containers.StartContainerAsync(
             client,
             response.ID,
             calculationDockerConfig.Network,
-            freePortProvider.GetFreeTcpPort(),
+            hostPort,
             cancellationToken
         );
     }

src/Adapters/Driven/Analysis.Instance.Docker/Config/CalculationDockerConfig.cs

@@ -4,5 +4,7 @@ public class CalculationDockerConfig
 {
     public required string DockerSocket { get; init; }
 
+    public int Port { get; init; } = 8080;
+
     public string? Network { get; init; }
 }

test/Adapters/Driven/Analysis.Instance.Docker.Tests/CalculationInstanceDockerAdapterTest.cs

@@ -17,7 +17,7 @@ public class CalculationInstanceDockerAdapterTest : IClassFixture<DockerFixture>
     private readonly DockerFixture _dockerFixture;
 
     private readonly CalculationInstanceDockerAdapter _calculationInstanceDockerAdapter;
-    private const int TestPort = 8080;
+    private readonly int _testPort;
 
     public CalculationInstanceDockerAdapterTest(
         DockerFixture dockerFixture,
@@ -32,7 +32,8 @@ ITestOutputHelper outputHelper
                 Network = DockerFixture.TestNetworkName,
             }
         );
-        _freePortProvider.GetFreeTcpPort().Returns(TestPort);
+        _testPort = new FreePortProvider().GetFreeTcpPort();
+        _freePortProvider.GetFreeTcpPort().Returns(_testPort);
 
         var loggerFactory = LoggerFactory.Create(builder =>
             builder.AddProvider(new XUnitLoggerProvider(outputHelper))
@@ -55,7 +56,7 @@ public async Task Allocate_ShouldCreateAndStartContainerAndReturnUrlWithNetworkI
         var containerUrl = await _calculationInstanceDockerAdapter.Allocate(instanceId, image);
 
         containerUrl.ShouldStartWith("http://");
-        containerUrl.ShouldContain($":{TestPort}");
+        containerUrl.ShouldContain($":{_testPort}");
         var containers = await _dockerFixture.DockerClient.Containers.ListContainersAsync(
             new ContainersListParameters { All = true }
         );
@@ -69,7 +70,7 @@ public async Task Allocate_ShouldCreateAndStartContainerAndReturnUrlWithNetworkI
             .NetworkSettings.Networks[DockerFixture.TestNetworkName]
             .IPAddress.ShouldNotBeNullOrEmpty();
         containerUrl.ShouldBe(
-            $"http://{ourContainer.NetworkSettings.Networks[DockerFixture.TestNetworkName].IPAddress}:{TestPort}"
+            $"http://{ourContainer.NetworkSettings.Networks[DockerFixture.TestNetworkName].IPAddress}:{_testPort}"
         );
     }