Repository navigation
test: document infrastructure and improve loop assertion diagnostics - #59
Merged
petesramek merged 2 commits intoOct 28, 2026
Merged
petesramek merged 2 commits into
petesramek merged 2 commits into
Conversation
Document test helpers, platform stubs, compile-contract probes and capacity assumptions. Loop-based assertions now report the failing index, length or value via TEST_ASSERT_*_MESSAGE and a diag() helper. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
8 of 9 tasks
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
14 tasks done
petesramek
added a commit
that referenced
this pull request
Oct 28, 2026
#62) * status.h replaced with ackresult * ackmessage rewritten * updated ackmessage * added telemetryevent.h * updated stats.h * updated stats.h * updated messagetype.h * updated version.h * updated handshakemessage.h * update logmessage.h * update loglevel * updated state.h * updated fletche16.h * update cobscodec.h * updated packet.h * removed packed.h * rtemoved duplicate code * updated tinylink.h * updated adapters * removed plan folder * added skills * removed tests * test: add native Fletcher-16 suite (#32) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * test: add comprehensive TinyLink native suite (#33) * test: add native Fletcher-16 suite Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * test: add comprehensive TinyLink native suite Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * test: cover remaining native components (#34) * test: add native Fletcher-16 suite Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * test: add comprehensive TinyLink native suite Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * test: cover remaining native components Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * readme * switch prompt and result order * merged src structure with large project flat structure issue. * updated model * feat(adapter): report transport write results (#52) Add a shared result type and propagate accepted-byte counts from all built-in adapters. Expand native adapter stubs and tests for complete, partial, failed, and zero-length writes. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * feat(engine): handle transport write results (#53) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * feat(engine): require trivially copyable payloads (#54) Enforce trivially copyable payload types on all toolchains including AVR, document opaque-byte portability limits, and add compile-contract checks. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * fix: saturating engine telemetry and ACK sequence validity (layer 4) (#55) * Saturate engine telemetry and validate ACK sequence Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * docs: changelog for stats saturation and ACK validity Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * fix(lifecycle): recover from transport loss (#56) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * test(adapters): cover POSIX adapter with PTYs (#57) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * test: organize suites by responsibility and split TinyLink tests (#58) Move suites to test/suites, isolate platform stubs under test/support/stubs, and split the TinyLink suite into behavior files. All 142 registrations preserved. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * test: document infrastructure and improve loop assertion diagnostics (#59) * test: document infrastructure and improve loop assertion diagnostics Document test helpers, platform stubs, compile-contract probes and capacity assumptions. Loop-based assertions now report the failing index, length or value via TEST_ASSERT_*_MESSAGE and a diag() helper. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * build(test): enforce strict native warnings and add sanitizer CI (#60) Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> * removed cod etesting skill * bumped version to 0.5.0 * fix: reconcile remedation merge with main duplicates (#63) Restore the verified stacked implementation (src, tests, tools, CI) where the main merge left duplicate TinyWriteResult/test files and broken TinyLink.h. Drop duplicate README write-result section and obsolete scripts/ probe. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> --------- Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
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
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.
Description
Layer 8 of the TinyLink remediation stack (base:
petesramek-test-suite-organization, #58). Documents the test infrastructure (FakeAdapter, helpers/fixture, platform stubs, compile-contract probes, capacity assumptions: max payload 64, packet ≤ 69 bytes, 8192-byte fake buffers) intest/README.mdand header comments, and makes loop-based assertions report the failing index/length/value throughTEST_ASSERT_*_MESSAGEand a newtinylink_test::diag()helper (test/support/tinylink/TestDiagnostics.h). No production code or test coverage changes; no assertions weakened.Closes #N/A
Type of Change
Checklist
pio test -e native).CHANGELOG.mdentry under[Unreleased].Testing Done
Windows host, PlatformIO native + Unity:
pio test -e native→ 125 test cases: 124 succeeded, 1 skipped (POSIX PTY suite is ignored on Windows), 0 failedpython tools/compile_contract/check.py→ PASS payload_valid.cpp, PASS payload_not_trivially_copyable.cpp, PASS payload_too_large.cppgit diff --check→ clean (exit 0)The POSIX suite edits compile only on Linux and were not compiled locally; they rely on the Linux CI job.
Screenshots / Logs (if applicable)
N/A