Merge branch 'dm/empty-collections' of https://github.com/Microsoft/OpenAPI.NET into dm/empty-collections

Author: darrelmiller

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

@@ -19,12 +19,7 @@ internal static partial class OpenApiV2Deserializer
     {
         public static FixedFieldMap<OpenApiDocument> OpenApiFixedFields = new FixedFieldMap<OpenApiDocument>
         {
-            {
-                "swagger", (o, n) =>
-                {
-                    o.SpecVersion = new Version(2, 0);
-                }
-            },
+            {"swagger", (o, n) => { } /* Version is valid field but we already parsed it */  },
             {"info", (o, n) => o.Info = LoadInfo(n)},
             {"host", (o, n) => n.Context.SetTempStorage("host", n.GetScalarValue())},
             {"basePath", (o, n) => n.Context.SetTempStorage("basePath", n.GetScalarValue())},

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

@@ -19,12 +19,7 @@ internal static partial class OpenApiV3Deserializer
     {
         public static FixedFieldMap<OpenApiDocument> OpenApiFixedFields = new FixedFieldMap<OpenApiDocument>
         {
-            {
-                "openapi", (o, n) =>
-                {
-                    o.SpecVersion = new Version(n.GetScalarValue());
-                }
-            },
+            {"openapi", (o, n) => { } /* Version is valid field but we already parsed it */  },
             {"info", (o, n) => o.Info = LoadInfo(n)},
             {"servers", (o, n) => o.Servers = n.CreateList(LoadServer)},
             {"paths", (o, n) => o.Paths = LoadPaths(n)},

src/Microsoft.OpenApi/Models/OpenApiDocument.cs

@@ -17,11 +17,6 @@ namespace Microsoft.OpenApi.Models
     /// </summary>
     public class OpenApiDocument : IOpenApiSerializable, IOpenApiExtensible
     {
-        /// <summary>
-        /// REQUIRED. This string MUST be the semantic version number of the OpenAPI Specification version 
-        /// that the OpenAPI document uses.
-        /// </summary>
-        public Version SpecVersion { get; set; }
 
         /// <summary>
         /// REQUIRED. Provides metadata about the API. The metadata MAY be used by tooling as required.
@@ -64,7 +59,7 @@ public class OpenApiDocument : IOpenApiSerializable, IOpenApiExtensible
         public IDictionary<string, IOpenApiAny> Extensions { get; set; } = new Dictionary<string, IOpenApiAny>();
 
         /// <summary>
-        /// Serialize <see cref="OpenApiDocument"/> to Open Api v3.0.
+        /// Serialize <see cref="OpenApiDocument"/> to Open Api v3.0.0
         /// </summary>
         public void SerializeAsV3(IOpenApiWriter writer)
         {
@@ -76,7 +71,7 @@ public void SerializeAsV3(IOpenApiWriter writer)
             writer.WriteStartObject();
 
             // openapi
-            writer.WriteProperty(OpenApiConstants.OpenApi, SpecVersion?.ToString());
+            writer.WriteProperty(OpenApiConstants.OpenApi, "3.0.0");
 
             // info
             writer.WriteRequiredObject(OpenApiConstants.Info, Info, (w, i) => i.SerializeAsV3(w));
@@ -118,7 +113,7 @@ public void SerializeAsV2(IOpenApiWriter writer)
             writer.WriteStartObject();
 
             // swagger
-            writer.WriteProperty(OpenApiConstants.Swagger, SpecVersion?.ToString());
+            writer.WriteProperty(OpenApiConstants.Swagger, "2.0");
 
             // info
             writer.WriteRequiredObject(OpenApiConstants.Info, Info, (w, i) => i.SerializeAsV2(w));

test/Microsoft.OpenApi.Readers.Tests/V2Tests/ComparisonTests.cs

@@ -30,9 +30,7 @@ public void EquivalentV2AndV3DocumentsShouldProductEquivalentObjects(string file
                 // Everything in the DOM read from V2 and V3 documents should be equal
                 // except the SpecVersion property (2.0 and 3.0.0)
                 openApiDocV3.ShouldBeEquivalentTo(
-                    openApiDocV2,
-                    options => options.Excluding(
-                        s => s.SelectedMemberPath == nameof(OpenApiDocument.SpecVersion)));
+                    openApiDocV2);
 
                 contextV2.ShouldBeEquivalentTo(contextV3);
             }

test/Microsoft.OpenApi.Readers.Tests/V3Tests/OpenApiDocumentTests.cs

@@ -40,7 +40,6 @@ public void ParseDocumentFromInlineStringShouldSucceed()
 
             openApiDoc.ShouldBeEquivalentTo(new OpenApiDocument()
             {
-                SpecVersion = new Version(3, 0, 0),
                 Info = new OpenApiInfo
                 {
 
@@ -66,7 +65,6 @@ public void ParseBasicDocumentWithMultipleServersShouldSucceed()
 
                 openApiDoc.ShouldBeEquivalentTo(new OpenApiDocument()
                 {
-                    SpecVersion = new Version(3,0,0),
                     Info = new OpenApiInfo()
                     {
                         Title = "The API",
@@ -98,7 +96,6 @@ public void ParseBrokenMinimalDocumentShouldYieldExpectedDiagnostic()
 
                 openApiDoc.ShouldBeEquivalentTo(new OpenApiDocument()
                 {
-                    SpecVersion = new Version(3, 0, 0),
                     Info = new OpenApiInfo
                     {
                         Version = "0.9"
@@ -124,7 +121,6 @@ public void ParseMinimalDocumentShouldSucceed()
 
                 openApiDoc.ShouldBeEquivalentTo(new OpenApiDocument()
                 {
-                    SpecVersion = new Version(3, 0, 0),
                     Info = new OpenApiInfo
                     {
 
@@ -253,7 +249,6 @@ public void ParseStandardPetStoreDocumentShouldSucceed()
 
                 var expected = new OpenApiDocument()
                 {
-                    SpecVersion = new Version(3, 0, 0),
                     Info = new OpenApiInfo()
                     {
                         Version = "1.0.0",
@@ -711,7 +706,6 @@ public void ParseModifiedPetStoreDocumentWithTagAndSecurityShouldSucceed()
 
                 var expected = new OpenApiDocument()
                 {
-                    SpecVersion = new Version(3, 0, 0),
                     Info = new OpenApiInfo()
                     {
                         Version = "1.0.0",