#104 support type casting of int/float path parameters

connexion/api.py

@@ -118,7 +118,8 @@ def add_operation(self, method, path, swagger_operation):
         operation_id = operation.operation_id
         logger.debug('... Adding %s -> %s', method.upper(), operation_id, extra=vars(operation))
 
-        self.blueprint.add_url_rule(path, operation.endpoint_name, operation.function, methods=[method])
+        flask_path = utils.flaskify_path(path, operation.get_path_parameter_types())
+        self.blueprint.add_url_rule(flask_path, operation.endpoint_name, operation.function, methods=[method])
 
     def add_paths(self, paths=None):
         """
@@ -129,7 +130,6 @@ def add_paths(self, paths=None):
         paths = paths or self.specification.get('paths', dict())
         for path, methods in paths.items():
             logger.debug('Adding %s%s...', self.base_url, path)
-            path = utils.flaskify_path(path)
             # TODO Error handling
             for method, endpoint in methods.items():
                 try:

connexion/operation.py

@@ -134,6 +134,9 @@ def resolve_parameters(self, parameters):
             param = self.resolve_reference(param)
             yield param
 
+    def get_path_parameter_types(self):
+        return {p['name']: p.get('type') for p in self.parameters if p['in'] == 'path'}
+
     @property
     def body_schema(self):
         """

tests/fakeapi/api.yaml

@@ -563,6 +563,14 @@ paths:
         - name: someint
           in: path
           type: integer
+  /test-float-path/{somefloat}:
+    get:
+      summary: Test type casting of path parameter
+      operationId: fakeapi.hello.test_get_somefloat
+      parameters:
+        - name: somefloat
+          in: path
+          type: number
 definitions:
   new_stack:
     type: object

tests/fakeapi/hello.py

@@ -186,4 +186,8 @@ def test_schema_int(test_int):
 
 
 def test_get_someint(someint):
-    return str(type(someint))
+    return type(someint).__name__
+
+
+def test_get_somefloat(somefloat):
+    return type(somefloat).__name__

tests/test_app.py

@@ -528,4 +528,10 @@ def test_resolve_classmethod(app):
 def test_path_parameter_someint(app):
     app_client = app.app.test_client()
     resp = app_client.get('/v1.0/test-int-path/123')  # type: flask.Response
-    assert resp.data.decode() == '"<type \'int\'>"'
+    assert resp.data.decode() == '"int"'
+
+
+def test_path_parameter_somefloat(app):
+    app_client = app.app.test_client()
+    resp = app_client.get('/v1.0/test-float-path/123.45')  # type: flask.Response
+    assert resp.data.decode() == '"float"'