search

SMH110 / widgets

1 stars 0 forks source link

Add automated tests #19

Closed massimocode closed 8 years ago

massimocode commented 8 years ago

I have been wanting to request some new features but I'm a little worried that we have no automated tests. Quite a few issues have been raised against the initial implementation and have since been fixed, which goes to show that the logic isn't very simple. If we want to change the logic to add new features then we will have a massive overhead of manually retesting all the different behaviours of the calculator.

Based on this, I think we should add some automated tests for the calculator's existing behaviour.

massimocode commented 8 years ago

I'll put in a PR for some basic changes that will enable testing

massimocode commented 8 years ago

Ok I've put in the basic changes to enable testing. Feel free to add tests for various scenarios.

massimocode commented 8 years ago

Completed in 2b9193a9d83bcd1f50ab0f98506aed932be32ec4