updated unittests of python client

Author: geekerzp

modules/swagger-codegen/src/main/resources/python/swagger.mustache

@@ -105,7 +105,7 @@ class ApiClient(object):
   def to_path_value(self, obj):
     """
     Convert a string or object to a path-friendly value
-    
+
     :param obj: object or string value
 
     :return string: quoted value
@@ -259,8 +259,11 @@ class ApiClient(object):
     Return `Accept` based on an array of accepts provided
     """
     if not accepts:
-      return 
-    if 'application/json'.lower() in accepts:
+      return
+
+    accepts = list(map(lambda x: x.lower(), accepts))
+
+    if 'application/json' in accepts:
       return 'application/json'
     else:
       return ', '.join(accepts)
@@ -272,9 +275,10 @@ class ApiClient(object):
     """
     if not content_types:
       return 'application/json'
-    if 'application/json'.lower() in content_types:
+
+    content_types = list(map(lambda x: x.lower(), content_types))
+
+    if 'application/json' in content_types:
       return 'application/json'
     else:
       return content_types[0]
-
-

samples/client/petstore/python/SwaggerPetstore-python/SwaggerPetstore/swagger.py

@@ -105,7 +105,7 @@ def call_api(self, resource_path, method, path_params=None, query_params=None, h
   def to_path_value(self, obj):
     """
     Convert a string or object to a path-friendly value
-    
+
     :param obj: object or string value
 
     :return string: quoted value
@@ -259,8 +259,11 @@ def select_header_accept(accepts):
     Return `Accept` based on an array of accepts provided
     """
     if not accepts:
-      return 
-    if 'application/json'.lower() in accepts:
+      return
+
+    accepts = list(map(lambda x: x.lower(), accepts))
+
+    if 'application/json' in accepts:
       return 'application/json'
     else:
       return ', '.join(accepts)
@@ -272,9 +275,10 @@ def select_header_content_type(content_types):
     """
     if not content_types:
       return 'application/json'
-    if 'application/json'.lower() in content_types:
+
+    content_types = list(map(lambda x: x.lower(), content_types))
+
+    if 'application/json' in content_types:
       return 'application/json'
     else:
       return content_types[0]
-
-

samples/client/petstore/python/SwaggerPetstore-python/tests/test_api_client.py

@@ -22,6 +22,10 @@ def setUp(self):
         self.api_client = SwaggerPetstore.ApiClient(HOST)
 
     def test_select_header_accept(self):
+        accepts = ['APPLICATION/JSON', 'APPLICATION/XML']
+        accept = SwaggerPetstore.ApiClient.select_header_accept(accepts)
+        self.assertEqual(accept, 'application/json')
+        
         accepts = ['application/json', 'application/xml']
         accept = SwaggerPetstore.ApiClient.select_header_accept(accepts)
         self.assertEqual(accept, 'application/json')
@@ -39,6 +43,10 @@ def test_select_header_accept(self):
         self.assertEqual(accept, None)
 
     def test_select_header_content_type(self):
+        content_types = ['APPLICATION/JSON', 'APPLICATION/XML']
+        content_type = SwaggerPetstore.ApiClient.select_header_content_type(content_types)
+        self.assertEqual(content_type, 'application/json')
+        
         content_types = ['application/json', 'application/xml']
         content_type = SwaggerPetstore.ApiClient.select_header_content_type(content_types)
         self.assertEqual(content_type, 'application/json')