toolshed icon indicating copy to clipboard operation
toolshed copied to clipboard

Add unit tests

Open fhunleth opened this issue 3 years ago • 1 comments

Very little of toolshed is unit tested and this is not ideal for long term maintenance. Since many of the tools just wrap functions that can be presumed tested, I don't think that it's necessary to test the functionality deeply. Perhaps only test that the helper exists and passing an argument basically works.

It would be good to write tests for one or two helpers first to how this looks and how valuable it is.

fhunleth avatar Mar 01 '21 13:03 fhunleth

I think it is worth it to minimally test at least one happy path for each function. Also that effort might reveal some room for improvement. I'll write some minimal tests.

mnishiguchi avatar Jun 28 '22 12:06 mnishiguchi

Closing since basic unit tests for many functions are in place now and we can improve in future PRs.

fhunleth avatar Jan 14 '23 17:01 fhunleth