Created tests for wrangler worker

Seneca-CDOT / my-photohub

Making it easy to share your photos using GitHub Pages

BSD 2-Clause "Simplified" License

14 stars 13 forks source link

Created tests for wrangler worker #18

Closed devils2ndself closed 1 year ago

devils2ndself commented 1 year ago

updated package.json with new npm scripts and dependencies
- refactored raw response and login form
- fixed and rewrote worker.fetch unit test

Resolves #13

devils2ndself commented 1 year ago

I got a bit lost in POST request code, and I genuinely don't think we could write a unit test for such a complex action. If there are any other useful tests I could do, please let me know

@SerpentBytes could you help me with your thoughts and suggestions?

devils2ndself commented 1 year ago

Changed the description, and added a couple of tests for checking the headers

SerpentBytes commented 1 year ago

@devils2ndself Could you please update the CONTRIBUTING.md with information on how to test the code?

devils2ndself commented 1 year ago

Done! If there's nothing else regarding the test, would it be possible to merge it by end of tomorrow? I wanted to use this one as my pull request for Release 0.3