Skip to content

Commit

Permalink
fix: pylint quality issues
Browse files Browse the repository at this point in the history
  • Loading branch information
AfaqShuaib09 committed Oct 15, 2024
1 parent e54ab60 commit 8ed9123
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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'
Expand All @@ -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'
Expand Down
9 changes: 6 additions & 3 deletions course_discovery/apps/course_metadata/tests/test_models.py
Original file line number Diff line number Diff line change
Expand Up @@ -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.
"""
Expand All @@ -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

Expand Down

0 comments on commit 8ed9123

Please sign in to comment.