abcorrea / powerlifted

Powerlifted Planner
GNU General Public License v3.0
28 stars 12 forks source link

Improve automated dev tests #12

Closed abcorrea closed 4 years ago

abcorrea commented 4 years ago

We would like to improve the tests in dev/. Right now, the tests do not work after the integration with different exit codes. Ideally, we would like to add the following:

We probably also want to split into different tests (i.e., check translator, search, and all planner components).

gfrances commented 4 years ago

For the Python translator tests I'd definitely go with pytest. For C++ test I would also consider some testing framework, but if you have already something that works there, let's go with that!

abcorrea commented 4 years ago

For C++ I only know unit testing tools (like Google testing), but the tests we are running now are basically just checking output and plans and verifying if they make sense. Do you think this is fine enough for this issue or should we extend it?

gfrances commented 4 years ago

no, that sounds great :-)