Merge pull request #926 from /issues/842

Added fix to handle case where produces or consumes is empty array

Author: baywet

src/Microsoft.OpenApi.Readers/V2/OpenApiDocumentDeserializer.cs

@@ -36,11 +36,22 @@ internal static partial class OpenApiV2Deserializer
             },
             {
                 "consumes",
-                (o, n) => n.Context.SetTempStorage(TempStorageKeys.GlobalConsumes, n.CreateSimpleList(s => s.GetScalarValue()))
+                (o, n) => {
+                    var consumes = n.CreateSimpleList(s => s.GetScalarValue());
+                    if (consumes.Count > 0)
+                    {
+                        n.Context.SetTempStorage(TempStorageKeys.GlobalConsumes, consumes);
+                    }
+                   }
             },
             {
-                "produces",
-                (o, n) => n.Context.SetTempStorage(TempStorageKeys.GlobalProduces, n.CreateSimpleList(s => s.GetScalarValue()))
+                "produces", (o, n) => {
+                    var produces = n.CreateSimpleList(s => s.GetScalarValue());
+                    if (produces.Count > 0)
+                    {
+                        n.Context.SetTempStorage(TempStorageKeys.GlobalProduces, produces);
+                    }
+                }
             },
             {"paths", (o, n) => o.Paths = LoadPaths(n)},
             {

src/Microsoft.OpenApi.Readers/V2/OpenApiOperationDeserializer.cs

@@ -57,14 +57,20 @@ internal static partial class OpenApiV2Deserializer
                     }
                 },
                 {
-                    "consumes", (o, n) => n.Context.SetTempStorage(
-                        TempStorageKeys.OperationConsumes,
-                        n.CreateSimpleList(s => s.GetScalarValue()))
+                    "consumes", (o, n) => {
+                        var consumes = n.CreateSimpleList(s => s.GetScalarValue());
+                        if (consumes.Count > 0) {
+                        n.Context.SetTempStorage(TempStorageKeys.OperationConsumes,consumes);
+                        }
+                    }
                 },
                 {
-                    "produces", (o, n) => n.Context.SetTempStorage(
-                        TempStorageKeys.OperationProduces,
-                        n.CreateSimpleList(s => s.GetScalarValue()))
+                    "produces", (o, n) => {
+                        var produces = n.CreateSimpleList(s => s.GetScalarValue());
+                        if (produces.Count > 0) {
+                            n.Context.SetTempStorage(TempStorageKeys.OperationProduces, produces);
+                        }
+                    }
                 },
                 {
                     "responses", (o, n) =>