Skip to content

Commit 2cd9d74

Browse files
authored
Merge pull request #60 from willwade/fix/59-gemini-provider-filter
Fix Gemini provider filtering in voice utilities
2 parents 8e3955d + 4ebfb47 commit 2cd9d74

2 files changed

Lines changed: 15 additions & 10 deletions

File tree

src/__tests__/voice-utils.test.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,14 @@
1+
import { filterByProvider } from "../core/voice-utils";
2+
import { GeminiTTSClient } from "../engines/gemini";
3+
4+
describe("VoiceUtils", () => {
5+
it("filters Gemini unified voices by provider", async () => {
6+
const client = new GeminiTTSClient({ apiKey: "test-api-key" });
7+
const voices = await client.getVoices();
8+
9+
const geminiVoices = filterByProvider(voices, "gemini");
10+
11+
expect(geminiVoices).toHaveLength(voices.length);
12+
expect(geminiVoices.every((voice) => voice.provider === "gemini")).toBe(true);
13+
});
14+
});

src/core/voice-utils.ts

Lines changed: 1 addition & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -33,16 +33,7 @@ export function filterByGender(
3333
*/
3434
export function filterByProvider(
3535
voices: UnifiedVoice[],
36-
provider:
37-
| "azure"
38-
| "google"
39-
| "ibm"
40-
| "elevenlabs"
41-
| "polly"
42-
| "witai"
43-
| "playht"
44-
| "openai"
45-
| "sherpa"
36+
provider: UnifiedVoice["provider"]
4637
): UnifiedVoice[] {
4738
return voices.filter((voice) => voice.provider === provider);
4839
}

0 commit comments

Comments
 (0)