singularity/singularity-forge
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 1

test(tui): update provider-manager tests for confirmation-based removal

Browse source 
Tests now match the new hasAuth guard and double-press r confirmation
flow introduced in the TUI review PR.
This commit is contained in:
Jeremy 2026-04-01 16:24:14 -05:00
parent 92ef605fef
commit a3a2f2e3b3
1 changed files with 30 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

32
src/tests/provider-manager-remove.test.ts
View file

@ -86,7 +86,7 @@ function createComponent(options: {
};
}
test("provider manager removes provider models and refreshes even when no auth is stored", (t) => {
test("provider manager skips remove when provider has no auth", (t) => {
const modelsJsonPath = createTempModelsJsonPath();
const rootDir = join(modelsJsonPath, "..");
t.after(() => rmSync(rootDir, { recursive: true, force: true }));
@ -98,10 +98,35 @@ test("provider manager removes provider models and refreshes even when no auth i
component.handleInput("r");
// No auth means remove is a no-op
assert.deepEqual(removedProviders, []);
assert.deepEqual(readProviders(modelsJsonPath), ["custom"]);
assert.equal(getRefreshCalls(), 0);
assert.equal(getRenderCalls(), 0);
});
test("provider manager removes provider models with confirmation when auth is stored", (t) => {
const modelsJsonPath = createTempModelsJsonPath();
const rootDir = join(modelsJsonPath, "..");
t.after(() => rmSync(rootDir, { recursive: true, force: true }));
const { component, removedProviders, getRefreshCalls, getRenderCalls } = createComponent({
modelsJsonPath,
authProviders: ["custom"],
providers: [{ name: "custom", modelIds: ["local-model"] }],
});
// First press enters confirmation mode
component.handleInput("r");
assert.deepEqual(removedProviders, []);
assert.equal((component as any).confirmingRemove, true);
// Second press confirms removal
component.handleInput("r");
assert.deepEqual(removedProviders, ["custom"]);
assert.deepEqual(readProviders(modelsJsonPath), []);
assert.equal(getRefreshCalls(), 1);
assert.equal(getRenderCalls(), 1);
assert.ok(getRenderCalls() >= 2);
assert.ok(!(component as any).providers.some((provider: { name: string; modelCount: number }) =>
provider.name === "custom" || provider.modelCount > 0,
));
@ -125,6 +150,9 @@ test("provider manager clamps selection after removing the selected provider", (
(component as any).selectedIndex = (component as any).providers.findIndex(
(provider: { name: string }) => provider.name === "zeta",
);
// Double-press r to confirm removal
component.handleInput("r");
component.handleInput("r");
assert.deepEqual(readProviders(modelsJsonPath), ["alpha"]);