From 17bef565f1dabe94727d01a895ca5000fc218cf1 Mon Sep 17 00:00:00 2001 From: David Baumgold Date: Wed, 3 Jul 2013 15:46:10 -0400 Subject: [PATCH] Added another test --- cms/djangoapps/contentstore/tests/test_users.py | 15 +++++++++++++++ cms/djangoapps/contentstore/views/user.py | 2 +- 2 files changed, 16 insertions(+), 1 deletion(-) create mode 100644 cms/djangoapps/contentstore/tests/test_users.py diff --git a/cms/djangoapps/contentstore/tests/test_users.py b/cms/djangoapps/contentstore/tests/test_users.py new file mode 100644 index 0000000000..f945ef50fc --- /dev/null +++ b/cms/djangoapps/contentstore/tests/test_users.py @@ -0,0 +1,15 @@ +import json +from .utils import CourseTestCase +from django.core.urlresolvers import reverse + + +class UsersTestCase(CourseTestCase): + def setUp(self): + super(UsersTestCase, self).setUp() + self.url = reverse("add_user", kwargs={"location": ""}) + + def test_empty(self): + resp = self.client.post(self.url) + self.assertEqual(resp.status_code, 400) + content = json.loads(resp.content) + self.assertEqual(content["Status"], "Failed") diff --git a/cms/djangoapps/contentstore/views/user.py b/cms/djangoapps/contentstore/views/user.py index 6904b9e95f..dae0d246a5 100644 --- a/cms/djangoapps/contentstore/views/user.py +++ b/cms/djangoapps/contentstore/views/user.py @@ -73,7 +73,7 @@ def add_user(request, location): This POST-back view will add a user - specified by email - to the list of editors for the specified course ''' - email = request.POST["email"] + email = request.POST.get("email") if not email: msg = {