search

therungg / therun-frontend

Frontend for therun.gg
https://therun-fr-therun.vercel.app
MIT License
26 stars 8 forks source link

Implement a test suite for E2E testing and write some initial tests #19

Closed developerrowan closed 5 months ago

developerrowan commented 1 year ago

Inevitably, TheRun will continue to grow. Refactors are sure to come within the next half-year, but they won't be the last.

It would then be prudent to implement, and begin writing tests with, a testing framework. This way, we can ensure frontend functionality does not become corrupted throughout (inevitable) refactors.

Vercel provides wonderful documentation on doing it. https://nextjs.org/docs/pages/building-your-application/optimizing/testing

But we don't need to just set up the suite - setting precedent by writing some good, initial tests is of paramount importance.

therungg commented 1 year ago

Agreed. I used Cypress a lot, but never really bothered to implement it for this frontend. I have extensive testing in the backend, and it would be good to have at least some for the crucial processes in the frontend as well.

developerrowan commented 5 months ago

e2e testing may be tricker than expected, for now closing this. It can be re-opened when we're more comfortable approaching this topic.