Remove explicit port binding in Azurite container

Switched from explicitly binding ports to relying on automatic assignment to avoid manual management of ports. Simplifies configuration and reduces potential conflicts.

Author: ChlorUpload

packages/modules/azurite/src/azurite-container.test.ts

@@ -113,34 +113,6 @@ describe("Azurite", () => {
   });
   // }
 
-  // customPorts {
-  it("should be able to specify custom ports", async () => {
-    const blobPort = 13000;
-    const queuePort = 14000;
-    const tablePort = 15000;
-    const container = await new AzuriteContainer()
-      .withBlobPort(blobPort)
-      .withQueuePort(queuePort)
-      .withTablePort(tablePort)
-      .start();
-
-    expect(container.getBlobPort()).toBe(blobPort);
-    expect(container.getQueuePort()).toBe(queuePort);
-    expect(container.getTablePort()).toBe(tablePort);
-
-    const connectionString = container.getConnectionString();
-    expect(connectionString).toContain("13000");
-    expect(connectionString).toContain("14000");
-    expect(connectionString).toContain("15000");
-
-    const serviceClient = BlobServiceClient.fromConnectionString(connectionString);
-    const containerClient = serviceClient.getContainerClient("test");
-    await containerClient.createIfNotExists();
-
-    await container.stop();
-  });
-  // }
-
   // inMemoryPersistence {
   it("should be able to use in-memory persistence", async () => {
     const container = await new AzuriteContainer().withInMemoryPersistence().start();

packages/modules/azurite/src/azurite-container.ts

@@ -21,9 +21,6 @@ export class AzuriteContainer extends GenericContainer {
       .withStartupTimeout(120_000);
   }
 
-  private blobPort: number = BLOB_PORT;
-  private queuePort: number = QUEUE_PORT;
-  private tablePort: number = TABLE_PORT;
   private accountName: string = DEFAULT_ACCOUNT_NAME;
   private accountKey: string = DEFAULT_ACCOUNT_KEY;
 
@@ -49,33 +46,6 @@ export class AzuriteContainer extends GenericContainer {
     return this;
   }
 
-  /**
-   * Sets the port to expose the Blob service on.
-   * @param port The port to expose the Blob service on. Default is 10000.
-   */
-  public withBlobPort(port: number): this {
-    this.blobPort = port;
-    return this;
-  }
-
-  /**
-   * Sets the port to expose the Queue service on.
-   * @param port The port to expose the Queue service on. Default is 10001.
-   */
-  public withQueuePort(port: number): this {
-    this.queuePort = port;
-    return this;
-  }
-
-  /**
-   * Sets the port to expose the Table service on.
-   * @param port The port to expose the Table service on. Default is 10002.
-   */
-  public withTablePort(port: number): this {
-    this.tablePort = port;
-    return this;
-  }
-
   /**
    * Disable persisting any data to disk and only store data in-memory. If the Azurite process is terminated, all data is lost.
    */
@@ -119,11 +89,7 @@ export class AzuriteContainer extends GenericContainer {
       command.push("--skipApiVersionCheck");
     }
 
-    this.withCommand(command).withExposedPorts(
-      { container: BLOB_PORT, host: this.blobPort },
-      { container: QUEUE_PORT, host: this.queuePort },
-      { container: TABLE_PORT, host: this.tablePort }
-    );
+    this.withCommand(command).withExposedPorts(BLOB_PORT, QUEUE_PORT, TABLE_PORT);
 
     if (this.accountName !== DEFAULT_ACCOUNT_NAME || this.accountKey !== DEFAULT_ACCOUNT_KEY) {
       this.withEnvironment({
@@ -137,9 +103,6 @@ export class AzuriteContainer extends GenericContainer {
       startedContainer,
       this.accountName,
       this.accountKey,
-      this.blobPort,
-      this.queuePort,
-      this.tablePort
     );
   }
 }
@@ -149,9 +112,6 @@ export class StartedAzuriteContainer extends AbstractStartedContainer {
     startedTestContainer: StartedTestContainer,
     private readonly accountName: string,
     private readonly accountKey: string,
-    private readonly blobPort: number,
-    private readonly queuePort: number,
-    private readonly tablePort: number
   ) {
     super(startedTestContainer);
   }
@@ -164,28 +124,28 @@ export class StartedAzuriteContainer extends AbstractStartedContainer {
     return this.accountKey;
   }
 
-  public getBlobPort(): number {
-    return this.blobPort;
+  public getMappedBlobPort(): number {
+    return this.getMappedPort(BLOB_PORT);
   }
 
-  public getQueuePort(): number {
-    return this.queuePort;
+  public getMappedQueuePort(): number {
+    return this.getMappedPort(QUEUE_PORT);
   }
 
-  public getTablePort(): number {
-    return this.tablePort;
+  public getMappedTablePort(): number {
+    return this.getMappedPort(TABLE_PORT);
   }
 
   public getBlobEndpoint(): string {
-    return this.getEndpoint(this.blobPort, this.accountName);
+    return this.getEndpoint(this.getMappedBlobPort(), this.accountName);
   }
 
   public getQueueEndpoint(): string {
-    return this.getEndpoint(this.queuePort, this.accountName);
+    return this.getEndpoint(this.getMappedQueuePort(), this.accountName);
   }
 
   public getTableEndpoint(): string {
-    return this.getEndpoint(this.tablePort, this.accountName);
+    return this.getEndpoint(this.getMappedTablePort(), this.accountName);
   }
 
   /**