Add testing via jest

Question

Add testing via jest

dkundel opened this issue 6 years ago · 5 comments

dkundel commented 6 years ago

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

Answer 1 · 2019-10-01T17:10:30.000Z

I'd like to work on this

Answer 2 · 2019-10-01T17:12:14.000Z

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

Answer 3 · 2019-10-01T17:15:00.000Z

Is there a reason node_modules are checked in?

Answer 4 · 2019-10-01T17:23:11.000Z

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

Answer 5 · 2019-10-02T15:01:08.000Z

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.