From 013488275bfc1c36947dc1886ea54b754ab048fb Mon Sep 17 00:00:00 2001 From: Adam Stankiewicz Date: Thu, 19 Sep 2024 16:08:55 -0400 Subject: [PATCH] chore: cleanup --- .../apps/content_assignments/api.py | 2 -- .../automatically_nudge_assignments.py | 13 --------- .../content_assignments/tests/test_api.py | 29 ------------------- 3 files changed, 44 deletions(-) diff --git a/enterprise_access/apps/content_assignments/api.py b/enterprise_access/apps/content_assignments/api.py index e2b97ebf..02e9aca6 100644 --- a/enterprise_access/apps/content_assignments/api.py +++ b/enterprise_access/apps/content_assignments/api.py @@ -602,8 +602,6 @@ def _get_parent_content_key(assignment_configuration, content_key): course_content_metadata = _get_content_summary(assignment_configuration, content_key) metadata_content_key = course_content_metadata.get('content_key') - print('_get_parent_content_key?!: ', metadata_content_key, content_key) - # Check if the assignment's content_key matches the returned content_key. If so, this is a course key # which has no parent key. if content_key == metadata_content_key: diff --git a/enterprise_access/apps/content_assignments/management/commands/automatically_nudge_assignments.py b/enterprise_access/apps/content_assignments/management/commands/automatically_nudge_assignments.py index 8c247315..cd9a6e7a 100644 --- a/enterprise_access/apps/content_assignments/management/commands/automatically_nudge_assignments.py +++ b/enterprise_access/apps/content_assignments/management/commands/automatically_nudge_assignments.py @@ -172,19 +172,6 @@ def handle(self, *args, **options): else False ) - print( - 'NUDGE?!?!?!:', - { - 'assignment_uuid': str(assignment.uuid), - 'content_key': assignment.content_key, - 'parent_content_key': assignment.parent_content_key, - 'start_date': start_date, - 'datetime_start_date': datetime_start_date, - 'num_days': days_before_course_start_date, - 'result': can_send_nudge_notification_in_advance, - }, - ) - if not can_send_nudge_notification_in_advance: logger.info( ( diff --git a/enterprise_access/apps/content_assignments/tests/test_api.py b/enterprise_access/apps/content_assignments/tests/test_api.py index a93709c7..56f3ce5c 100644 --- a/enterprise_access/apps/content_assignments/tests/test_api.py +++ b/enterprise_access/apps/content_assignments/tests/test_api.py @@ -504,18 +504,6 @@ def test_allocate_assignments_happy_path( 'state': LearnerContentAssignmentStateChoices.ERRORED, }) - for assignment in LearnerContentAssignment.objects.all(): - print( - 'original assignment:', - str(assignment.uuid), - assignment.learner_email, - assignment.state, - assignment.content_key, - assignment.parent_content_key, - assignment.is_assigned_course_run, - assignment.preferred_course_run_key, - ) - allocation_results = allocate_assignments( self.assignment_configuration, learners_to_assign, @@ -544,19 +532,6 @@ def test_allocate_assignments_happy_path( for record in assignments_to_refresh: record.refresh_from_db() - for assignment in LearnerContentAssignment.objects.all(): - print( - 'assignment after allocation:', - str(assignment.uuid), - assignment.learner_email, - assignment.state, - assignment.content_key, - assignment.parent_content_key, - assignment.is_assigned_course_run, - assignment.preferred_course_run_key, - ) - - # Create list of assignments expected to be updated, including: # - The allocated assignments with outdated: preferred course run, parent_content_key, is_assigned_course_run. # - Errored assignments @@ -577,16 +552,12 @@ def test_allocate_assignments_happy_path( allocated_assignment_old_run_based, ) - print('actual updated uuids:', {str(record.uuid) for record in allocation_results['updated']}) - if is_assigned_course_run: expected_updated_assignments += ( allocated_assignment_old_parent_content_key, allocated_assignment_old_run_based, ) - print('expected updated uuids:', {str(record.uuid) for record in expected_updated_assignments}) - self.assertEqual( {record.uuid for record in allocation_results['updated']}, {assignment.uuid for assignment in expected_updated_assignments},