apidoc: Set default_in correctly for Schema/fields

default_in was changed to a mandatory kwarg parameter
in apispec 3.x
In a flask-apispec 0.8.4 this was changed to be set to "body"

But this does not work correctly for GET urls where the params
should be set to query-params base on the existig default_in value.
Now, we retain the previous behavior where if a default-in is provided
use that, if not provided - use "body"

Author: AbdealiLoKo

flask_apispec/apidoc.py

@@ -98,7 +98,7 @@ def get_parameters(self, rule, view, docs, parent=None):
             locations = options.pop('locations', None)
             if locations:
                 options['default_in'] = locations[0]
-            else:
+            elif 'default_in' not in options:
                 options['default_in'] = 'body'
             extra_params += converter(schema, **options) if args else []
 

tests/test_openapi.py

@@ -299,7 +299,7 @@ def test_params(self, app, path):
         assert params == expected
 
 
-class TestFiledsNoLocationProvided:
+class TestGetFieldsNoLocationProvided:
 
     @pytest.fixture
     def function_view(self, app):
@@ -320,26 +320,19 @@ def path(self, app, spec, function_view):
 
     def test_params(self, app, path):
         params = path['get']['parameters']
-        rule = app.url_map._rules_by_endpoint['get_band'][0]
-        expected = (
-                [{
-                    'in': 'body',
-                    'name': 'body',
-                    'required': False,
-                    'schema': {
-                        'properties': {
-                            'address': {
-                                'type': 'string'
-                            },
-                            'name': {
-                                'type': 'string'
-                            }
-                        },
-                        'type': 'object'
-                    },
-                }] + rule_to_params(rule)
-        )
-        assert params == expected
+        assert {
+            'in': 'body',
+            'name': 'body',
+            'required': False,
+            'schema': {
+                'properties': {
+                    'address': {'type': 'string'},
+                    'name': {'type': 'string'},
+                },
+                'type': 'object',
+            },
+        } in params
+
 
 class TestSchemaNoLocationProvided:
 
@@ -364,13 +357,5 @@ def path(self, app, spec, function_view):
 
     def test_params(self, app, path):
         params = path['get']['parameters']
-        rule = app.url_map._rules_by_endpoint['get_band'][0]
-        expected = (
-                [{
-                    'in': 'body',
-                    'name': 'body',
-                    'required': False,
-                    'schema': {'$ref': '#/definitions/Body'}
-                }] + rule_to_params(rule)
-        )
-        assert params == expected
+        assert {'name': 'body', 'in': 'body', 'required': False,
+                'schema': {'$ref': '#/definitions/Body'}} in params