Merge pull request #668 from dtkav/fix_forms

Fix form defaults
spec-first · Aug 27, 2018 · cd2b8e4 · cd2b8e4
2 parents ca1607a + 18de95e
commit cd2b8e4
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/connexion/operations/swagger2.py b/connexion/operations/swagger2.py
@@ -250,9 +250,9 @@ def _get_body_argument(self, body, arguments, has_kwargs, sanitize):
                       for p in self.parameters
                       if p['in'] == 'formData'}
 
-        default_form_params = {sanitize(p['name']): p['default']
-                               for p in form_defns
-                               if 'default' in p}
+        default_form_params = {k: v['default']
+                               for k, v in form_defns.items()
+                               if 'default' in v}
 
         # Add body parameters
         if body_name:

diff --git a/tests/api/test_parameters.py b/tests/api/test_parameters.py
@@ -87,12 +87,12 @@ def test_array_form_param(simple_app):
     data = 'items=A&items=B&items=C&items=D,E,F'
     response = app_client.post(url, headers=headers, data=data)
     array_response = json.loads(response.data.decode('utf-8', 'replace'))  # type: [str] multi array with csv format
-    assert array_response == ['A', 'B', 'C', 'D', 'E', 'F']
+    assert array_response == ['D', 'E', 'F']
     url = '/v1.0/test_array_pipes_form_param'
     data = 'items=4&items=5&items=6&items=7|8|9'
     response = app_client.post(url, headers=headers, data=data)
     array_response = json.loads(response.data.decode('utf-8', 'replace'))  # type: [int] multi array with pipes format
-    assert array_response == [4, 5, 6, 7, 8, 9]
+    assert array_response == [7, 8, 9]
 
 
 def test_extra_query_param(simple_app):