diff --git a/django/contrib/gis/db/backends/postgis/operations.py b/django/contrib/gis/db/backends/postgis/operations.py index 4d0c2b8865..3830b3f0db 100644 --- a/django/contrib/gis/db/backends/postgis/operations.py +++ b/django/contrib/gis/db/backends/postgis/operations.py @@ -73,7 +73,7 @@ class PostGISOperations(BaseSpatialOperations, DatabaseOperations): 'left': PostGISOperator(op='<<'), 'right': PostGISOperator(op='>>'), 'strictly_below': PostGISOperator(op='<<|'), - 'stricly_above': PostGISOperator(op='|>>'), + 'strictly_above': PostGISOperator(op='|>>'), 'same_as': PostGISOperator(op='~='), 'exact': PostGISOperator(op='~='), # alias of same_as 'contains_properly': PostGISOperator(func='ST_ContainsProperly'), diff --git a/docs/releases/1.8.6.txt b/docs/releases/1.8.6.txt index 1c2af01c7d..a17af96380 100644 --- a/docs/releases/1.8.6.txt +++ b/docs/releases/1.8.6.txt @@ -34,3 +34,6 @@ Bugfixes and retrieval (:ticket:`25563`). This prevents any models generated in data migrations using ``QuerySet.defer()`` from leaking to test and application code. + +* Fixed a typo in the name of the `strictly_above` PostGIS lookup + (:ticket:`25592`). diff --git a/tests/gis_tests/geoapp/tests.py b/tests/gis_tests/geoapp/tests.py index 0dbf3fa5c5..b4b82b67a0 100644 --- a/tests/gis_tests/geoapp/tests.py +++ b/tests/gis_tests/geoapp/tests.py @@ -15,7 +15,7 @@ from django.test import TestCase, ignore_warnings, skipUnlessDBFeature from django.utils import six from django.utils.deprecation import RemovedInDjango20Warning -from ..utils import no_oracle, oracle, postgis, spatialite +from ..utils import no_oracle, oracle, postgis, skipUnlessGISLookup, spatialite from .models import ( City, Country, Feature, MinusOneSRID, NonConcreteModel, PennsylvaniaCity, State, Track, @@ -349,6 +349,20 @@ class GeoLookupTest(TestCase): for c in qs: self.assertIn(c.name, cities) + @skipUnlessGISLookup("strictly_above", "strictly_below") + def test_strictly_above_below_lookups(self): + dallas = City.objects.get(name='Dallas') + self.assertQuerysetEqual( + City.objects.filter(point__strictly_above=dallas.point).order_by('name'), + ['Chicago', 'Lawrence', 'Oklahoma City', 'Pueblo', 'Victoria'], + lambda b: b.name + ) + self.assertQuerysetEqual( + City.objects.filter(point__strictly_below=dallas.point).order_by('name'), + ['Houston', 'Wellington'], + lambda b: b.name + ) + def test_equals_lookups(self): "Testing the 'same_as' and 'equals' lookup types." pnt = fromstr('POINT (-95.363151 29.763374)', srid=4326) diff --git a/tests/gis_tests/utils.py b/tests/gis_tests/utils.py index 3a43f8852b..6eb029c1d5 100644 --- a/tests/gis_tests/utils.py +++ b/tests/gis_tests/utils.py @@ -1,13 +1,30 @@ -from unittest import skip +import unittest +from functools import wraps from django.conf import settings -from django.db import DEFAULT_DB_ALIAS +from django.db import DEFAULT_DB_ALIAS, connection + + +def skipUnlessGISLookup(*gis_lookups): + """ + Skip a test unless a database supports all of gis_lookups. + """ + def decorator(test_func): + @wraps(test_func) + def skip_wrapper(*args, **kwargs): + if any(key not in connection.ops.gis_operators for key in gis_lookups): + raise unittest.SkipTest( + "Database doesn't support all the lookups: %s" % ", ".join(gis_lookups) + ) + return test_func(*args, **kwargs) + return skip_wrapper + return decorator def no_backend(test_func, backend): "Use this decorator to disable test on specified backend." if settings.DATABASES[DEFAULT_DB_ALIAS]['ENGINE'].rsplit('.')[-1] == backend: - @skip("This test is skipped on '%s' backend" % backend) + @unittest.skip("This test is skipped on '%s' backend" % backend) def inner(): pass return inner