This repository has been archived by the owner on Mar 12, 2023. It is now read-only.
Task: add integration/server tests #80
Labels
needs triage
The issue or pull request could use eyes on it
Comments
zysim
pushed a commit
to zysim/leaderboard-backend-go
that referenced
this issue
Jun 28, 2022
* Refactored Requests and Tests * Moved Requests into a single flat folder instead of a folder per resource * Adjusted the Test libs for sending requests to allow for a nicer lib API * Moved TestApi code into its own folder The stuff for API helpers living with the other helpers didn't make a ton of sense so moving it out into its own place. * Minor typo fixes Left one test assertion as Unauthorized by accident, and forgot a using directive * Implement Suggestions Co-authored-by: Sim <[email protected]>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
We would like to merge #78 due to circumstances around the new POC. We will need to add integration tests, which could be in the
serverpackage or could be some sort of external package.Affected Functionality
Integration tests would be added. We'd probably need to introduce a flag that will make the integration tests run, rather than needing to run them any time a dev runs
go test.The text was updated successfully, but these errors were encountered: