Code cleanup - Remove pdb traces and remarked out code from test code

Author: johnbaldwin

tests/models/test_course_daily_metrics_model.py

@@ -130,13 +130,11 @@ def test_with_valid_average_progress(self, average_progress):
             course_id='course-v1:SomeOrg+ABC01+2121',
             enrollment_count=11,
             active_learners_today=1,
-            # average_progress=Decimal(str(average_progress)),
             average_progress=str(average_progress),
             average_days_to_complete=5,
             num_learners_completed=10
         )
         metrics = CourseDailyMetrics.objects.create(**rec)
-        # assert metrics.average_progress == Decimal(str(average_progress))
         assert metrics.average_progress == average_progress
         metrics.clean_fields()
 
@@ -154,7 +152,6 @@ def test_with_valid_average_progress_2(self, average_progress):
             defaults=dict(
                 enrollment_count=11,
                 active_learners_today=1,
-                # average_progress=Decimal(str(average_progress)),
                 average_progress=str(average_progress),
                 average_days_to_complete=5,
                 num_learners_completed=10,
@@ -163,10 +160,6 @@ def test_with_valid_average_progress_2(self, average_progress):
         cdm, created = CourseDailyMetrics.objects.update_or_create(**rec)
         cdm.save()
 
-        # import pdb; pdb.set_trace()
-
-
-        # assert metrics.average_progress == Decimal(str(average_progress))
         assert cdm.average_progress == str(average_progress)
         cdm.clean_fields()
 

tests/views/test_learner_details_view.py

@@ -301,7 +301,6 @@ def test_serializer(self):
         queryset = enrollments.filter(user=self.my_site_users[0])
         assert queryset
         serializer = LearnerCourseDetailsSerializer(queryset[0])
-        # import pdb; pdb.set_trace()
         # We're asserting that we can get the serializer `data` property without
         # error, not checking the contents of the data. That should be done in
         # the serializer specific tests (see tests/test_serializers.py).

tests/views/test_site_monthly_metrics_viewset.py

@@ -105,7 +105,10 @@ def check_response(self, response, endpoint):
         return True
 
     def run_request(self, endpoint, user_reg_test_data):
-        import pdb; pdb.set_trace()
+        """
+        Stub
+        """
+        pass
 
     @pytest.mark.skip(reason='We need to confirm who is in total registered users')
     def test_registered_learners(self, monkeypatch, user_reg_test_data):