search

twilio-labs / actions-sms

Send an SMS through GitHub Actions
MIT License
148 stars 40 forks source link

Add testing via jest #5

Open dkundel opened 5 years ago

dkundel commented 5 years ago

We should add some basic testing using jest. This might require some refactoring to make the code a bit more testable.

dangilkerson commented 5 years ago

I'd like to work on this

dkundel commented 5 years ago

That would be great :) let me know if you need anything from me!

dangilkerson commented 5 years ago

Is there a reason node_modules are checked in?

dkundel commented 5 years ago

Yes those are the production dependencies only. It's a GitHub actions thing. i described it in #6

dangilkerson commented 4 years ago

I exported a function in main to test since I wanted to minimize the changes in case there were other changes on going. Probably should pull the code out of main.js when things are stable.