_prepare_*_doc callbacks: make app, spec and method kw-only

Author: lafrech

flask_smorest/arguments.py

@@ -91,7 +91,7 @@ def wrapper(*f_args, **f_kwargs):
 
         return decorator
 
-    def _prepare_arguments_doc(self, doc, doc_info, spec, **kwargs):
+    def _prepare_arguments_doc(self, doc, doc_info, *, spec, **kwargs):
         # This callback should run first as it overrides existing parameters
         # in doc. Following callbacks should append to parameters list.
         operation = doc_info.get('arguments')

flask_smorest/etag.py

@@ -247,7 +247,7 @@ def _set_etag_in_response(self, response, etag_schema):
                 self._check_not_modified(new_etag)
             response.set_etag(new_etag)
 
-    def _prepare_etag_doc(self, doc, doc_info, app, method, **kwargs):
+    def _prepare_etag_doc(self, doc, doc_info, *, app, method, **kwargs):
         if (
                 doc_info.get('etag', False) and
                 not app.config.get('ETAG_DISABLED', False)

flask_smorest/pagination.py

@@ -285,7 +285,7 @@ def _document_pagination_metadata(self, spec, resp_doc):
             }
         }
 
-    def _prepare_pagination_doc(self, doc, doc_info, spec, **kwargs):
+    def _prepare_pagination_doc(self, doc, doc_info, *, spec, **kwargs):
         operation = doc_info.get('pagination')
         if operation:
             doc.setdefault('parameters', []).append(operation['parameters'])

flask_smorest/response.py

@@ -157,7 +157,7 @@ def _prepare_response_content(data):
         return data
 
     @staticmethod
-    def _prepare_response_doc(doc, doc_info, spec, **kwargs):
+    def _prepare_response_doc(doc, doc_info, *, spec, **kwargs):
         operation = doc_info.get('response')
         if operation:
             for response in operation['responses'].values():

tests/test_examples.py

@@ -527,7 +527,7 @@ def _prepare_response_doc(doc, doc_info, spec, **kwargs):
                                 )
                 return super(
                     WrapperBlueprint, WrapperBlueprint
-                )._prepare_response_doc(doc, doc_info, spec, **kwargs)
+                )._prepare_response_doc(doc, doc_info, spec=spec, **kwargs)
 
         app.config['OPENAPI_VERSION'] = openapi_version
         api = Api(app)