Commit 99efb6c0 by Awais Committed by Awais Qureshi

Adding error message on course-edit form.

ECOM-7666
parent 14bba674
......@@ -2165,6 +2165,17 @@ class CourseEditViewTests(TestCase):
response = self.client.get(self.edit_page_url)
self.assertNotContains(response, 'VIDEO LINK')
def test_update_course_with_non_errors(self):
"""
Verify that page shows error if any required field data is missing.
"""
self.user.groups.add(Group.objects.get(name=ADMIN_GROUP_NAME))
post_data = self._post_data(self.organization_extension)
post_data['title'] = ''
response = self.client.post(self.edit_page_url, data=post_data)
self.assertContains(response, 'Please fill all required fields.')
@ddt.ddt
class CourseRunEditViewTests(TestCase):
......
......@@ -398,6 +398,11 @@ class CourseEditView(mixins.PublisherPermissionMixin, UpdateView):
messages.success(self.request, _('Course updated successfully.'))
return HttpResponseRedirect(self.get_success_url())
def form_invalid(self, form):
# pylint: disable=no-member
messages.error(self.request, _('Please fill all required fields.'))
return self.render_to_response(self.get_context_data(form=form))
class CourseDetailView(mixins.LoginRequiredMixin, mixins.PublisherPermissionMixin, DetailView):
""" Course Detail View."""
......
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