Skip to content

C
course-discovery
Commit b03a1a86 by Awais Jibran
Options

addressing comments

parent b99d4b5c
Showing with 3 additions and 3 deletions
course_discovery/apps/publisher/forms.py
...@@ -464,10 +464,11 @@ class SeatForm(BaseForm): ...@@ -464,10 +464,11 @@ class SeatForm(BaseForm):
if not course_run: if not course_run:
return return
all_course_run_seats = course_run.seats.all() all_course_run_seats = course_run.seats.all()
seats_data_is_bogus = all_course_run_seats.count() > 2 all_course_run_seats_count = all_course_run_seats.count()
seats_data_is_bogus = all_course_run_seats_count > 2
if seats_data_is_bogus: if seats_data_is_bogus:
logger.info('Removing bogus course run [%d] seats [%d]', course_run.id, all_course_run_seats.count())
all_course_run_seats.delete() all_course_run_seats.delete()
logger.info('Removed bogus course run [%d] seats [%d]', course_run.id, all_course_run_seats_count)
class CourseEntitlementForm(BaseForm): class CourseEntitlementForm(BaseForm):
......
course_discovery/apps/publisher/tests/test_views.py
...@@ -3117,7 +3117,6 @@ class CourseRunEditViewTests(SiteMixin, TestCase): ...@@ -3117,7 +3117,6 @@ class CourseRunEditViewTests(SiteMixin, TestCase):
self.assertContains(response, 'CERTIFICATE TYPE AND PRICE', status_code=200) self.assertContains(response, 'CERTIFICATE TYPE AND PRICE', status_code=200)
def login_with_course_user_role(self, course_run, role=PublisherUserRole.CourseTeam): def login_with_course_user_role(self, course_run, role=PublisherUserRole.CourseTeam):
self.client.logout()
user = course_run.course.course_user_roles.get(role=role).user user = course_run.course.course_user_roles.get(role=role).user
self.client.login(username=user.username, password=USER_PASSWORD) self.client.login(username=user.username, password=USER_PASSWORD)
return user return user
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment