search

nasa / bplib

Apache License 2.0
27 stars 13 forks source link

Fix #179, implement test cases for "common" submodule #180

Closed jphickey closed 1 year ago

jphickey commented 1 year ago

Adds CRC and RB tree test cases to get better code coverage. Move the "rbtest" functional test from apps to the common submodule as well, as this focuses on the RB tree functionality.

This achieves 100% line and function coverage, and 99.4% on branches (all but 1).

That final "unchecked" branch/conditional could be removed but IMO the code is more future proof with it in there, so leaving it for now. Could be a topic of discussion.

Fixes #179

jphickey commented 1 year ago

Discussed at 2022-10-26 CCB, approved