Skip to content

Commit c969aab

Browse files
Repair test based on dates (test_create_featured_items) (#2229)
* Repair test based on dates * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
1 parent b006819 commit c969aab

File tree

1 file changed

+12
-26
lines changed

1 file changed

+12
-26
lines changed

cms/api_test.py

Lines changed: 12 additions & 26 deletions
Original file line numberDiff line numberDiff line change
@@ -308,38 +308,26 @@ def test_create_featured_items():
308308
# Course that starts in the future but is open for enrollment
309309
enrollable_future_course = CourseFactory.create(page=None, live=True)
310310
CoursePageFactory.create(course=enrollable_future_course, live=True)
311-
enrollable_future_courserun = CourseRunFactory.create(
311+
enrollable_future_courserun = CourseRunFactory.create( # noqa: F841
312312
course=enrollable_future_course,
313313
live=True,
314-
in_future=True,
314+
start_date=future_date,
315+
enrollment_start=further_past_date,
316+
enrollment_end=further_future_date,
317+
end_date=furthest_future_date,
315318
)
316-
enrollable_future_courserun.enrollment_start = further_past_date
317-
enrollable_future_courserun.start_date = future_date
318-
enrollable_future_courserun.enrollment_end = further_future_date
319-
enrollable_future_courserun.end_date = furthest_future_date
320-
enrollable_future_courserun.save()
321319

322320
# Course that is open for enrollment, but starts after the one above
323321
enrollable_other_future_course = CourseFactory.create(page=None, live=True)
324322
CoursePageFactory.create(course=enrollable_other_future_course, live=True)
325-
enrollable_other_future_courserun = CourseRunFactory.create(
323+
enrollable_other_future_courserun = CourseRunFactory.create( # noqa: F841
326324
course=enrollable_other_future_course,
327325
live=True,
328-
in_future=True,
326+
enrollment_start=further_past_date,
327+
start_date=future_date + timedelta(days=2),
328+
enrollment_end=further_future_date + timedelta(days=2),
329+
end_date=furthest_future_date + timedelta(days=2),
329330
)
330-
enrollable_other_future_courserun.enrollment_start = (
331-
enrollable_future_courserun.enrollment_start
332-
)
333-
enrollable_other_future_courserun.start_date = (
334-
enrollable_future_courserun.start_date + timedelta(days=2)
335-
)
336-
enrollable_other_future_courserun.enrollment_end = (
337-
enrollable_future_courserun.enrollment_end + timedelta(days=2)
338-
)
339-
enrollable_other_future_courserun.end_date = (
340-
enrollable_future_courserun.end_date + timedelta(days=2)
341-
)
342-
enrollable_other_future_courserun.save()
343331

344332
# A self-paced course that is open for enrollment
345333
enrollable_self_paced_course = CourseFactory.create(page=None, live=True)
@@ -348,6 +336,7 @@ def test_create_featured_items():
348336
course=enrollable_self_paced_course,
349337
live=True,
350338
in_progress=True,
339+
enrollment_end=further_future_date + timedelta(days=2),
351340
)
352341
self_paced_run.is_self_paced = True
353342
self_paced_run.save()
@@ -358,11 +347,8 @@ def test_create_featured_items():
358347
course=in_progress_course,
359348
live=True,
360349
in_progress=True,
350+
enrollment_end=further_future_date + timedelta(days=2),
361351
)
362-
in_progress_course.enrollment_start = further_past_date
363-
in_progress_course.start_date = past_date
364-
in_progress_course.enrollment_end = future_date
365-
in_progress_course.end_date = further_future_date
366352

367353
unenrollable_course = CourseFactory.create(page=None, live=False)
368354
CoursePageFactory.create(course=unenrollable_course, live=False)

0 commit comments

Comments
 (0)