summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTim Graham <timograham@gmail.com>2016-06-16 09:07:51 -0400
committerTim Graham <timograham@gmail.com>2016-06-16 10:32:06 -0400
commit13d16697a0de87b37a046d5418426a02b810a5b0 (patch)
treecbf169d90ab07d2ba14c808dd3be720de9661445
parent1cfcc02fc3a2352868fb102118aff22b6e05c610 (diff)
[1.9.x] Fixed flake8 2.6 warnings.
Backport of ea34426ae789d31b036f58c8fd59ce299649e91e from master
-rw-r--r--django/contrib/gis/db/backends/base/models.py8
-rw-r--r--django/contrib/gis/db/models/sql/aggregates.py10
-rw-r--r--tests/postgres_tests/migrations/0002_create_test_models.py5
-rw-r--r--tests/servers/tests.py14
4 files changed, 12 insertions, 25 deletions
diff --git a/django/contrib/gis/db/backends/base/models.py b/django/contrib/gis/db/backends/base/models.py
index a84b23dfe8..7367a9f6a0 100644
--- a/django/contrib/gis/db/backends/base/models.py
+++ b/django/contrib/gis/db/backends/base/models.py
@@ -45,14 +45,14 @@ class SpatialRefSysMixin(object):
try:
self._srs = gdal.SpatialReference(self.wkt)
return self.srs
- except Exception as msg:
- pass
+ except Exception as e:
+ msg = e
try:
self._srs = gdal.SpatialReference(self.proj4text)
return self.srs
- except Exception as msg:
- pass
+ except Exception as e:
+ msg = e
raise Exception('Could not get OSR SpatialReference from WKT: %s\nError:\n%s' % (self.wkt, msg))
else:
diff --git a/django/contrib/gis/db/models/sql/aggregates.py b/django/contrib/gis/db/models/sql/aggregates.py
deleted file mode 100644
index e3fb049cf5..0000000000
--- a/django/contrib/gis/db/models/sql/aggregates.py
+++ /dev/null
@@ -1,10 +0,0 @@
-from django.db.models.sql import aggregates
-from django.db.models.sql.aggregates import * # NOQA
-
-__all__ = ['Collect', 'Extent', 'Extent3D', 'MakeLine', 'Union'] + aggregates.__all__
-
-
-warnings.warn(
- "django.contrib.gis.db.models.sql.aggregates is deprecated. Use "
- "django.contrib.gis.db.models.aggregates instead.",
- RemovedInDjango110Warning, stacklevel=2)
diff --git a/tests/postgres_tests/migrations/0002_create_test_models.py b/tests/postgres_tests/migrations/0002_create_test_models.py
index 4be219f722..0152f88a9f 100644
--- a/tests/postgres_tests/migrations/0002_create_test_models.py
+++ b/tests/postgres_tests/migrations/0002_create_test_models.py
@@ -3,7 +3,10 @@ from __future__ import unicode_literals
from django.db import migrations, models
-from ..fields import * # NOQA
+from ..fields import (
+ ArrayField, BigIntegerRangeField, DateRangeField, DateTimeRangeField,
+ FloatRangeField, HStoreField, IntegerRangeField, JSONField,
+)
class Migration(migrations.Migration):
diff --git a/tests/servers/tests.py b/tests/servers/tests.py
index dbb298e003..fa5e5d4690 100644
--- a/tests/servers/tests.py
+++ b/tests/servers/tests.py
@@ -103,12 +103,9 @@ class LiveServerViews(LiveServerBase):
Ensure that the LiveServerTestCase serves 404s.
Refs #2879.
"""
- try:
+ with self.assertRaises(HTTPError) as err:
self.urlopen('/')
- except HTTPError as err:
- self.assertEqual(err.code, 404, 'Expected 404 response')
- else:
- self.fail('Expected 404 response')
+ self.assertEqual(err.exception.code, 404, 'Expected 404 response')
def test_view(self):
"""
@@ -132,12 +129,9 @@ class LiveServerViews(LiveServerBase):
tries to access a static file that isn't explicitly put under
STATIC_ROOT.
"""
- try:
+ with self.assertRaises(HTTPError) as err:
self.urlopen('/static/another_app/another_app_static_file.txt')
- except HTTPError as err:
- self.assertEqual(err.code, 404, 'Expected 404 response')
- else:
- self.fail('Expected 404 response (got %d)' % err.code)
+ self.assertEqual(err.exception.code, 404, 'Expected 404 response')
def test_media_files(self):
"""