[C#][netcore] fix binary response (#8593)

* fix binary response

* update test file hash

Author: wing328

bin/utils/test_file_list.yaml

@@ -1,6 +1,6 @@
 ---
 # csharp-netcore test files and image for upload
 - filename: "samples/client/petstore/csharp-netcore/OpenAPIClient/src/Org.OpenAPITools.Test/Api/PetApiTests.cs"
-  sha256: b74ef9eefa4b41fd3233e083fe2355babf25a77f9073d28e1aa81ae2e0a5f1d0
+  sha256: aceebba316148a2a803a15ef4e13bbd0b0a1b8d15006cd88adb9b39a620ee451
 - filename: "samples/client/petstore/csharp-netcore/OpenAPIClient/src/Org.OpenAPITools.Test/linux-logo.png"
   sha256: 0a67c32728197e942b13bdda064b73793f12f5c795f1e5cf35a3adf69c973230

modules/openapi-generator/src/main/resources/csharp-netcore/ApiClient.mustache

@@ -472,6 +472,19 @@ namespace {{packageName}}.Client
                 response = client.Execute<T>(req);
             }
 
+            // if the response type is oneOf/anyOf, call FromJSON to deserialize the data
+            if (typeof({{{packageName}}}.{{modelPackage}}.AbstractOpenAPISchema).IsAssignableFrom(typeof(T)))
+            {
+                T instance = (T)Activator.CreateInstance(typeof(T));
+                MethodInfo method = typeof(T).GetMethod("FromJson");
+                method.Invoke(instance, new object[] { response.Content });
+                response.Data = instance;
+            }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
+
             InterceptResponse(req, response);
 
             var result = ToApiResponse(response);
@@ -583,6 +596,10 @@ namespace {{packageName}}.Client
                 method.Invoke(instance, new object[] { response.Content });
                 response.Data = instance;
             }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
 
             InterceptResponse(req, response);
 

samples/client/petstore/csharp-netcore/OpenAPIClient-net47/src/Org.OpenAPITools/Client/ApiClient.cs

@@ -477,6 +477,19 @@ private ApiResponse<T> Exec<T>(RestRequest req, IReadableConfiguration configura
                 response = client.Execute<T>(req);
             }
 
+            // if the response type is oneOf/anyOf, call FromJSON to deserialize the data
+            if (typeof(Org.OpenAPITools.Model.AbstractOpenAPISchema).IsAssignableFrom(typeof(T)))
+            {
+                T instance = (T)Activator.CreateInstance(typeof(T));
+                MethodInfo method = typeof(T).GetMethod("FromJson");
+                method.Invoke(instance, new object[] { response.Content });
+                response.Data = instance;
+            }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
+
             InterceptResponse(req, response);
 
             var result = ToApiResponse(response);
@@ -587,6 +600,10 @@ private ApiResponse<T> Exec<T>(RestRequest req, IReadableConfiguration configura
                 method.Invoke(instance, new object[] { response.Content });
                 response.Data = instance;
             }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
 
             InterceptResponse(req, response);
 

samples/client/petstore/csharp-netcore/OpenAPIClient-net5.0/src/Org.OpenAPITools/Client/ApiClient.cs

@@ -477,6 +477,19 @@ private ApiResponse<T> Exec<T>(RestRequest req, IReadableConfiguration configura
                 response = client.Execute<T>(req);
             }
 
+            // if the response type is oneOf/anyOf, call FromJSON to deserialize the data
+            if (typeof(Org.OpenAPITools.Model.AbstractOpenAPISchema).IsAssignableFrom(typeof(T)))
+            {
+                T instance = (T)Activator.CreateInstance(typeof(T));
+                MethodInfo method = typeof(T).GetMethod("FromJson");
+                method.Invoke(instance, new object[] { response.Content });
+                response.Data = instance;
+            }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
+
             InterceptResponse(req, response);
 
             var result = ToApiResponse(response);
@@ -587,6 +600,10 @@ private ApiResponse<T> Exec<T>(RestRequest req, IReadableConfiguration configura
                 method.Invoke(instance, new object[] { response.Content });
                 response.Data = instance;
             }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
 
             InterceptResponse(req, response);
 

samples/client/petstore/csharp-netcore/OpenAPIClient/src/Org.OpenAPITools.Test/Api/PetApiTests.cs

@@ -199,6 +199,19 @@ public void TestGetPetByIdAsync()
             Assert.Equal("sample category name2", response.Category.Name);
         }
 
+        /* a simple test for binary response. no longer in use.
+        [Fact]
+        public void TestGetByIdBinaryResponse()
+        {
+            PetApi petApi = new PetApi(c1);
+            Stream response = petApi.GetPetByIdBinaryResponse(petId);
+            Assert.IsType<System.IO.MemoryStream>(response);
+            StreamReader reader = new StreamReader(response);
+            // the following will fail for sure
+            //Assert.Equal("someting", reader.ReadToEnd());
+        }
+        */
+
         /// <summary>
         /// Test GetPetByIdWithHttpInfoAsync
         /// </summary>

samples/client/petstore/csharp-netcore/OpenAPIClient/src/Org.OpenAPITools/Client/ApiClient.cs

@@ -476,6 +476,19 @@ private ApiResponse<T> Exec<T>(RestRequest req, IReadableConfiguration configura
                 response = client.Execute<T>(req);
             }
 
+            // if the response type is oneOf/anyOf, call FromJSON to deserialize the data
+            if (typeof(Org.OpenAPITools.Model.AbstractOpenAPISchema).IsAssignableFrom(typeof(T)))
+            {
+                T instance = (T)Activator.CreateInstance(typeof(T));
+                MethodInfo method = typeof(T).GetMethod("FromJson");
+                method.Invoke(instance, new object[] { response.Content });
+                response.Data = instance;
+            }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
+
             InterceptResponse(req, response);
 
             var result = ToApiResponse(response);
@@ -586,6 +599,10 @@ private ApiResponse<T> Exec<T>(RestRequest req, IReadableConfiguration configura
                 method.Invoke(instance, new object[] { response.Content });
                 response.Data = instance;
             }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
 
             InterceptResponse(req, response);
 

samples/client/petstore/csharp-netcore/OpenAPIClientCore/src/Org.OpenAPITools/Client/ApiClient.cs

@@ -477,6 +477,19 @@ private ApiResponse<T> Exec<T>(RestRequest req, IReadableConfiguration configura
                 response = client.Execute<T>(req);
             }
 
+            // if the response type is oneOf/anyOf, call FromJSON to deserialize the data
+            if (typeof(Org.OpenAPITools.Model.AbstractOpenAPISchema).IsAssignableFrom(typeof(T)))
+            {
+                T instance = (T)Activator.CreateInstance(typeof(T));
+                MethodInfo method = typeof(T).GetMethod("FromJson");
+                method.Invoke(instance, new object[] { response.Content });
+                response.Data = instance;
+            }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
+
             InterceptResponse(req, response);
 
             var result = ToApiResponse(response);
@@ -587,6 +600,10 @@ private ApiResponse<T> Exec<T>(RestRequest req, IReadableConfiguration configura
                 method.Invoke(instance, new object[] { response.Content });
                 response.Data = instance;
             }
+            else if (typeof(T).Name == "Stream") // for binary response
+            {
+                response.Data = (T)(object)new MemoryStream(response.RawBytes);
+            }
 
             InterceptResponse(req, response);