fix: wait for startup scripts to complete by codingkiddo · Pull Request #156 · floci-io/testcontainers-floci (original) (raw)
Summary
Update FlociContainer to wait for Floci initialization readiness instead of only the health endpoint.
/_floci/health can return 200 while start.d scripts are still running. Waiting for /_floci/init with "ready": true ensures startup scripts have completed before container.start() returns.
Related issue
Fixes #151
Type of change
- Bug fix (
fix:commit) - New feature (
feat:commit) - Breaking change (
feat!:commit orBREAKING CHANGE:footer) - Documentation or chore
Checklist
mvn verifypasses locally (requires Docker)- New or changed behaviour is covered by tests
- All commit messages follow [Conventional Commits](https://www.conventionalcommits.org/) (
feat:,fix:,chore:, etc.)
Verification
mvn -pl testcontainers-floci -Dtest=FlociContainerTest testmvn -pl testcontainers-floci -DskipTests package
Note: full mvn verify had unrelated baseline failures locally in CloudFront, Config, and Neptune service tests before this change.