Modernize the code for Python 3.9

with the help of pyupgrade

https://github.com/asottile/pyupgrade

Author: ulgens

rest_framework_gis/__init__.py

@@ -3,12 +3,12 @@
 
 
 def get_version():
-    version = '%s.%s' % (VERSION[0], VERSION[1])
+    version = f'{VERSION[0]}.{VERSION[1]}'
     if VERSION[2]:
-        version = '%s.%s' % (version, VERSION[2])
+        version = f'{version}.{VERSION[2]}'
     if VERSION[3:] == ('alpha', 0):
         version = '%s pre-alpha' % version
     else:
         if VERSION[3] != 'final':
-            version = '%s %s' % (version, VERSION[3])
+            version = f'{version} {VERSION[3]}'
     return version

rest_framework_gis/fields.py

@@ -72,9 +72,7 @@ def to_internal_value(self, value):
                 )
             )
         except (ValueError, TypeError, GDALException) as e:
-            raise ValidationError(
-                _('Unable to convert to python object: {}'.format(str(e)))
-            )
+            raise ValidationError(_(f'Unable to convert to python object: {str(e)}'))
 
     def validate_empty_values(self, data):
         if data == '':

rest_framework_gis/filters.py

@@ -46,7 +46,7 @@ def get_filter_bbox(self, request):
             p1x, p1y, p2x, p2y = (float(n) for n in bbox_string.split(','))
         except ValueError:
             raise ParseError(
-                'Invalid bbox string supplied for parameter {0}'.format(self.bbox_param)
+                f'Invalid bbox string supplied for parameter {self.bbox_param}'
             )
 
         x = Polygon.from_bbox((p1x, p1y, p2x, p2y))
@@ -66,7 +66,7 @@ def filter_queryset(self, request, queryset, view):
         bbox = self.get_filter_bbox(request)
         if not bbox:
             return queryset
-        return queryset.filter(Q(**{'%s__%s' % (filter_field, geoDjango_filter): bbox}))
+        return queryset.filter(Q(**{f'{filter_field}__{geoDjango_filter}': bbox}))
 
     def get_schema_operation_parameters(self, view):
         return [
@@ -127,7 +127,7 @@ def get_filter_bbox(self, request):
             z, x, y = (int(n) for n in tile_string.split('/'))
         except ValueError:
             raise ParseError(
-                'Invalid tile string supplied for parameter {0}'.format(self.tile_param)
+                f'Invalid tile string supplied for parameter {self.tile_param}'
             )
 
         bbox = Polygon.from_bbox(tile_edges(x, y, z))
@@ -158,7 +158,7 @@ def get_filter_point(self, request, **kwargs):
             (x, y) = (float(n) for n in point_string.split(','))
         except ValueError:
             raise ParseError(
-                'Invalid geometry string supplied for parameter {0}'.format(
+                'Invalid geometry string supplied for parameter {}'.format(
                     self.point_param
                 )
             )
@@ -212,7 +212,7 @@ def filter_queryset(self, request, queryset, view):
             dist = float(dist_string)
         except ValueError:
             raise ParseError(
-                'Invalid distance string supplied for parameter {0}'.format(
+                'Invalid distance string supplied for parameter {}'.format(
                     self.dist_param
                 )
             )
@@ -222,7 +222,7 @@ def filter_queryset(self, request, queryset, view):
             dist = self.dist_to_deg(dist, point[1])
 
         return queryset.filter(
-            Q(**{'%s__%s' % (filter_field, geoDjango_filter): (point, dist)})
+            Q(**{f'{filter_field}__{geoDjango_filter}': (point, dist)})
         )
 
     def get_schema_operation_parameters(self, view):

rest_framework_gis/schema.py

@@ -101,9 +101,7 @@ def _map_geo_field(self, serializer, geo_field_name):
         try:
             return self.GEO_FIELD_TO_SCHEMA[geo_field.__class__]
         except KeyError:
-            warnings.warn(
-                "Geometry generation for {field} is not supported.".format(field=field)
-            )
+            warnings.warn(f"Geometry generation for {field} is not supported.")
             return {}
 
     def map_field(self, field):

rest_framework_gis/serializers.py

@@ -81,9 +81,7 @@ def __init__(self, *args, **kwargs):
         def check_excludes(field_name, field_role):
             """make sure the field is not excluded"""
             if hasattr(meta, 'exclude') and field_name in meta.exclude:
-                raise ImproperlyConfigured(
-                    "You cannot exclude your '{0}'.".format(field_role)
-                )
+                raise ImproperlyConfigured(f"You cannot exclude your '{field_role}'.")
 
         def add_to_fields(field_name):
             """Make sure the field is included in the fields"""

tests/django_restframework_gis_tests/test_bbox.py

@@ -53,7 +53,7 @@ def test_list(self):
             elif fid == 2:
                 self.assertEqual(feature['bbox'], self.bl2.bbox_geometry.extent)
             else:
-                self.fail("Unexpected id: {0}".format(fid))
+                self.fail(f"Unexpected id: {fid}")
         BoxedLocation.objects.all().delete()
 
     def test_post_location_list_geojson(self):

tests/django_restframework_gis_tests/test_fields.py

@@ -116,7 +116,7 @@
 class BaseTestCase(TestCase):
     @staticmethod
     def get_instance(data_dict):
-        class Model(object):
+        class Model:
             def __init__(self, geojson_dict):
                 self.geometry = GEOSGeometry(json.dumps(geojson_dict))
 

tests/django_restframework_gis_tests/test_filters.py

@@ -274,13 +274,13 @@ def test_DistanceToPointFilter_filtering(self):
         distance = 5000  # meters
         point_on_alcatraz = [-122.4222, 37.82667]
 
-        url_params = '?dist=%0.4f&point=hello&format=json' % (distance,)
+        url_params = f'?dist={distance:0.4f}&point=hello&format=json'
         response = self.client.get(
-            '%s%s' % (self.location_within_distance_of_point_list_url, url_params)
+            f'{self.location_within_distance_of_point_list_url}{url_params}'
         )
         self.assertEqual(response.status_code, 400)
 
-        url_params = '?dist=%0.4f&point=%0.4f,%0.4f&format=json' % (
+        url_params = '?dist={:0.4f}&point={:0.4f},{:0.4f}&format=json'.format(
             distance,
             point_on_alcatraz[0],
             point_on_alcatraz[1],
@@ -299,21 +299,21 @@ def test_DistanceToPointFilter_filtering(self):
 
         # Make sure we only get back the ones within the distance
         response = self.client.get(
-            '%s%s' % (self.location_within_distance_of_point_list_url, url_params)
+            f'{self.location_within_distance_of_point_list_url}{url_params}'
         )
         self.assertEqual(len(response.data['features']), 1)
         for result in response.data['features']:
             self.assertEqual(result['properties']['name'], treasure_island.name)
 
         # Make sure we get back all the ones within the distance
         distance = 7000
-        url_params = '?dist=%0.4f&point=%0.4f,%0.4f&format=json' % (
+        url_params = '?dist={:0.4f}&point={:0.4f},{:0.4f}&format=json'.format(
             distance,
             point_on_alcatraz[0],
             point_on_alcatraz[1],
         )
         response = self.client.get(
-            '%s%s' % (self.location_within_distance_of_point_list_url, url_params)
+            f'{self.location_within_distance_of_point_list_url}{url_params}'
         )
         self.assertEqual(len(response.data['features']), 2)
         for result in response.data['features']:
@@ -323,7 +323,7 @@ def test_DistanceToPointFilter_filtering(self):
 
         # Make sure we only get back the ones within the distance
         degrees = 0.05
-        url_params = '?dist=%0.4f&point=%0.4f,%0.4f&format=json' % (
+        url_params = '?dist={:0.4f}&point={:0.4f},{:0.4f}&format=json'.format(
             degrees,
             point_on_alcatraz[0],
             point_on_alcatraz[1],
@@ -348,7 +348,7 @@ def test_DistanceToPointOrderingFilter_filtering(self):
 
         url_params = '?point=hello&format=json'
         response = self.client.get(
-            '%s%s' % (self.location_order_distance_to_point, url_params)
+            f'{self.location_order_distance_to_point}{url_params}'
         )
         self.assertEqual(response.status_code, 400)
 
@@ -381,7 +381,7 @@ def test_DistanceToPointOrderingFilter_filtering(self):
 
         url_params = '?point=%i,%i&format=json' % (point[0], point[1])
         response = self.client.get(
-            '%s%s' % (self.location_order_distance_to_point, url_params)
+            f'{self.location_order_distance_to_point}{url_params}'
         )
         self.assertEqual(len(response.data['features']), 8)
         self.assertEqual(
@@ -400,7 +400,7 @@ def test_DistanceToPointOrderingFilter_filtering(self):
 
         url_params = '?point=%i,%i&order=desc&format=json' % (point[0], point[1])
         response = self.client.get(
-            '%s%s' % (self.location_order_distance_to_point, url_params)
+            f'{self.location_order_distance_to_point}{url_params}'
         )
         self.assertEqual(len(response.data['features']), 8)
         self.assertEqual(
@@ -445,11 +445,9 @@ def test_GeometryField_filtering(self):
         except AttributeError:
             quoted_param = urllib.parse.quote(point_inside_ggpark_geojson)
 
-        url_params = "?contains_properly=%s" % (quoted_param,)
+        url_params = f"?contains_properly={quoted_param}"
 
-        response = self.client.get(
-            '{0}{1}'.format(self.geojson_contained_in_geometry, url_params)
-        )
+        response = self.client.get(f'{self.geojson_contained_in_geometry}{url_params}')
         self.assertEqual(len(response.data), 1)
 
         geometry_response = GEOSGeometry(json.dumps(response.data[0]['geometry']))
@@ -511,7 +509,7 @@ def test_TileFilter_ValueError(self):
     def test_DistanceToPointFilter_filtering_none(self):
         url_params = '?dist=5000&point=&format=json'
         response = self.client.get(
-            '%s%s' % (self.location_within_distance_of_point_list_url, url_params)
+            f'{self.location_within_distance_of_point_list_url}{url_params}'
         )
         self.assertDictEqual(
             response.data, {'type': 'FeatureCollection', 'features': []}
@@ -522,7 +520,7 @@ def test_DistanceToPointFilter_filter_field_none(self):
         GeojsonLocationWithinDistanceOfPointList.distance_filter_field = None
         url_params = '?dist=5000&point=&format=json'
         response = self.client.get(
-            '%s%s' % (self.location_within_distance_of_point_list_url, url_params)
+            f'{self.location_within_distance_of_point_list_url}{url_params}'
         )
         self.assertDictEqual(
             response.data, {'type': 'FeatureCollection', 'features': []}
@@ -532,7 +530,7 @@ def test_DistanceToPointFilter_filter_field_none(self):
     def test_DistanceToPointFilter_ValueError_point(self):
         url_params = '?dist=500.0&point=hello&format=json'
         response = self.client.get(
-            '%s%s' % (self.location_within_distance_of_point_list_url, url_params)
+            f'{self.location_within_distance_of_point_list_url}{url_params}'
         )
         self.assertEqual(
             response.data['detail'],
@@ -542,7 +540,7 @@ def test_DistanceToPointFilter_ValueError_point(self):
     def test_DistanceToPointFilter_ValueError_distance(self):
         url_params = '?dist=wrong&point=12.0,42.0&format=json'
         response = self.client.get(
-            '%s%s' % (self.location_within_distance_of_point_list_url, url_params)
+            f'{self.location_within_distance_of_point_list_url}{url_params}'
         )
         self.assertEqual(
             response.data['detail'],
@@ -552,7 +550,7 @@ def test_DistanceToPointFilter_ValueError_distance(self):
     def test_DistanceToPointOrderingFilter_filtering_none(self):
         url_params = '?point=&format=json'
         response = self.client.get(
-            '%s%s' % (self.location_order_distance_to_point, url_params)
+            f'{self.location_order_distance_to_point}{url_params}'
         )
         self.assertDictEqual(
             response.data, {'type': 'FeatureCollection', 'features': []}
@@ -565,7 +563,7 @@ def test_DistanceToPointOrderingFilter_ordering_filter_field_none(self):
         GeojsonLocationOrderDistanceToPointList.distance_ordering_filter_field = None
         url_params = '?point=&format=json'
         response = self.client.get(
-            '%s%s' % (self.location_order_distance_to_point, url_params)
+            f'{self.location_order_distance_to_point}{url_params}'
         )
         self.assertDictEqual(
             response.data, {'type': 'FeatureCollection', 'features': []}

tests/django_restframework_gis_tests/test_performance.py

@@ -49,7 +49,7 @@ class Meta:
             with Timer() as t:
                 JSONRenderer().render(serializer.data)
             # print results
-            msg = 'GeoJSON rendering of {0} objects ' 'completed in {1}'.format(
+            msg = 'GeoJSON rendering of {} objects ' 'completed in {}'.format(
                 self.NUMBER_OF_LOCATIONS, t.elapsed
             )
-            print('\n\033[95m{0}\033[0m'.format(msg))
+            print(f'\n\033[95m{msg}\033[0m')

tests/django_restframework_gis_tests/tests.py

@@ -662,7 +662,7 @@ def test_geometry_empty_representation(self):
         )
         for geom_type in geom_types:
             with self.subTest(geom_type=geom_type):
-                value = f.to_representation(GEOSGeometry('{} EMPTY'.format(geom_type)))
+                value = f.to_representation(GEOSGeometry(f'{geom_type} EMPTY'))
                 self.assertIsNotNone(value)
                 if geom_type == 'LINEARRING':
                     geom_type = 'LINESTRING'
@@ -699,9 +699,7 @@ def test_geojson_pagination(self):
         response = self.client.get(self.geojson_location_list_url)
         self.assertEqual(response.data['type'], 'FeatureCollection')
         self.assertEqual(len(response.data['features']), 2)
-        response = self.client.get(
-            '{0}?page_size=1'.format(self.geojson_location_list_url)
-        )
+        response = self.client.get(f'{self.geojson_location_list_url}?page_size=1')
         self.assertEqual(response.data['type'], 'FeatureCollection')
         self.assertEqual(len(response.data['features']), 1)
         self.assertIn('next', response.data)