Alan-Jowett / sonde / 23397830315
82%

DEFAULT BRANCH: main
Ran
Jobs 1
Files 72
Run time 1min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 82.699% (+0.005%) from 82.694%
23397830315

push

github

web-flow 
test(protocol): add 7 missing error path and malformed input tests (T-P019a..T-P039) (#403)

* test(protocol): add 7 missing error path and malformed input tests (T-P019a..T-P039)

Add tests closing 7 validation gaps identified in issue #346:

- T-P019a: DecodeError::TooLong - decode_frame() rejects >250 bytes
- T-P019b: DecodeError::CborError - invalid CBOR bytes in payload
- T-P019c: DecodeError::InvalidFieldType - type-mismatched CBOR field
- T-P036: missing-field detection for Command, GetChunk, Chunk,
  ProgramAck, AppData, AppDataReply
- T-P037: unknown CBOR keys silently ignored in all non-Wake messages
- T-P038: COMMAND nested payload CBOR byte inspection (UpdateProgram
  has nested map at key 5; Nop/Reboot omit key 5)
- T-P039: large u64 values round-trip without truncation; CBOR
  encoding uses correct 4-byte and 8-byte integer prefixes

Closes #346

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

* fix(protocol): address review feedback on test maintainability

- Remove hard-coded test range from module doc header so it stays
  accurate as new tests are added.
- Use `MAX_FRAME_SIZE + 1` instead of magic `251` in `test_p019a`
  so the test stays correct if the frame budget changes.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

---------

Co-authored-by: Alan Jowett <alan.jowett@microsoft.com>
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

16974 of 20525 relevant lines covered (82.7%)

49.14 hits per line

Jobs
ID Job ID Ran Files Coverage
1 23397830315.1 72
82.7
 GitHub Action Run
Source Files on build 23397830315