Skip to content

Commit f471d73

Browse files
committed
Merge branch 'feat/test/check-ignore' of github.com:VeryGoodOpenSource/very_good_cli into feat/test/check-ignore
2 parents 2e870eb + d8b9f9e commit f471d73

2 files changed

Lines changed: 19 additions & 5 deletions

File tree

lib/src/mcp/mcp_server.dart

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -325,9 +325,7 @@ Only one value can be selected.
325325
if (args['recursive'] == true) {
326326
cliArgs.add('-r');
327327
}
328-
if (args['optimization'] == true) {
329-
cliArgs.add('--optimization');
330-
} else {
328+
if (args['optimization'] == false) {
331329
cliArgs.add('--no-optimization');
332330
}
333331
if (args['concurrency'] != null) {

test/src/mcp/mcp_server_test.dart

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -284,12 +284,29 @@ void main() {
284284
});
285285

286286
group('Tool: test', () {
287-
test('handles basic case with --no-optimization', () async {
287+
test('handles default args (no flags added)', () async {
288288
await sendRequest(
289289
CallToolRequest.methodName,
290290
_params(CallToolRequest(name: 'test', arguments: {})),
291291
);
292292

293+
final capturedArgs =
294+
verify(() => mockCommandRunner.run(captureAny())).captured.first
295+
as List<String>;
296+
expect(capturedArgs, ['test']);
297+
});
298+
299+
test('passes --no-optimization when explicitly false', () async {
300+
await sendRequest(
301+
CallToolRequest.methodName,
302+
_params(
303+
CallToolRequest(
304+
name: 'test',
305+
arguments: {'optimization': false},
306+
),
307+
),
308+
);
309+
293310
final capturedArgs =
294311
verify(() => mockCommandRunner.run(captureAny())).captured.first
295312
as List<String>;
@@ -333,7 +350,6 @@ void main() {
333350
'test',
334351
'--coverage',
335352
'-r',
336-
'--optimization',
337353
'-j',
338354
'8',
339355
'-t',

0 commit comments

Comments
 (0)