diff --git a/course_discovery/apps/api/tests/test_mixins.py b/course_discovery/apps/api/tests/test_mixins.py index 2ebd3466b1..ec65daf897 100644 --- a/course_discovery/apps/api/tests/test_mixins.py +++ b/course_discovery/apps/api/tests/test_mixins.py @@ -18,7 +18,7 @@ class TestAPIView(AnonymousUserThrottleAuthenticatedEndpointMixin, APIView): permission_classes = () authentication_classes = () - def get(self, request, *_args, **_kwargs): # pylint: disable=unused-argument + def get(self, request, *_args, **_kwargs): return Response( status=status.HTTP_200_OK, data="Hello, World" diff --git a/course_discovery/apps/course_metadata/data_loaders/tests/test_csv_loader.py b/course_discovery/apps/course_metadata/data_loaders/tests/test_csv_loader.py index bbb361f21b..b64340db48 100644 --- a/course_discovery/apps/course_metadata/data_loaders/tests/test_csv_loader.py +++ b/course_discovery/apps/course_metadata/data_loaders/tests/test_csv_loader.py @@ -462,7 +462,10 @@ def test_ingest_flow_for_preexisting_published_course_with_new_run_creation(self CourseRunFactory( course=course, start=datetime.datetime(2014, 3, 1, tzinfo=UTC), - end=datetime.datetime(2024, 3, 1, tzinfo=UTC), + # 2050 end date is to ensure the course run is present among active runs and thus + # non-draft entries are created. If the discovery is there till 2050, you would need to update the + # tests after Jan 1, 2050. + end=datetime.datetime(2050, 1, 1, tzinfo=UTC), key=self.COURSE_RUN_KEY, type=self.course_run_type, status='published', diff --git a/course_discovery/apps/course_metadata/management/commands/populate_course_length.py b/course_discovery/apps/course_metadata/management/commands/populate_course_length.py index 3385538e66..5215e0830b 100644 --- a/course_discovery/apps/course_metadata/management/commands/populate_course_length.py +++ b/course_discovery/apps/course_metadata/management/commands/populate_course_length.py @@ -45,8 +45,7 @@ def get_query_results_from_snowflake(self): try: cs.execute(SNOWFLAKE_POPULATE_COURSE_LENGTH_QUERY) rows = cs.fetchall() - for row in rows: - yield row + yield from rows finally: cs.close() ctx.close() diff --git a/course_discovery/apps/course_metadata/management/commands/refresh_course_reviews.py b/course_discovery/apps/course_metadata/management/commands/refresh_course_reviews.py index 7cb10ef2f5..79011dc9d3 100644 --- a/course_discovery/apps/course_metadata/management/commands/refresh_course_reviews.py +++ b/course_discovery/apps/course_metadata/management/commands/refresh_course_reviews.py @@ -46,8 +46,7 @@ def get_query_results_from_snowflake(self): try: cs.execute(SNOWFLAKE_REFRESH_COURSE_REVIEWS_QUERY) rows = cs.fetchall() - for row in rows: - yield row + yield from rows finally: cs.close() ctx.close() diff --git a/course_discovery/apps/course_metadata/tests/test_admin.py b/course_discovery/apps/course_metadata/tests/test_admin.py index 52985c1899..625f523d5e 100644 --- a/course_discovery/apps/course_metadata/tests/test_admin.py +++ b/course_discovery/apps/course_metadata/tests/test_admin.py @@ -29,7 +29,6 @@ from course_discovery.apps.course_metadata.tests import factories -# pylint: disable=no-member @ddt.ddt class AdminTests(SiteMixin, TestCase): """ Tests Admin page."""