jrgemignani opened a new pull request, #2333: URL: https://github.com/apache/age/pull/2333
Note: The PR was created with the help of AI tools and a human. Fix two root causes of CI test failures reproduced in ubuntu:24.04 container with Zulu Java 17 + Gradle 9.3.1 (matching GitHub Actions): build.gradle.kts: Set javac encoding to UTF-8 to prevent Unicode test failures (e.g. Greek Omega) when the CI runner locale defaults to US-ASCII. Without this, source characters are garbled at compile time, causing deterministic assertion failures in AgtypeFactoryTest and AgtypeUtilTest. BaseDockerizedTest.java: Fix Testcontainers setup reliability - - Add Wait.forLogMessage wait strategy with 60s timeout so PostgreSQL is fully ready before JDBC connection attempt - Use getHost() instead of hardcoded "localhost" for correct behavior in Docker-in-Docker and varied CI environments - Add sslmode=disable to JDBC URL since PostgreSQL driver 42.6.0+ attempts SSL negotiation by default and the AGE Docker image has no SSL configured - Remove silent catch/println that swallowed connection errors and produced misleading NullPointerExceptions downstream Verified: 77/77 tests pass across multiple consecutive runs. modified: drivers/jdbc/lib/build.gradle.kts modified: drivers/jdbc/lib/src/test/java/org/apache/age/jdbc/BaseDockerizedTest.java -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
