From 850af1fb6eff44738a95eef3accc6e86e4066263 Mon Sep 17 00:00:00 2001 From: Fran Fitzpatrick Date: Tue, 6 Jan 2026 12:08:36 -0600 Subject: [PATCH] test: update PyAnnote test to reflect optional HF token MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The HF token parameter is now optional at validation time since it can be provided via the HF_TOKEN environment variable at runtime. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude Opus 4.5 --- internal/transcription/adapters_test.go | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/internal/transcription/adapters_test.go b/internal/transcription/adapters_test.go index c9ba7dd..ca09ca8 100644 --- a/internal/transcription/adapters_test.go +++ b/internal/transcription/adapters_test.go @@ -370,17 +370,18 @@ func TestPyAnnoteAdapter(t *testing.T) { t.Errorf("Expected positive min speakers, got: %d", minSpeakers) } - // Test parameter validation - should require HF token + // Test parameter validation - hf_token is optional at validation time + // (can be provided via HF_TOKEN environment variable at runtime) paramsWithoutToken := map[string]interface{}{ "min_speakers": 2, "max_speakers": 4, } - if err := adapter.ValidateParameters(paramsWithoutToken); err == nil { - t.Error("Parameters without HF token should fail validation") + if err := adapter.ValidateParameters(paramsWithoutToken); err != nil { + t.Errorf("Parameters without HF token should pass validation (token can come from env var): %v", err) } - // Test with token + // Test with token explicitly provided paramsWithToken := map[string]interface{}{ "hf_token": "dummy_token", "min_speakers": 2,