Merge pull request #172 from rafaelcaricio/master

Type cast in path parameters

README.rst

@@ -265,7 +265,6 @@ available type castings are:
 
 +--------------+-------------+
 | Swagger Type | Python Type |
-|              |             |
 +==============+=============+
 | integer      | int         |
 +--------------+-------------+

connexion/decorators/parameter.py

@@ -65,6 +65,8 @@ def parameter_to_arg(parameters, function):
                    for parameter in parameters if parameter['in'] == 'query'}  # type: dict[str, str]
     form_types = {parameter['name']: parameter
                   for parameter in parameters if parameter['in'] == 'formData'}
+    path_types = {parameter['name']: parameter
+                  for parameter in parameters if parameter['in'] == 'path'}
     arguments = get_function_arguments(function)
     default_query_params = {param['name']: param['default']
                             for param in parameters if param['in'] == 'query' and 'default' in param}
@@ -83,6 +85,12 @@ def wrapper(*args, **kwargs):
         if default_body and not request_body:
             request_body = default_body
 
+        # Parse path parameters
+        for key, path_param_definitions in path_types.items():
+            if key in kwargs:
+                kwargs[key] = get_val_from_param(kwargs[key],
+                                                 path_param_definitions)
+
         # Add body parameters
         if request_body is not None:
             if body_name not in arguments:

docs/conf.py

@@ -12,8 +12,8 @@
 # All configuration values have a default; values that are commented out
 # serve to show the default.
 
-import sys
-import os
+# import sys
+# import os
 
 # If extensions (or modules to document with autodoc) are in another directory,
 # add these directories to sys.path here. If the directory is relative to the

tests/api/test_parameters.py

@@ -170,3 +170,12 @@ def test_parameters_defined_in_path_level(simple_app):
 
     resp = app_client.get('/v1.0/parameters-in-root-path')
     assert resp.status_code == 400
+
+
+def test_array_in_path(simple_app):
+    app_client = simple_app.app.test_client()
+    resp = app_client.get('/v1.0/test-array-in-path/one_item')
+    assert json.loads(resp.data.decode()) == ["one_item"]
+
+    resp = app_client.get('/v1.0/test-array-in-path/one_item,another_item')
+    assert json.loads(resp.data.decode()) == ["one_item", "another_item"]

tests/fakeapi/__init__.py

@@ -1,2 +1,2 @@
 def get():
-    return ''
+    return ''

tests/fakeapi/foo_bar.py

@@ -1,4 +1,5 @@
 #!/usr/bin/env python3
 
+
 def search():
     return ''

tests/fakeapi/hello.py

@@ -281,3 +281,7 @@ def path_parameters_in_get_method(title):
 
 def test_default_missmatch_definition(age):
     return 'OK'
+
+
+def test_array_in_path(names):
+    return names, 200

tests/fixtures/simple/swagger.yaml

@@ -474,6 +474,28 @@ paths:
               type: string
           description: 204 no content
 
+  /test-array-in-path/{names}:
+    get:
+      operationId: fakeapi.hello.test_array_in_path
+      produces:
+        - application/json
+      parameters:
+        - name: names
+          description: List of names.
+          in: path
+          type: array
+          items:
+            type: string
+          required: true
+          collectionFormat: csv
+      responses:
+        200:
+          description: OK
+          schema:
+            type: array
+            items:
+              type: string
+
 
 definitions:
   new_stack: