From 3220d1d528363646ea5cca97e204e863b8a007f7 Mon Sep 17 00:00:00 2001 From: Danny Kopping Date: Wed, 25 Mar 2026 13:04:53 +0200 Subject: [PATCH] fix(coderd/x/chatd): use `*_TEST_API_KEY` env vars in integration tests instead of `*_API_KEY` (#23567) *Disclaimer: implemented by a Coder Agent and reviewed by me.* Renames the env vars used by chatd integration tests from the canonical `SOMEPROVIDER_API_KEY` (e.g. `ANTHROPIC_API_KEY`, `OPENAI_API_KEY`) to `SOMEPROVIDER_TEST_API_KEY` (e.g. `ANTHROPIC_TEST_API_KEY`, `OPENAI_TEST_API_KEY`) so that test-specific keys don't collide with production/canonical provider credentials. Relates to https://github.com/coder/internal/issues/1425 See also: https://codercom.slack.com/archives/C0AGTPWLA3U/p1774433646799499 --- coderd/x/chatd/integration_test.go | 18 +++++++++--------- 1 file changed, 9 insertions(+), 9 deletions(-) diff --git a/coderd/x/chatd/integration_test.go b/coderd/x/chatd/integration_test.go index 01be17c4a6..7704e4d43c 100644 --- a/coderd/x/chatd/integration_test.go +++ b/coderd/x/chatd/integration_test.go @@ -33,13 +33,13 @@ import ( // web_search tool use with id srvtoolu_... was found without a // corresponding web_search_tool_result block // -// The test requires ANTHROPIC_API_KEY to be set. +// The test requires ANTHROPIC_TEST_API_KEY to be set. func TestAnthropicWebSearchRoundTrip(t *testing.T) { t.Parallel() - apiKey := os.Getenv("ANTHROPIC_API_KEY") + apiKey := os.Getenv("ANTHROPIC_TEST_API_KEY") if apiKey == "" { - t.Skip("ANTHROPIC_API_KEY not set; skipping Anthropic integration test") + t.Skip("ANTHROPIC_TEST_API_KEY not set; skipping Anthropic integration test") } baseURL := os.Getenv("ANTHROPIC_BASE_URL") @@ -293,13 +293,13 @@ func logMessages(t *testing.T, msgs []codersdk.ChatMessage) { // Item 'rs_xxx' of type 'reasoning' was provided without its // required following item. // -// The test requires OPENAI_API_KEY to be set. +// The test requires OPENAI_TEST_API_KEY to be set. func TestOpenAIReasoningRoundTrip(t *testing.T) { t.Parallel() - apiKey := os.Getenv("OPENAI_API_KEY") + apiKey := os.Getenv("OPENAI_TEST_API_KEY") if apiKey == "" { - t.Skip("OPENAI_API_KEY not set; skipping OpenAI integration test") + t.Skip("OPENAI_TEST_API_KEY not set; skipping OpenAI integration test") } baseURL := os.Getenv("OPENAI_BASE_URL") @@ -447,13 +447,13 @@ func TestOpenAIReasoningRoundTrip(t *testing.T) { // Item with id 'msg_xxx' not found. Items are not persisted when // store is set to false. // -// The test requires OPENAI_API_KEY to be set. +// The test requires OPENAI_TEST_API_KEY to be set. func TestOpenAIReasoningRoundTripStoreFalse(t *testing.T) { t.Parallel() - apiKey := os.Getenv("OPENAI_API_KEY") + apiKey := os.Getenv("OPENAI_TEST_API_KEY") if apiKey == "" { - t.Skip("OPENAI_API_KEY not set; skipping OpenAI integration test") + t.Skip("OPENAI_TEST_API_KEY not set; skipping OpenAI integration test") } baseURL := os.Getenv("OPENAI_BASE_URL")