From 8ed9123ee850bc525bc60dddbf190b7ab284b2d5 Mon Sep 17 00:00:00 2001 From: Muhammad Afaq Shuaib Date: Tue, 15 Oct 2024 12:38:03 +0500 Subject: [PATCH] fix: pylint quality issues --- .../tests/test_populate_executive_education_data_csv.py | 4 ++-- .../apps/course_metadata/tests/test_models.py | 9 ++++++--- 2 files changed, 8 insertions(+), 5 deletions(-) diff --git a/course_discovery/apps/course_metadata/management/commands/tests/test_populate_executive_education_data_csv.py b/course_discovery/apps/course_metadata/management/commands/tests/test_populate_executive_education_data_csv.py index df108aa012a..6fb210df9dd 100644 --- a/course_discovery/apps/course_metadata/management/commands/tests/test_populate_executive_education_data_csv.py +++ b/course_discovery/apps/course_metadata/management/commands/tests/test_populate_executive_education_data_csv.py @@ -302,7 +302,7 @@ def test_successful_file_data_population_with_getsmarter_flag_with_future_varian reader = csv.DictReader(csv_file) data_row = next(reader) assert data_row['Variant Id'] == self.variant_1['id'] - assert data_row['Publish Date'] == self.variant_1['startDate'] + assert data_row['Publish Date'] == self.variant_1['startDate'] assert data_row['Start Time'] == '00:00:00' assert data_row['Start Date'] == self.variant_1['startDate'] assert data_row['End Time'] == '00:00:00' @@ -317,7 +317,7 @@ def test_successful_file_data_population_with_getsmarter_flag_with_future_varian data_row = next(reader) assert data_row['Variant Id'] == self.variant_2['id'] - assert data_row['Publish Date'] == str(date.today().isoformat()) + assert data_row['Publish Date'] == str(date.today().isoformat()) assert data_row['Start Time'] == '00:00:00' assert data_row['Start Date'] == self.variant_2['startDate'] assert data_row['End Time'] == '00:00:00' diff --git a/course_discovery/apps/course_metadata/tests/test_models.py b/course_discovery/apps/course_metadata/tests/test_models.py index 730b11219d6..a0465a7c2fb 100644 --- a/course_discovery/apps/course_metadata/tests/test_models.py +++ b/course_discovery/apps/course_metadata/tests/test_models.py @@ -1109,7 +1109,7 @@ def test_availability(self, start, end, expected_availability): ) @ddt.unpack def test_external_course_availability(self, product_status, expected_availability): - """ + """ Verify that the course availability is determined based on the start and end dates, not on the product status, for a given course run. """ @@ -1123,8 +1123,11 @@ def test_external_course_availability(self, product_status, expected_availabilit ) ), start=(datetime.datetime.now(pytz.UTC) - datetime.timedelta(days=5)), - end=(datetime.datetime.now(pytz.UTC) + datetime.timedelta(days=100) - if product_status.value != 'archived' else datetime.datetime.now(pytz.UTC) - datetime.timedelta(days=10)) + end=( + datetime.datetime.now(pytz.UTC) + datetime.timedelta(days=100) + if product_status.value != "archived" + else datetime.datetime.now(pytz.UTC) - datetime.timedelta(days=10) + ), ) assert course_run.availability == expected_availability