Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with HTTPS or Subversion.

Download ZIP
Browse files

Fixed flake8 warnings introduced in recent commits.

  • Loading branch information...
commit 79f05616fbf48cf7c205ef17666af0c3d47b3c1e 1 parent 0e45669
@charettes charettes authored
View
2  django/db/backends/postgresql_psycopg2/base.py
@@ -4,8 +4,6 @@
Requires psycopg 2: http://initd.org/projects/psycopg2
"""
-import sys
-
from django.conf import settings
from django.db.backends import (BaseDatabaseFeatures, BaseDatabaseWrapper,
BaseDatabaseValidation)
View
17 tests/get_or_create/tests.py
@@ -19,40 +19,38 @@ def setUp(self):
)
def test_get_or_create_method_with_get(self):
- p, created = Person.objects.get_or_create(
+ created = Person.objects.get_or_create(
first_name="John", last_name="Lennon", defaults={
"birthday": date(1940, 10, 9)
}
- )
+ )[1]
self.assertFalse(created)
self.assertEqual(Person.objects.count(), 1)
-
def test_get_or_create_method_with_create(self):
- p, created = Person.objects.get_or_create(
+ created = Person.objects.get_or_create(
first_name='George', last_name='Harrison', defaults={
'birthday': date(1943, 2, 25)
}
- )
+ )[1]
self.assertTrue(created)
self.assertEqual(Person.objects.count(), 2)
-
def test_get_or_create_redundant_instance(self):
"""
If we execute the exact same statement twice, the second time,
it won't create a Person.
"""
- george, created = Person.objects.get_or_create(
+ Person.objects.get_or_create(
first_name='George', last_name='Harrison', defaults={
'birthday': date(1943, 2, 25)
}
)
- evil_george, created = Person.objects.get_or_create(
+ created = Person.objects.get_or_create(
first_name='George', last_name='Harrison', defaults={
'birthday': date(1943, 2, 25)
}
- )
+ )[1]
self.assertFalse(created)
self.assertEqual(Person.objects.count(), 2)
@@ -67,6 +65,7 @@ def test_get_or_create_invalid_params(self):
Person.objects.get_or_create, first_name="Tom", last_name="Smith"
)
+
class GetOrCreateTestsWithManualPKs(TestCase):
def setUp(self):
View
2  tests/prefetch_related/tests.py
@@ -1066,7 +1066,7 @@ def setUp(self):
for _ in range(3):
house = House.objects.create()
for _ in range(3):
- self.rooms.append(Room.objects.create(house = house))
+ self.rooms.append(Room.objects.create(house=house))
def test_bug(self):
prefetcher = get_prefetcher(self.rooms[0], 'house')[0]
View
3  tests/test_discovery_sample/doctests.py
@@ -3,6 +3,7 @@
https://docs.python.org/3/library/doctest.html
"""
+
def factorial(n):
"""Return the factorial of n, an exact integer >= 0.
@@ -35,7 +36,7 @@ def factorial(n):
raise ValueError("n must be >= 0")
if math.floor(n) != n:
raise ValueError("n must be exact integer")
- if n+1 == n: # catch a value like 1e300
+ if n + 1 == n: # catch a value like 1e300
raise OverflowError("n too large")
result = 1
factor = 2
View
1  tests/test_runner/test_discover_runner.py
@@ -105,7 +105,6 @@ def test_ignore_adjacent(self):
def test_testcase_ordering(self):
suite = DiscoverRunner().build_suite(["test_discovery_sample/"])
- tc_names = [case.__class__.__name__ for case in suite._tests]
self.assertEqual(
suite._tests[0].__class__.__name__,
'TestDjangoTestCase',
Please sign in to comment.
Something went wrong with that request. Please try again.