Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove dependency of live URLs/endpoints for unit tests #233

Open
michaelgira23 opened this issue Aug 29, 2023 · 0 comments
Open

Remove dependency of live URLs/endpoints for unit tests #233

michaelgira23 opened this issue Aug 29, 2023 · 0 comments

Comments

@michaelgira23
Copy link
Member

michaelgira23 commented Aug 29, 2023

In our unit tests, we must fill out this test config file in order to pass the tests. While this should be important to test once in a while, all of the current maintainers do not attend MyMICDS and therefore do not have ready access to an up-to-date portal/canvas URL. We should save a cache of these (maybe deanonymize the data?) and pretend that is the live URL. That way, we can run all the tests offline.

This will have compounding effects on developer productivity when we need to fix bugs, to make sure we don't cause a regression in functionality.

@michaelgira23 michaelgira23 changed the title Remove dependency of live URLs/endpoints Remove dependency of live URLs/endpoints for unit tests Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant