Allow variant option to test script, improve option handling

hildogjr / KiCost

Build cost spreadsheet for a KiCad project.

MIT License

494 stars 97 forks source link

Allow variant option to test script, improve option handling #469

Closed mdeweerd closed 3 years ago

mdeweerd commented 3 years ago

Extended test_single.sh script with possibility to pass on a --variant option and fixed some "hidden" bugs.

set-soft commented 3 years ago

Hi @mdeweerd ! Having this support is good. But I'll suggest you to take a look at tests/tests_kicost.py you can control all the options from there and add permanent tests that will run on every github commit.

mdeweerd commented 3 years ago

Historically I made a log of changes to this test script to automate testing. I saw 'test_kicost.py' but I could not get it running ("easily") in my environment.

Just added support for variant names in the output - apparently our naming is on par for this. So adding another pull request for that.

set-soft commented 3 years ago

test_kicost.py runs using pytest. At the root of the project you just run the pytest or pytest-3 command. That's a very common testing mechanism.