Skip to content

Conversation

@terry-docker
Copy link
Contributor

  • Add HttpWaitStrategy for HTTP endpoint waiting with TLS, auth, headers support
  • Add HealthcheckWaitStrategy for Docker health check status monitoring
  • Add PortWaitStrategy for TCP port availability checking
  • Add FileExistsWaitStrategy for filesystem file existence waiting
  • Add CompositeWaitStrategy for combining multiple wait conditions
  • Include comprehensive unit tests for all new strategies
  • Enhanced error messages with actionable hints for debugging
  • Maintain backward compatibility with existing LogMessageWaitStrategy
  • Fix mypy type annotations for all new wait strategies

- Add HttpWaitStrategy for HTTP endpoint waiting with TLS, auth, headers support - Add HealthcheckWaitStrategy for Docker health check status monitoring - Add PortWaitStrategy for TCP port availability checking - Add FileExistsWaitStrategy for filesystem file existence waiting - Add CompositeWaitStrategy for combining multiple wait conditions - Include comprehensive unit tests for all new strategies - Enhanced error messages with actionable hints for debugging - Maintain backward compatibility with existing LogMessageWaitStrategy - Fix mypy type annotations for all new wait strategies
@codecov
Copy link

codecovbot commented Aug 6, 2025

Codecov Report

❌ Patch coverage is 66.80328% with 81 lines in your changes missing coverage. Please review.
⚠️ Please upload report for BASE (main@003046b). Learn more about missing BASE report.

Files with missing linesPatch %Lines
core/testcontainers/core/wait_strategies.py66.80%75 Missing and 6 partials ⚠️
Additional details and impacted files
@@ Coverage Diff @@## main #855 +/- ## ======================================= Coverage ? 78.67% ======================================= Files ? 14 Lines ? 1130 Branches ? 172 ======================================= Hits ? 889 Misses ? 201 Partials ? 40 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@alexanderankin
Copy link
Member

the failing test is a docker registry timeout. i will save some gha credits and just simply merge.

@alexanderankinalexanderankin merged commit 60d21f8 into testcontainers:mainAug 27, 2025
7 of 8 checks passed
alexanderankin pushed a commit that referenced this pull request Sep 9, 2025
🤖 I have created a release *beep* *boop* --- ## [4.13.0](testcontainers-v4.12.0...testcontainers-v4.13.0) (2025-08-27) ### Features * **azurite:** Enhance connection string generation for network and local access ([#859](#859)) ([b21e5e3](b21e5e3)) * **core:** add enhanced wait strategies ([#855](#855)) ([60d21f8](60d21f8)) * **core:** DockerCompose: support list of env_files ([#847](#847)) ([fe206eb](fe206eb)) ### Bug Fixes * assert-in-get_container_host_ip-before-start ([#862](#862)) ([fc4155e](fc4155e)) * **core:** improper reading of .testcontainers.properties ([#863](#863)) ([350f246](350f246)) * **core:** Make TC_POOLING_INTERVAL/sleep_time a float ([#839](#839)) ([a072f3f](a072f3f)) --- This PR was generated with [Release Please](https://github.com/googleapis/release-please). See [documentation](https://github.com/googleapis/release-please#release-please). Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@terry-docker@alexanderankin