From 00759e799453905a0a07648d0076f7859101a769 Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Wed, 27 May 2026 23:14:33 +0700 Subject: [PATCH] Fix provider-selection test for synthetic default row The selector rework now synthesizes a "Default" row for ready enabled providers without explicit models, so they share the same `kind: "models"` shape as providers with real models. Updated the stale assertion that still expected the old `providerDefault` kind. --- .../provider-selection.test.ts | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/packages/app/src/provider-selection/provider-selection.test.ts b/packages/app/src/provider-selection/provider-selection.test.ts index a20901ace..50d45bc87 100644 --- a/packages/app/src/provider-selection/provider-selection.test.ts +++ b/packages/app/src/provider-selection/provider-selection.test.ts @@ -68,7 +68,7 @@ describe("combined model selector data", () => { ]); }); - it("represents ready enabled providers without explicit models as provider-default selection", () => { + it("synthesizes a default model row for ready enabled providers without explicit models", () => { expect( buildSelectableProviderSelectorProviders([ snapshotEntry({ @@ -81,7 +81,20 @@ describe("combined model selector data", () => { { id: "deepseek-tui", label: "DeepSeek TUI", - modelSelection: { kind: "providerDefault", label: "Default" }, + modelSelection: { + kind: "models", + rows: [ + { + favoriteKey: "deepseek-tui:", + provider: "deepseek-tui", + providerLabel: "DeepSeek TUI", + modelId: "", + modelLabel: "Default", + description: undefined, + isDefault: true, + }, + ], + }, }, ]); });