Skip to content

Commit e7f644c

Browse files
committed
test: speed up model fallback tests
1 parent ae52be9 commit e7f644c

2 files changed

Lines changed: 17 additions & 6 deletions

File tree

src/agents/model-fallback.test.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ import {
2323
FallbackSummaryError,
2424
testing,
2525
runWithImageModelFallback,
26-
runWithModelFallback,
26+
runWithModelFallback as runWithModelFallbackBase,
2727
} from "./model-fallback.js";
2828
import { classifyEmbeddedPiRunResultForModelFallback } from "./pi-embedded-runner/result-fallback-classifier.js";
2929
import type { EmbeddedPiRunResult } from "./pi-embedded-runner/types.js";
@@ -166,6 +166,10 @@ vi.mock("./model-fallback-auth.runtime.js", () => authRuntimeMock.runtime);
166166
const makeCfg = makeModelFallbackCfg;
167167
let authTempRoot = "";
168168
let authTempCounter = 0;
169+
const emptyManifestPlugins = [] as const;
170+
171+
const runWithModelFallback: typeof runWithModelFallbackBase = (params) =>
172+
runWithModelFallbackBase({ manifestPlugins: emptyManifestPlugins, ...params });
169173

170174
beforeAll(() => {
171175
setDefaultPluginMetadataSnapshot();

src/agents/model-fallback.ts

Lines changed: 12 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -732,23 +732,30 @@ function resolveFallbackCandidateCacheKey(
732732
}
733733
const workspaceDir = getActivePluginRegistryWorkspaceDirFromState();
734734
const env = process.env;
735+
const pluginMetadata = getCurrentPluginMetadataSnapshot({
736+
env,
737+
workspaceDir,
738+
allowWorkspaceScopedSnapshot: true,
739+
});
740+
const providerLoadMetadata = getCurrentPluginMetadataSnapshot({
741+
config: params.cfg,
742+
env,
743+
workspaceDir,
744+
allowWorkspaceScopedSnapshot: true,
745+
});
735746
if (
736747
isPluginProvidersLoadInFlight({
737748
config: params.cfg,
738749
workspaceDir,
739750
env,
751+
...(providerLoadMetadata ? { pluginMetadataSnapshot: providerLoadMetadata } : {}),
740752
activate: false,
741753
bundledProviderAllowlistCompat: true,
742754
bundledProviderVitestCompat: true,
743755
})
744756
) {
745757
return null;
746758
}
747-
const pluginMetadata = getCurrentPluginMetadataSnapshot({
748-
env,
749-
workspaceDir,
750-
allowWorkspaceScopedSnapshot: true,
751-
});
752759
const registryState = getPluginRegistryState();
753760
return JSON.stringify({
754761
provider: params.provider,

0 commit comments

Comments
 (0)