meteor-transactions icon indicating copy to clipboard operation
meteor-transactions copied to clipboard

Add Velocity-powered automated tests

Open rjsmith opened this issue 10 years ago • 1 comments

The correct operation of this package is essential for applications that use it, because it deals with mutating application state. To improve and validate the quality of the implementation, there should be a suite of automated tests.

If all the functionality is one package, we can include a testapp folder that contains a Meteor application that includes server and client - side unit and integration tests.

rjsmith avatar Mar 26 '15 08:03 rjsmith

There are now enough automated tests that I can sleep better at night after a new release, but not so many that I'm confident I've caught a whole lot of edge cases.

More unit tests are definitely required (I think there's only four!) and although there are almost 30 server integration tests at the time of writing, I'd say a package like this would need about four or five times that many to be considered really solid.

I will continue to add to the test suite, little by little, as time goes by and issues are raised. If anyone wants to write some tests, I'd be all over accepting PRs!

JackAdams avatar Nov 26 '15 11:11 JackAdams