Skip to content

Add native HistoryCodec unit tests#39

Merged
angeloINTJ merged 1 commit into
angeloINTJ:mainfrom
LorenzoLongaretto:main
Jun 4, 2026
Merged

Add native HistoryCodec unit tests#39
angeloINTJ merged 1 commit into
angeloINTJ:mainfrom
LorenzoLongaretto:main

Conversation

@LorenzoLongaretto

Copy link
Copy Markdown
Contributor

Description

Add 22 Unity tests covering encode/decode roundtrip, NaN sentinel preservation, anchor frame boundaries, varint, edge cases, and buffer overflow protection.

  • New env [env:native_history] in platformio.ini
  • New test/test_history_codec/test_main.cpp
  • Compiles production HistoryCodec.cpp directly (no copies)
  • All tests self-contained, no hardware dependencies
  • Run: pio test -e native_history

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring
  • Other (describe):

Testing

Describe how you tested these changes:

  • Compiled with pio run -e pico_w_release
  • Ran unit tests (pio test -e native)
  • Tested on hardware

Checklist

  • Code follows project conventions
  • Comments are in English
  • No new compiler warnings introduced
  • Flash usage remains under 100%

Related Issues

Closes #38

Add 22 Unity tests covering encode/decode roundtrip, NaN sentinel preservation, anchor frame boundaries, varint, edge cases, and buffer overflow protection.

- New env [env:native_history] in platformio.ini
- New test/test_history_codec/test_main.cpp
- Compiles production HistoryCodec.cpp directly (no copies)
- All tests self-contained, no hardware dependencies
Run: pio test -e native_history
@angeloINTJ angeloINTJ merged commit a4c1de5 into angeloINTJ:main Jun 4, 2026
1 check passed
@angeloINTJ

Copy link
Copy Markdown
Owner

Merged — thanks @LorenzoLongaretto! Really solid work on the test suite. The anchor frame boundary tests and NaN compression coverage are exactly what this codec needed.

First community contribution to SIMUT 🎉 Hope you stick around — there are more subsystems waiting for tests if you're interested.

@LorenzoLongaretto

Copy link
Copy Markdown
Contributor Author

Wow, thanks for the kind words! Genuinely glad the coverage landed well. I'll definitely stick around — already curious about those other subsystems. Just point me in the right direction whenever 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add unit tests for HistoryCodec roundtrip

2 participants