From 381d374c7bc083e63e9f78c669065c47cba758ca Mon Sep 17 00:00:00 2001 From: Jaehee Shin Date: Fri, 25 Sep 2026 06:30:01 +0900 Subject: [PATCH] test: point apigee-llm-proxy-url at the renamed test method The apigee-llm-proxy-url surefire execution in core/pom.xml is meant to run one ApigeeLlmTest method with APIGEE_PROXY_URL set, so that ApigeeLlm's fallback to that environment variable is tested. It selects the method by name. df737840 replaced that method with three, including build_withoutProxyUrlAndEnvVarSet_readsFromEnvironment for this case, and did not update the execution, which still names build_withoutProxyUrl_readsFromEnvironment. Since then it runs no tests: [INFO] --- surefire:3.5.2:test (apigee-llm-proxy-url) @ google-adk --- [INFO] Tests run: 0, Failures: 0, Errors: 0, Skipped: 0 The new method skips itself unless APIGEE_PROXY_URL is set, and this is the only execution that sets it, so the build has skipped the method since the change. The build still passes because surefire's failIfNoSpecifiedTests check only looks for a matching test class, and ApigeeLlmTest still matches. Point the execution at the new method name. --- core/pom.xml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/core/pom.xml b/core/pom.xml index caaf149fb..e7b06b73f 100644 --- a/core/pom.xml +++ b/core/pom.xml @@ -305,7 +305,7 @@ test - ApigeeLlmTest#build_withoutProxyUrl_readsFromEnvironment + ApigeeLlmTest#build_withoutProxyUrlAndEnvVarSet_readsFromEnvironment api-key