test: add 63 advanced tests, eliminate all compiler warnings, expand test framework — bump to v0.28.0

- 63 new tests: framework macros, resource template edge cases, content builders,
  server API surface, JSON-RPC dispatch, rate limiter, heap monitor, annotations
- Eliminated all 30 compiler warnings from mock/test code
- 8 new assertion macros: ASSERT_TRUE/FALSE, GT/LT/GE/LE, STR_EQ, NEAR
- Total: 946 tests (883 → 946), 0 warnings, 0 errors

Author: Nicola Spieser

.gitignore

@@ -19,3 +19,4 @@ test/native/test_auth_platform
 test/native/test_robustness
 test/native/test_session
 test/native/test_content_transport
+test/native/test_advanced

CHANGELOG.md

@@ -2,6 +2,36 @@
 
 All notable changes to this project will be documented in this file.
 
+## [0.28.0] - 2026-02-21
+
+### Tests
+- Added **63 new advanced tests** (`test_advanced.cpp`):
+  - Test framework macro validation (ASSERT_TRUE/FALSE, GT/LT/GE/LE, STR_EQ, NEAR)
+  - Resource template matching: single/multiple vars, edge cases (empty var, wrong prefix, shorter URI, adjacent vars, numbers in vars, literal-only templates)
+  - MCPContent factory tests: makeText, makeImage, makeAudio, makeResource, makeResourceBlob
+  - MCPToolResult builder: text, error, chaining, image+alt, audio+desc, JSON serialization
+  - Tool annotations: spec defaults (destructive=true, openWorld=true), readOnly clears destructive, fluent API
+  - Server API surface: name/port, add/remove tool/resource/prompt/root/template, endpoint/mDNS/pageSize config, multiple tools
+  - JSON-RPC dispatch: ping, method not found, invalid JSON, initialize, tools/list, tools/call, resources/list, prompts/list, logging/setLevel, roots/list
+  - RateLimiter: disabled default, burst exact, stats, disable, config values
+  - HeapMonitor: initial state, threshold, usage percent
+  - Version/protocol format validation
+- **Total: 946 tests** (883 → 946)
+
+### Code Quality
+- **Eliminated all compiler warnings** (previously 30 warnings from mock/test code):
+  - Added `(void)param` casts for all unused parameters in `arduino_mock.h` mocks
+  - Removed unused `_port` field from `WiFiServer` mock
+  - Added explicit copy constructor for `JsonVariant` mock (fixes `-Wdeprecated-copy`)
+  - Guarded `MCPD_TEST` macro redefinition in `test_mcp_http.cpp`
+  - Removed unused `_mock_millis_offset` variable in `test_session.cpp`
+  - Fixed string literal comparison in `test_auth_platform.cpp` (`ASSERT_STR_CONTAINS` instead of `==`)
+  - Added `-Wno-unused-parameter` to test Makefile (test lambdas naturally have unused params)
+- **Expanded test framework** with 8 new assertion macros:
+  - `ASSERT_TRUE`, `ASSERT_FALSE`, `ASSERT_GT`, `ASSERT_LT`, `ASSERT_GE`, `ASSERT_LE`
+  - `ASSERT_STR_EQ` (proper C-string comparison via `strcmp`)
+  - `ASSERT_NEAR` (floating-point approximate equality)
+
 ## [0.27.5] - 2026-02-21
 
 ### Tests

README.md

@@ -33,7 +33,7 @@
 |---|:---:|:---:|:---:|
 | Runs on the MCU | ✅ | ✅ | ❌ CLI tool |
 | MCP spec compliant | ✅ 2025-03-26 | ❌ custom WS | ❌ |
-| Actually compiles | ✅ 883 tests | ❌ self-described | N/A |
+| Actually compiles | ✅ 946 tests | ❌ self-described | N/A |
 | Streamable HTTP + SSE | ✅ | ❌ | ❌ |
 | WebSocket transport | ✅ | ✅ | ❌ |
 | Claude Desktop bridge | ✅ | ❌ | ❌ |

src/mcpd.h

@@ -42,7 +42,7 @@
 #include "MCPTransportBLE.h"
 #endif
 
-#define MCPD_VERSION "0.27.5"
+#define MCPD_VERSION "0.28.0"
 #define MCPD_MCP_PROTOCOL_VERSION "2025-03-26"
 
 namespace mcpd {

test/arduino_mock.h

@@ -189,10 +189,10 @@ inline long map(long x, long in_min, long in_max, long out_min, long out_max) {
 }
 
 // LEDC stubs
-inline void ledcSetup(int ch, int freq, int res) {}
-inline void ledcAttachPin(int pin, int ch) {}
-inline void ledcWrite(int ch, int duty) {}
-inline void ledcDetachPin(int pin) {}
+inline void ledcSetup(int ch, int freq, int res) { (void)ch; (void)freq; (void)res; }
+inline void ledcAttachPin(int pin, int ch) { (void)pin; (void)ch; }
+inline void ledcWrite(int ch, int duty) { (void)ch; (void)duty; }
+inline void ledcDetachPin(int pin) { (void)pin; }
 
 // ── Timing ─────────────────────────────────────────────────────────────
 
@@ -206,7 +206,7 @@ inline unsigned long pulseIn(int pin, int state, unsigned long timeout = 1000000
 
 // ── Random ─────────────────────────────────────────────────────────────
 
-inline void randomSeed(unsigned long seed) {}
+inline void randomSeed(unsigned long seed) { (void)seed; }
 inline long random(long max) { return rand() % max; }
 inline long random(long min, long max) { return min + rand() % (max - min); }
 
@@ -226,6 +226,7 @@ inline std::map<int, std::function<void()>>& _mockInterruptHandlers() {
 }
 
 inline void attachInterrupt(int pin, void (*isr)(), int mode) {
+    (void)mode;
     _mockInterruptHandlers()[pin] = isr;
 }
 
@@ -280,20 +281,20 @@ struct HardwareSerial {
     size_t _rxPos = 0;
     unsigned long _timeout = 1000;
 
-    void begin(long baud) { (void)baud; }
-    void begin(long baud, int config, int rxPin, int txPin) { (void)baud; (void)config; (void)rxPin; (void)txPin; }
+    void begin(long baud_) { (void)baud_; }
+    void begin(long baud_, int config, int rxPin, int txPin) { (void)baud_; (void)config; (void)rxPin; (void)txPin; }
     void end() {}
     void setTimeout(unsigned long t) { _timeout = t; }
     int available() { return (int)(_rxBuffer.size() - _rxPos); }
     int read() {
         if (_rxPos < _rxBuffer.size()) return (unsigned char)_rxBuffer[_rxPos++];
         return -1;
     }
-    size_t write(uint8_t b) { (void)b; return 1; }
+    size_t write(uint8_t b_) { (void)b_; return 1; }
     size_t print(const String& s) { return s.length(); }
     size_t print(const char* s) { return strlen(s); }
     template<typename... Args>
-    void printf(const char* fmt, Args... args) { (void)fmt; }
+    void printf(const char* fmt, Args... args) { (void)fmt; ((void)args, ...); }
 
     // Test helper
     void _setRxBuffer(const std::string& data) { _rxBuffer = data; _rxPos = 0; }
@@ -305,7 +306,7 @@ inline HardwareSerial Serial2;
 // ── Serial Mock ────────────────────────────────────────────────────────
 
 struct SerialMock {
-    void begin(int baud) {}
+    void begin(int baud) { (void)baud; }
     void print(const char* s) { fprintf(stderr, "%s", s); }
     void print(const String& s) { fprintf(stderr, "%s", s.c_str()); }
     void println(const char* s = "") { fprintf(stderr, "%s\n", s); }
@@ -359,8 +360,8 @@ struct WiFiMock {
     int channel(int i) { return 1 + (i % 11); }
     int scanNetworks() { return 3; }
     void scanDelete() {}
-    int encryptionType(int i) { return 4; }
-    void begin(const char* ssid, const char* pass) {}
+    int encryptionType(int i) { (void)i; return 4; }
+    void begin(const char* ssid, const char* pass) { (void)ssid; (void)pass; }
 };
 inline WiFiMock WiFi;
 
@@ -396,12 +397,10 @@ class WiFiClient {
 
 class WiFiServer {
 public:
-    WiFiServer(uint16_t port) : _port(port) {}
+    WiFiServer(uint16_t port) { (void)port; }
     void begin() {}
     void stop() {}
     WiFiClient available() { return WiFiClient(); }
-private:
-    uint16_t _port;
 };
 
 // ── WebServer Mock ─────────────────────────────────────────────────────
@@ -442,7 +441,7 @@ class WebServer {
 
     WiFiClient client() { return WiFiClient(); }
 
-    void collectHeaders(const char** headers, int count) {}
+    void collectHeaders(const char** headers, int count) { (void)headers; (void)count; }
 
     String arg(const char* name) {
         if (strcmp(name, "plain") == 0) return _body;
@@ -486,9 +485,9 @@ class WebServer {
 // ── mDNS Mock ──────────────────────────────────────────────────────────
 
 struct MDNSMock {
-    bool begin(const char* name) { return true; }
-    void addService(const char* service, const char* proto, uint16_t port) {}
-    void addServiceTxt(const char* service, const char* proto, const char* key, const char* value) {}
+    bool begin(const char* name) { (void)name; return true; }
+    void addService(const char* service, const char* proto, uint16_t port) { (void)service; (void)proto; (void)port; }
+    void addServiceTxt(const char* service, const char* proto, const char* key, const char* value) { (void)service; (void)proto; (void)key; (void)value; }
 };
 inline MDNSMock MDNS;
 
@@ -497,12 +496,12 @@ inline MDNSMock MDNS;
 class TwoWire {
 public:
     void begin() {}
-    void beginTransmission(uint8_t addr) {}
+    void beginTransmission(uint8_t addr) { (void)addr; }
     uint8_t endTransmission() { return 0; }
-    uint8_t requestFrom(uint8_t addr, uint8_t count) { return count; }
+    uint8_t requestFrom(uint8_t addr, uint8_t count) { (void)addr; return count; }
     int available() { return 0; }
     int read() { return 0; }
-    void write(uint8_t b) {}
+    void write(uint8_t b) { (void)b; }
 };
 inline TwoWire Wire;
 

test/mock_includes/ArduinoJson.h

@@ -62,6 +62,7 @@ class JsonVariant {
     std::shared_ptr<JsonNode> _node;
 
     JsonVariant() : _node(std::make_shared<JsonNode>()) {}
+    JsonVariant(const JsonVariant&) = default;
     JsonVariant(std::shared_ptr<JsonNode> n) : _node(n ? n : std::make_shared<JsonNode>()) {}
 
     bool isNull() const { return !_node || _node->type == JsonNodeType::Null; }

test/native/.gitignore

@@ -0,0 +1 @@
+test_advanced

test/native/Makefile

@@ -1,9 +1,9 @@
 CXX ?= g++
-CXXFLAGS = -std=c++17 -Wall -Wextra -DMCPD_TEST -pthread
+CXXFLAGS = -std=c++17 -Wall -Wextra -Wno-unused-parameter -DMCPD_TEST -pthread
 INCLUDES = -I../../src -I../mock_includes -I..
 
 # Targets
-TESTS = test_jsonrpc test_tools test_mcp_http test_infrastructure test_modules test_auth_platform test_robustness test_session test_content_transport
+TESTS = test_jsonrpc test_tools test_mcp_http test_infrastructure test_modules test_auth_platform test_robustness test_session test_content_transport test_advanced
 
 .PHONY: all clean test
 
@@ -36,6 +36,9 @@ test_session: ../test_session.cpp ../arduino_mock.h ../test_framework.h ../../sr
 test_content_transport: ../test_content_transport.cpp ../arduino_mock.h ../test_framework.h ../../src/mcpd.h ../../src/mcpd.cpp
 	$(CXX) $(CXXFLAGS) $(INCLUDES) -o $@ ../test_content_transport.cpp
 
+test_advanced: ../test_advanced.cpp ../arduino_mock.h ../test_framework.h ../../src/mcpd.h ../../src/mcpd.cpp
+	$(CXX) $(CXXFLAGS) $(INCLUDES) -o $@ ../test_advanced.cpp
+
 test: $(TESTS)
 	@echo ""
 	@echo "══════════════════════════════════════════"
@@ -51,6 +54,7 @@ test: $(TESTS)
 	@./test_robustness
 	@./test_session
 	@./test_content_transport
+	@./test_advanced
 	@echo "All test suites completed."
 
 clean:

test/native/test_mcp_http.cpp

@@ -6,7 +6,9 @@
  */
 
 // Must define MCPD_TEST before includes
+#ifndef MCPD_TEST
 #define MCPD_TEST
+#endif
 
 // Include mocks
 #include "../arduino_mock.h"