rapid7 / awsaml

Awsaml is an application for providing automatically rotated temporary AWS credentials.
MIT License
138 stars 45 forks source link

Project: Totally Testable #32

Closed fmitchell-r7 closed 7 years ago

fmitchell-r7 commented 8 years ago

Awsaml doesn't currently have any automated testing around it. This creates a barrier to entry for contributors, since it's hard to know if your changes broke anything. We should set up automated tests for Awsaml and have them run by Travis CI when pull requests are created.

Issues related to this project are filed under the "totally-testable" milestone.

tquetano-r7 commented 8 years ago

You should consider using tape as your unit testing setup. For nodejs development it doesn't require a test harness when running the tests (just regular node scripts), and if you bolster tape with proxyquire and sinon you can handle a lot of your integration testing as well.

I only call this out because picking these up are way simpler than a kitchen sink setup like Mocha or Jasmine, and its an alternative worth considering. I use them and it is quite a pleasure. :smile: