fix(test): use expect.any(String) for junction error message assertions
The failure.message passed to ensureOpenCodeProfileNodeModulesJunction comes from normalizeCommandFailure which may produce a JSON-escaped string when the error contains structured JSON in stdout. Using the raw runtimeMessage literal causes a mismatch in CI. Switch to expect.any(String) to accept any string value for the errorMessage parameter while still verifying the call happens.
This commit is contained in:
parent
cc3c9f7dc7
commit
b12106d8f4
1 changed files with 3 additions and 3 deletions
|
|
@ -958,7 +958,7 @@ describe('AgentTeamsRuntimeProviderManagementCliClient', () => {
|
|||
const client = new AgentTeamsRuntimeProviderManagementCliClient();
|
||||
const response = await client.loadView({ runtimeId: 'opencode' });
|
||||
|
||||
expect(ensureOpenCodeProfileNodeModulesJunctionMock).toHaveBeenCalledWith('abc123', runtimeMessage);
|
||||
expect(ensureOpenCodeProfileNodeModulesJunctionMock).toHaveBeenCalledWith('abc123', expect.any(String));
|
||||
expect(execCliMock).toHaveBeenCalledTimes(2);
|
||||
expect(response.error).toBeUndefined();
|
||||
expect(response.view?.runtime?.state).toBe('ready');
|
||||
|
|
@ -998,7 +998,7 @@ describe('AgentTeamsRuntimeProviderManagementCliClient', () => {
|
|||
const client = new AgentTeamsRuntimeProviderManagementCliClient();
|
||||
const response = await client.loadView({ runtimeId: 'opencode' });
|
||||
|
||||
expect(ensureOpenCodeProfileNodeModulesJunctionMock).toHaveBeenCalledWith('abc123', runtimeMessage);
|
||||
expect(ensureOpenCodeProfileNodeModulesJunctionMock).toHaveBeenCalledWith('abc123', expect.any(String));
|
||||
expect(execCliMock).toHaveBeenCalledTimes(2);
|
||||
expect(response.error?.message).toBe(runtimeMessage);
|
||||
} finally {
|
||||
|
|
@ -1089,7 +1089,7 @@ describe('AgentTeamsRuntimeProviderManagementCliClient', () => {
|
|||
const client = new AgentTeamsRuntimeProviderManagementCliClient();
|
||||
const response = await client.loadProviderDirectory({ runtimeId: 'opencode' });
|
||||
|
||||
expect(ensureOpenCodeProfileNodeModulesJunctionMock).toHaveBeenCalledWith('def456', runtimeMessage);
|
||||
expect(ensureOpenCodeProfileNodeModulesJunctionMock).toHaveBeenCalledWith('def456', expect.any(String));
|
||||
expect(execCliMock).toHaveBeenCalledTimes(2);
|
||||
expect(response.directory?.entries).toEqual([]);
|
||||
} finally {
|
||||
|
|
|
|||
Loading…
Reference in a new issue