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.

Sign up for GitHub

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

Jump to bottom

Add more unit tests for Emeritus API Ingestion #3028

Closed

asadali145 opened this issue Jun 26, 2024 · 0 comments

Assignees

Labels

Contributor

asadali145 commented Jun 26, 2024 •

edited

Loading

Description/Context

We worked on https://github.com/mitodl/hq/issues/4032 and added minimal unit tests. We should add some more tests to make it robust.

Here are a few tests that we will add:

Add tests for the task courses.tasks.task_sync_emeritus_course_runs
Add tests for mitxpro.utils.clean_url and mitxpro.utils.strip_datetime
Add tests for the stats in courses.sync_external_courses.emeritus_api.update_emeritus_course_runs
Add test for courses.sync_external_courses.emeritus_api.create_or_update_emeritus_course_run when existing course run start and end dates are Null.
Add tests for the return values of:

The text was updated successfully, but these errors were encountered:

asadali145 added the product:xpro label

arslanashraf7 mentioned this issue

fix(external course sync): sync course run dates if they are missing #3027

Merged

asadali145 self-assigned this

asadali145 mentioned this issue

test(emeritus_api): add more tests for emeritus API ingestion #3032

Merged

asadali145 closed this as completed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment