Update test containers strategy arguments

[djthorpe]

This pull request updates the OptPostgres configuration in pkg/test/opt.go to improve type safety and compatibility with the latest testcontainers-go API. The key change is switching the port parameter in the wait strategy from a string to a network.Port type, and updating the code to use the Port() method accordingly.

Dependency and API compatibility:

• Added an import for github.com/moby/moby/api/types/network to support the network.Port type.
• Updated the OptPostgres wait strategy to accept a network.Port type for the port parameter and extract the port string using the Port() method, ensuring compatibility with the updated testcontainers-go API.

[Copilot]

🟡 Changes recommended

The updated wait.ForSQL callback signature does not match the currently pinned testcontainers-go v0.42.0 API, which is likely to cause a compile-time failure unless dependencies are updated accordingly.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.

Pull request overview

This PR aims to update the PostgreSQL testcontainer wait strategy in pkg/test/opt.go for improved type safety by switching the wait.ForSQL URL callback’s port argument from string to a typed network.Port and using Port() to extract the numeric port.

Changes:

• Added an import for github.com/moby/moby/api/types/network.
• Updated the wait.ForSQL URL callback signature to accept network.Port and call port.Port() when constructing the connection URL.

File summaries

File	Description
pkg/test/opt.go	Updates the SQL wait strategy callback to use a typed port and extracts the port string via Port().

Review details

• Files reviewed: 1/1 changed files
• Comments generated: 1
• Review effort level: Lite

We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.

[github-actions]

Coverage for this change: 13.0%