From 1c84f8e2dd59923c454b7965ded849eb32e5502a Mon Sep 17 00:00:00 2001 From: David Ormsbee Date: Tue, 21 Aug 2012 13:00:06 -0400 Subject: [PATCH] Remove student removal (we shouldn't ever delete users) --- lms/djangoapps/django_comment_client/tests.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lms/djangoapps/django_comment_client/tests.py b/lms/djangoapps/django_comment_client/tests.py index 822dd6f082..0e8f786692 100644 --- a/lms/djangoapps/django_comment_client/tests.py +++ b/lms/djangoapps/django_comment_client/tests.py @@ -35,8 +35,10 @@ class PermissionsTestCase(TestCase): def tearDown(self): self.student_enrollment.delete() self.moderator_enrollment.delete() - self.student.delete() - self.moderator.delete() + +# Do we need to have this? We shouldn't be deleting students, ever +# self.student.delete() +# self.moderator.delete() def testDefaultRoles(self): self.assertTrue(self.student_role in self.student.roles.all())