All new code requires tests to ensure against regressions.
However, if your PR contains zero code changes, feel free to select the checkmark below to indicate so.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
Description of the Change
This PR is finally my refactor of tests.
Nearly all static data returns have been removed, where now it's up to the individual tests to mock as needed, to allow our tests to much more closely represent the production environment. Ideally allowing us to catch errors earlier, and prevent more bugs.
Additionally, due to these changes dev mode has effectively been removed, to where now, other than support for disabling Google API calls, all other API calls will occur no matter if you are running the server in dev mode. This means largely there is no easy safe way to run the server in dev mode. It is instead recommended to rely more heavily on tests to ensure functionality as needed.
Requirements
Filling out the template is required.
All new code requires tests to ensure against regressions.
[X] Have you ran tests against this code?
[ ] This PR contains zero code changes.
Description of the Change
This PR is finally my refactor of tests.
Nearly all static data returns have been removed, where now it's up to the individual tests to mock as needed, to allow our tests to much more closely represent the production environment. Ideally allowing us to catch errors earlier, and prevent more bugs.
Additionally, due to these changes dev mode has effectively been removed, to where now, other than support for disabling Google API calls, all other API calls will occur no matter if you are running the server in dev mode. This means largely there is no easy safe way to run the server in dev mode. It is instead recommended to rely more heavily on tests to ensure functionality as needed.