Bisa / factorio-init

Factorio init script
MIT License
415 stars 82 forks source link

Automated Tests #123

Closed Bisa closed 3 years ago

Bisa commented 6 years ago

Since the script has become such a large set of logic I would prefer if it could be tested to ensure quality.

tmzasz commented 6 years ago

I would not be 100% sure on how to use it but my research points to this https://github.com/sstephenson/bats as being a posible testing suite for automated tests if it helps :)

jhawkwind commented 5 years ago

Also, if there was a defined subset of *nix "flavors" to target these tests, it would help narrow the field of work, both from a testing perspective and a development perspective.

I would suggest targeting:

I don't know if there are any other flavors that are more common than those two among the Factorio community? Also, might want to look at https://travis-ci.com/ integration to help with the testing hooks with GitHub.

Bisa commented 4 years ago

note to @Bisa - remember to adjust the default tests to run on "latest" factorio, not some hard coded default.

Bisa commented 3 years ago

With the release of 3.0.0 I'll retire this ticket, while tests are not automatic they are at least introduced into the repository and can be built upon going forward.