浏览代码

Deleting a user should also delete its user profile

Jon Natkins 13 年之前
父节点
当前提交
745cc37e89
共有 2 个文件被更改,包括 6 次插入0 次删除
  1. 4 0
      apps/useradmin/src/useradmin/tests.py
  2. 2 0
      apps/useradmin/src/useradmin/views.py

+ 4 - 0
apps/useradmin/src/useradmin/tests.py

@@ -305,8 +305,12 @@ def test_user_admin():
               "Inactivated user gets redirected to login page")
 
   # Delete that regular user
+  funny_profile = UserProfile.objects.get(user=test_user)
   response = c_su.post('/useradmin/users/delete/%s' % (FUNNY_NAME_QUOTED,))
   assert_true("Hue Users" in response.content)
+  assert_false(User.objects.filter(username=FUNNY_NAME).exists())
+  assert_false(UserProfile.objects.filter(id=funny_profile.id).exists())
+
   # You shouldn't be able to create a user without a password
   response = c_su.post('/useradmin/users/new', dict(username="test"))
   assert_true("You must specify a password when creating a new user." in response.content)

+ 2 - 0
apps/useradmin/src/useradmin/views.py

@@ -59,6 +59,8 @@ def delete_user(request, username):
       try:
         user = User.objects.get(username=username)
         _check_remove_last_super(user)
+        user_profile = UserProfile.objects.get(user=user)
+        user_profile.delete()
         user.delete()
       finally:
         __users_lock.release()