Skip to content

test: Use local FakeServer in tests to avoid real network calls#2165

Merged
lqiu96 merged 1 commit into
mainfrom
fix-graalvm-test-ssl
Jun 29, 2026
Merged

test: Use local FakeServer in tests to avoid real network calls#2165
lqiu96 merged 1 commit into
mainfrom
fix-graalvm-test-ssl

Conversation

@lqiu96

@lqiu96 lqiu96 commented Jun 26, 2026

Copy link
Copy Markdown
Member

Error from #2164


Rewrite testRequestShouldNotFollowRedirects to use a local FakeServer
instead of connecting to google.com, avoiding SSL handshake failures
in environments without proper trust stores (like GraalVM native image tests).

Also update testRequestCanSetHeaders to use localhost dummy URL.

TAG=agy
CONV=b7638222-8f45-4b8a-8698-ae587dfa7e88

@lqiu96 lqiu96 requested a review from a team as a code owner June 26, 2026 19:37
@product-auto-label product-auto-label Bot added the size: s Pull request size is small. label Jun 26, 2026
@lqiu96 lqiu96 force-pushed the fix-graalvm-test-ssl branch from 2257df0 to 3c7e4ca Compare June 26, 2026 19:49
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. and removed size: s Pull request size is small. labels Jun 26, 2026
@lqiu96 lqiu96 force-pushed the fix-graalvm-test-ssl branch from 3c7e4ca to e1aa3cf Compare June 26, 2026 19:51
@lqiu96 lqiu96 changed the title fix: resolve GraalVM SSL handshake test failure in ApacheHttpTransportTest test: resolve GraalVM SSL handshake failure in ApacheHttpTransportTest Jun 26, 2026
Rewrite redirect tests in ApacheHttpTransportTest (v2) and
Apache5HttpTransportTest (v5) to use a local FakeServer instead of
connecting to google.com, avoiding SSL handshake and network
unreachable failures in restricted environments.

Also update header tests to use localhost/FakeServer dummy URLs to
avoid external network calls.

TAG=agy
CONV=b7638222-8f45-4b8a-8698-ae587dfa7e88
@lqiu96 lqiu96 force-pushed the fix-graalvm-test-ssl branch from e1aa3cf to 0eafce0 Compare June 26, 2026 21:43
@lqiu96 lqiu96 changed the title test: resolve GraalVM SSL handshake failure in ApacheHttpTransportTest test: resolve GraalVM SSL/network failures in Apache transport tests Jun 26, 2026
@lqiu96 lqiu96 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 29, 2026
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 29, 2026
@lqiu96 lqiu96 changed the title test: resolve GraalVM SSL/network failures in Apache transport tests test: Use local FakeServer in tests to avoid real network calls Jun 29, 2026
@lqiu96 lqiu96 merged commit eb787d7 into main Jun 29, 2026
29 checks passed
@lqiu96 lqiu96 deleted the fix-graalvm-test-ssl branch June 29, 2026 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: m Pull request size is medium.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants