Add validation for number of arguments provided.

CalgaryToSpace / CTS-SAT-1-OBC-Firmware

Firmware for the CTS-SAT-1 ("FrontierSat") mission. Runs on the STM32-based Onboard Computer.

MIT License

8 stars 0 forks source link

Add validation for number of arguments provided. #137

Closed Makiv1 closed 3 months ago

Makiv1 commented 3 months ago

parse_full_telecommand() now validates that the correct number of arguments are provided. Returns 100 if the wrong number of arguments are provided. Corresponding unit tests have been added.

DeflateAwning commented 3 months ago

Let me know when this is ready for another review!

Makiv1 commented 3 months ago

Should be ready for another review :)

DeflateAwning commented 3 months ago

If you're getting tired of this task, feel free to let me know and I'd be happy to make the remaining changes, and/or you can get started on another issue!

DeflateAwning commented 3 months ago

Also, if you make that change, feel free to add yourself as a contributor in the README! Otherwise, do it in the future PR.