Skip to content
This repository

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse code

Fixed a small oversight in auth tests

Thanks to Vinicius Ruan Cainelli for the catch
closes #392
  • Loading branch information...
commit 01362745ba72286309ff1955219a5ffc32c760b0 1 parent c9c9a56
Preston Holmes authored September 27, 2012

Showing 1 changed file with 2 additions and 2 deletions. Show diff stats Hide diff stats

  1. 4  django/contrib/auth/tests/basic.py
4  django/contrib/auth/tests/basic.py
@@ -38,8 +38,8 @@ def test_user(self):
38 38
         self.assertFalse(u.is_superuser)
39 39
 
40 40
         # Check API-based user creation with no password
41  
-        User.objects.create_user('testuser2', 'test2@example.com')
42  
-        self.assertFalse(u.has_usable_password())
  41
+        u2 = User.objects.create_user('testuser2', 'test2@example.com')
  42
+        self.assertFalse(u2.has_usable_password())
43 43
 
44 44
     def test_user_no_email(self):
45 45
         "Check that users can be created without an email"

4 notes on commit 0136274

Jannis Leidel
Owner

@ptone I'm pretty sure that doesn't fix #392. Please make sure to use Trac ticket IDs to reduce the number of wrong comments there (see https://code.djangoproject.com/ticket/392)

Russell Keith-Magee

Yeah... I thought closing out a 7 year old ticket was too good to be true :-)

Jannis Leidel
Owner

Teehee!

Marc Tamlyn
Owner

This is a bit of an issue in the long run though - not that I have any idea what the solution is.

See this commit for instance: 24154b2 - doesn't have anything to do with #225

Please sign in to comment.
Something went wrong with that request. Please try again.