nasa / PSP

The Core Flight System (cFS) Platform Support Package (PSP)
Apache License 2.0
66 stars 56 forks source link

Fix #413, Add coverage tests for pc-rtems #414

Closed thnkslprpt closed 7 months ago

thnkslprpt commented 9 months ago

Checklist

Describe the contribution

Testing performed GitHub CI actions all passing successfully and local testing results in the following added coverage for pc-rtems: Screenshot 2023-10-16 15 55 22

Expected behavior changes No code changes - only adding new tests.

System(s) tested on Debian GNU/Linux 11 (bullseye) Current main branch of cFS bundle.

Contributor Info Avi Weiss   @thnkslprpt

thnkslprpt commented 9 months ago

I wasn't able to easily set up the context enough to test CFE_PSP_WriteToCDS and CFE_PSP_ReadFromCDS fully - if someone can get it working I'll add it in.

skliper commented 7 months ago

Recommending CCB review (and hopefully merge) so we can continue moving forward.

Future work: Minor coverage improvements, add to CI and enforce (already #18), stub out shared APIs and cover those in their own test cases, individual exe per file.