factorio-updater icon indicating copy to clipboard operation
factorio-updater copied to clipboard

Unit tests, pls

Open narc0tiq opened this issue 7 years ago • 4 comments

It's really not nice to have a complete lack of tests, even if it is unclear what can be tested. At the very least, an issue like #11 could be automatically checked for possible regression*.


* - I hesitate to say it wouldn't have happened, since I probably wouldn't have expected a split upgrade path until one actually presented itself, but now we know it can exist and we should make sure we continue supporting it.

narc0tiq avatar Dec 18 '17 15:12 narc0tiq

i posted this over in bisa's repo but gonna put it here too and again still not sure how to use it but this might help https://github.com/sstephenson/bats

tmzasz avatar May 24 '18 00:05 tmzasz

This isn't a bash script. Regardless, the how of testing isn't relevant to this issue.

narc0tiq avatar May 24 '18 06:05 narc0tiq

I stand corrected It was a long day when i posted that my fault :)

tmzasz avatar May 25 '18 18:05 tmzasz

No worries, I found out about bats :)

narc0tiq avatar May 25 '18 21:05 narc0tiq