use compact separators to keep json overlay small

mapbox · Dec 4, 2015 · 49834d3 · 49834d3
1 parent a04cb61
commit 49834d3
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 6 deletions.
diff --git a/mapbox/services/static.py b/mapbox/services/static.py
@@ -30,7 +30,8 @@ def image(self, mapid, lon=None, lat=None, z=None, features=None,
 
         if features:
             values['overlay'] = json.dumps({'type': 'FeatureCollection',
-                                            'features': features})
+                                            'features': features},
+                                           separators=(',', ':'))
 
             if len(values['overlay']) > 4087:  # limit is 4096 minus the 'geojson()'
                 raise ValueError("geojson is too large for the static maps API, "

diff --git a/tests/test_staticmaps.py b/tests/test_staticmaps.py
@@ -56,7 +56,7 @@ def test_staticmap_lonlatz_only():
 def test_staticmap_lonlatz_features(points):
 
     overlay = json.dumps({'type': 'FeatureCollection',
-                          'features': points})
+                          'features': points}, separators=(',', ':'))
     overlay = quote(overlay)
     url = ('https://api.mapbox.com/v4/mapbox.satellite/geojson({0})/'
            '-61.7,12.1,12/600x600.png256?access_token=pk.test'.format(overlay))
@@ -69,15 +69,15 @@ def test_staticmap_lonlatz_features(points):
         content_type='image/png')
 
     res = mapbox.Static(access_token='pk.test').image('mapbox.satellite',
-                                                          -61.7, 12.1, 12,
-                                                          points)
+                                                      -61.7, 12.1, 12,
+                                                      points)
     assert res.status_code == 200
 
 @responses.activate
 def test_staticmap_auto_features(points):
 
     overlay = json.dumps({'type': 'FeatureCollection',
-                          'features': points})
+                          'features': points}, separators=(',', ':'))
     overlay = quote(overlay)
     url = ('https://api.mapbox.com/v4/mapbox.satellite/geojson({0})/'
            'auto/600x600.png256?access_token=pk.test'.format(overlay))
@@ -90,7 +90,7 @@ def test_staticmap_auto_features(points):
         content_type='image/png')
 
     res = mapbox.Static(access_token='pk.test').image('mapbox.satellite',
-                                                          features=points)
+                                                      features=points)
     assert res.status_code == 200