Skip to content

Commit 4ef361b

Browse files
committed
fix: pass suppressWarnings through to executeXcodeBuildCommand in test_macos, test_device, and clean
1 parent 9b15eb0 commit 4ef361b

File tree

9 files changed

+32
-6
lines changed

9 files changed

+32
-6
lines changed

src/mcp/tools/device/__tests__/build_device.test.ts

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,12 @@ describe('build_device plugin', () => {
3737
expect(schema.safeParse({ projectPath: '/path/to/MyProject.xcodeproj' }).success).toBe(false);
3838

3939
const schemaKeys = Object.keys(buildDevice.schema).sort();
40-
expect(schemaKeys).toEqual(['derivedDataPath', 'extraArgs', 'preferXcodebuild', 'suppressWarnings']);
40+
expect(schemaKeys).toEqual([
41+
'derivedDataPath',
42+
'extraArgs',
43+
'preferXcodebuild',
44+
'suppressWarnings',
45+
]);
4146
});
4247
});
4348

src/mcp/tools/device/test_device.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -214,6 +214,7 @@ export async function testDeviceLogic(
214214
configuration: params.configuration ?? 'Debug',
215215
derivedDataPath: params.derivedDataPath,
216216
extraArgs,
217+
suppressWarnings: params.suppressWarnings,
217218
},
218219
{
219220
platform: (params.platform as XcodePlatform) || XcodePlatform.iOS,

src/mcp/tools/macos/__tests__/build_macos.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,9 @@ describe('build_macos plugin', () => {
4646
expect(schema.safeParse({ preferXcodebuild: 'yes' }).success).toBe(false);
4747

4848
const schemaKeys = Object.keys(buildMacOS.schema).sort();
49-
expect(schemaKeys).toEqual(['derivedDataPath', 'extraArgs', 'preferXcodebuild', 'suppressWarnings'].sort());
49+
expect(schemaKeys).toEqual(
50+
['derivedDataPath', 'extraArgs', 'preferXcodebuild', 'suppressWarnings'].sort(),
51+
);
5052
});
5153
});
5254

src/mcp/tools/macos/__tests__/build_run_macos.test.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,9 @@ describe('build_run_macos', () => {
3939
expect(schema.safeParse({ preferXcodebuild: 'yes' }).success).toBe(false);
4040

4141
const schemaKeys = Object.keys(tool.schema).sort();
42-
expect(schemaKeys).toEqual(['derivedDataPath', 'extraArgs', 'preferXcodebuild', 'suppressWarnings'].sort());
42+
expect(schemaKeys).toEqual(
43+
['derivedDataPath', 'extraArgs', 'preferXcodebuild', 'suppressWarnings'].sort(),
44+
);
4345
});
4446
});
4547

src/mcp/tools/macos/__tests__/test_macos.test.ts

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,13 @@ describe('test_macos plugin (unified)', () => {
4747

4848
const schemaKeys = Object.keys(testMacos.schema).sort();
4949
expect(schemaKeys).toEqual(
50-
['derivedDataPath', 'extraArgs', 'preferXcodebuild', 'suppressWarnings', 'testRunnerEnv'].sort(),
50+
[
51+
'derivedDataPath',
52+
'extraArgs',
53+
'preferXcodebuild',
54+
'suppressWarnings',
55+
'testRunnerEnv',
56+
].sort(),
5157
);
5258
});
5359
});

src/mcp/tools/macos/test_macos.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -265,6 +265,7 @@ export async function testMacosLogic(
265265
configuration: params.configuration ?? 'Debug',
266266
derivedDataPath: params.derivedDataPath,
267267
extraArgs,
268+
suppressWarnings: params.suppressWarnings,
268269
},
269270
{
270271
platform: XcodePlatform.macOS,

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

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,9 @@ describe('build_run_sim tool', () => {
4545
expect(schema.safeParse({ preferXcodebuild: 'yes' }).success).toBe(false);
4646

4747
const schemaKeys = Object.keys(buildRunSim.schema).sort();
48-
expect(schemaKeys).toEqual(['derivedDataPath', 'extraArgs', 'preferXcodebuild', 'suppressWarnings'].sort());
48+
expect(schemaKeys).toEqual(
49+
['derivedDataPath', 'extraArgs', 'preferXcodebuild', 'suppressWarnings'].sort(),
50+
);
4951
expect(schemaKeys).not.toContain('scheme');
5052
expect(schemaKeys).not.toContain('simulatorName');
5153
expect(schemaKeys).not.toContain('projectPath');

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

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,13 @@ describe('test_sim tool', () => {
4646

4747
const schemaKeys = Object.keys(testSim.schema).sort();
4848
expect(schemaKeys).toEqual(
49-
['derivedDataPath', 'extraArgs', 'preferXcodebuild', 'suppressWarnings', 'testRunnerEnv'].sort(),
49+
[
50+
'derivedDataPath',
51+
'extraArgs',
52+
'preferXcodebuild',
53+
'suppressWarnings',
54+
'testRunnerEnv',
55+
].sort(),
5056
);
5157
});
5258
});

src/mcp/tools/utilities/clean.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -124,6 +124,7 @@ export async function cleanLogic(
124124
configuration: params.configuration ?? 'Debug',
125125
derivedDataPath: params.derivedDataPath,
126126
extraArgs: params.extraArgs,
127+
suppressWarnings: params.suppressWarnings,
127128
};
128129

129130
// For clean operations, simulator platforms should be mapped to their device equivalents

0 commit comments

Comments
 (0)