Get rid of simplejson and fix detection of JSON renderer

cornice_swagger/swagger.py

@@ -5,6 +5,7 @@
 
 
 import colander
+from cornice import Service
 from cornice.util import to_list
 from pyramid.threadlocal import get_current_registry
 import six
@@ -615,7 +616,12 @@ def _extract_operation_from_view(self, view, args):
         # If 'produces' are not defined in the view, try get from renderers
         renderer = args.get('renderer', '')
 
-        if "json" in renderer:  # allows for "json" or "simplejson"
+        is_json_renderer = (
+            "json" in renderer  # allows for "json" or "simplejson"
+            or renderer == Service.renderer  # default renderer is json.
+        )
+
+        if is_json_renderer:
             produces = ['application/json']
         elif renderer == 'xml':
             produces = ['text/xml']

requirements.txt

@@ -7,7 +7,6 @@ plaster==1.0
 plaster-pastedeploy==0.7
 pyramid==1.10.4
 repoze.lru==0.7
-simplejson==3.17.2
 six==1.15.0
 translationstring==1.4
 venusian==3.0.0

tests/test_swagger.py

@@ -116,6 +116,19 @@ def test_swagger_call_is_deprecated(self, mocked):
 
 class ExtractContentTypesTest(unittest.TestCase):
 
+    def test_default_renderer(self):
+        service = Service("IceCream", "/icecream/{flavour}")
+
+        class IceCream(object):
+            @service.get()
+            def view_get(self, request):
+                return self.request.validated
+
+        swagger = CorniceSwagger([service])
+        spec = swagger.generate()
+        self.assertEquals(spec['paths']['/icecream/{flavour}']['get']['produces'],
+                          ['application/json'])
+
     def test_json_renderer(self):
 
         service = Service("IceCream", "/icecream/{flavour}")