mirror of
https://github.com/logos-co/logos-cpp-sdk.git
synced 2026-08-31 01:31:10 +00:00
Dario Gabriel LipicarandClaude Opus 5
885550b78c
#117 left tests/sdk/test_logos_json_bytes.cpp including logos_codec.h and logos_lp_client.h, which live in logos-protocol. That needed a target_include_directories on LOGOS_PROTOCOL_ROOT, and it does not resolve in this build: `fatal error: 'logos_codec.h' file not found`. The repo's own test target has been red on master since that merge. The right fix is not more include plumbing. The assertions were testing b64UrlEncode / bytesToJson, which this repo no longer defines — #117 moved them to their canonical home — so they moved with them, to logos-protocol tests/protocol/test_codec.cpp, in six cases covering more than the original did. Not replaced: a direct unit test of logos::jsonToBytes, the three-line lenient wrapper in logos_lp_client.h. It is built entirely on isTaggedBytes and b64UrlDecode, both now covered where they are defined, and testing it here would reintroduce exactly the cross-repo include this commit removes. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>