Add local-only on-target mco_TLS smoke test for mqtt.library - #16
Merged
Conversation
tests/library/libtls.c + tls-run.sh exercise mqtt.library's real TLS API (OpenLibrary/CreateClient with mco_TLS+mco_TLSInsecure/Connect/Subscribe/ GetMessage/Publish/Disconnect/DeleteClient) against a real TLS Mosquitto under Copperline, mirroring library-net-smoke's shape. Unlike the other library-*-smoke targets this one is local-only, not a CI/release gate: AmiSSL needs a real AmigaOS Devs:/Libs: install (its own AmiSSL: assign) and a real Kickstart ROM, neither of which the bundled-AROS/HostSocket boot the other tests use has, or which CI can be handed. tests/library/ README.md documents building the required image with amibake and running the test locally. Verified: full green run (all 8 checks + the host TLS observer receiving the publish) against a real TLS Mosquitto broker. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01QWq1CRLCQy9AaD9UtLRSM9
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
tests/library/libtls.c+tests/library/tls-run.sh: an on-target Copperline test exercising mqtt.library's real TLS API (mco_TLS/mco_TLSInsecure) against a real TLS Mosquitto broker, mirroringlibrary-net-smoke's shape/PASS-FAIL-RESULT-END contract.library-*-smoketargets, this one is local-only, not wired into CI: AmiSSL needs a real AmigaOSDevs:/Libs:install (its ownAmiSSL:assign) and a real Kickstart ROM, neither of which the bundled-AROS/HostSocket boot the other tests use provides, or which CI can be handed.tests/library/README.mddocuments building the required boot image with amibake and running the test.make libtls-m68k/make library-tls-smokeMakefile targets, referenced in the top-of-file help comment.Part of issue #3 (Phase 3: TLS support via AmiSSL) - the remaining on-target verification piece mentioned in the last progress comment there.
Test plan
build/libtls,build/mqtt.library(withmake fetch-amissl-sdksupport), andbuild/mqtt_pub-hostvia the m68k Docker toolchain and hostcc.sh tests/library/tls-run.shagainst a real amibake-built AmigaOS 3.2.2 + AmiSSL 5.27 image and a scratch TLS Mosquitto broker locally: full green run - all 8 on-target checks passed (openlibrary-nonnullthroughpublish-qos1-ok), plus the host TLS observer received the published message.ci.yml(local-only, as documented).🤖 Generated with Claude Code
https://claude.ai/code/session_01QWq1CRLCQy9AaD9UtLRSM9