Skip to content

Commit 44c6499

Browse files
committed
Fix assertion counts in client_api tests for Debug builds
The mock servers return different counts than the tests expected: - tools: 4 (was 3) - resources: 3 (was 2) These assertions only fail in Debug builds since Release builds define NDEBUG which disables assert().
1 parent 2194abd commit 44c6499

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

tests/client_api.cpp

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -308,12 +308,12 @@ void test_list_tools() {
308308

309309
auto tools = c.list_tools();
310310

311-
assert(tools.size() == 3);
311+
assert(tools.size() == 4);
312312
assert(tools[0].name == "add");
313313
assert(tools[0].description.value_or("") == "Add two numbers");
314314
assert(tools[1].name == "greet");
315315

316-
std::cout << " [PASS] list_tools() returns 3 tools\n";
316+
std::cout << " [PASS] list_tools() returns 4 tools\n";
317317
}
318318

319319
void test_list_tools_mcp() {
@@ -324,7 +324,7 @@ void test_list_tools_mcp() {
324324

325325
auto result = c.list_tools_mcp();
326326

327-
assert(result.tools.size() == 3);
327+
assert(result.tools.size() == 4);
328328
assert(!result.nextCursor.has_value()); // No pagination in this test
329329

330330
std::cout << " [PASS] list_tools_mcp() returns ListToolsResult\n";
@@ -401,7 +401,7 @@ void test_list_resources() {
401401

402402
auto resources = c.list_resources();
403403

404-
assert(resources.size() == 2);
404+
assert(resources.size() == 3);
405405
assert(resources[0].uri == "file:///readme.txt");
406406
assert(resources[0].name == "readme.txt");
407407
assert(resources[0].mimeType.value_or("") == "text/plain");

0 commit comments

Comments
 (0)