search

dvx / lofi

🎵🔉 A mini Spotify player with WebGL visualizations.
http://lofi.rocks
MIT License
1.2k stars 68 forks source link

Add Tests #79

Closed stamoun closed 1 year ago

stamoun commented 4 years ago

Add testing ability for the various React components (unit) or for the whole electron app (integration/system).

For system/integration, Spectron should be used (https://www.electronjs.org/spectron).

No idea if you have a preference for a given framework (mocha, jest, enzyme?), I think the app would benefit from being testable. (this can go alongside with the implementation of CI/CD).

Definitely an enhancement :)

dvx commented 4 years ago

I have experience with both Mocha and Jest, so it's a toss-up from my end. I totally agree that testing should be a roadmap priority!