Skip to content

Commit b22cca3

Browse files
committed
fix(tests): Update simulator/debugger failure assertions to new output format
The output standardization refactor (b6bde24) strips known prefixes (e.g. 'Install app in simulator operation failed:') inside buildInstall/ Launch/StopFailure and renders the header as 'Failed to install/launch/ stop app.' with the raw diagnostic under 'Errors (N):'. Update the stale assertions in install/launch/stop_app_sim and debug_attach_sim tests to match the rendered text, keeping the diagnostic-bullet assertions intact.
1 parent 7ddc117 commit b22cca3

4 files changed

Lines changed: 7 additions & 7 deletions

File tree

src/mcp/tools/debugging/__tests__/debugging-tools.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -266,7 +266,7 @@ describe('debug_attach_sim', () => {
266266

267267
expect(result.isError).toBe(true);
268268
const text = allText(result);
269-
expect(text).toContain('Failed to resolve simulator PID');
269+
expect(text).toContain('Failed to attach debugger');
270270
});
271271

272272
it('should include nextStepParams on success', async () => {

src/mcp/tools/simulator/__tests__/install_app_sim.test.ts

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -292,7 +292,7 @@ describe('install_app_sim tool', () => {
292292
);
293293

294294
const text = allText(result);
295-
expect(text).toContain('Install app in simulator operation failed');
295+
expect(text).toContain('Failed to install app');
296296
expect(text).toContain('Install failed');
297297
expect(result.isError).toBe(true);
298298
});
@@ -316,7 +316,7 @@ describe('install_app_sim tool', () => {
316316
);
317317

318318
const text = allText(result);
319-
expect(text).toContain('Install app in simulator operation failed');
319+
expect(text).toContain('Failed to install app');
320320
expect(text).toContain('Command execution failed');
321321
expect(result.isError).toBe(true);
322322
});
@@ -340,7 +340,7 @@ describe('install_app_sim tool', () => {
340340
);
341341

342342
const text = allText(result);
343-
expect(text).toContain('Install app in simulator operation failed');
343+
expect(text).toContain('Failed to install app');
344344
expect(text).toContain('String error');
345345
expect(result.isError).toBe(true);
346346
});

src/mcp/tools/simulator/__tests__/launch_app_sim.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -255,7 +255,7 @@ describe('launch_app_sim tool', () => {
255255
);
256256

257257
const text = result.content.map((c) => (c.type === 'text' ? c.text : '')).join('\n');
258-
expect(text).toContain('Launch app in simulator operation failed');
258+
expect(text).toContain('Failed to launch app');
259259
expect(text).toContain('Launch failed');
260260
expect(result.isError).toBe(true);
261261
});

src/mcp/tools/simulator/__tests__/stop_app_sim.test.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -233,7 +233,7 @@ describe('stop_app_sim tool', () => {
233233
);
234234

235235
const text = allText(result);
236-
expect(text).toContain('Stop app in simulator operation failed');
236+
expect(text).toContain('Failed to stop app');
237237
expect(text).toContain('Simulator not found');
238238
expect(result.isError).toBe(true);
239239
});
@@ -283,7 +283,7 @@ describe('stop_app_sim tool', () => {
283283
);
284284

285285
const text = allText(result);
286-
expect(text).toContain('Stop app in simulator operation failed');
286+
expect(text).toContain('Failed to stop app');
287287
expect(text).toContain('Unexpected error');
288288
expect(result.isError).toBe(true);
289289
});

0 commit comments

Comments
 (0)