Ensures Paths are copied in the copy constructor (#1008)

* Ensure paths are copied in the copy constructor

* Add test for the copy constructor

* Bump up lib. version

* Check for nullability

* Simplify assignment

* Revert to original

* Add constructor to the abstract class

* Call base constructor

* Update publicApi

Author: irvinesunday

src/Microsoft.OpenApi/Microsoft.OpenApi.csproj

@@ -11,7 +11,7 @@
         <Company>Microsoft</Company>
         <Title>Microsoft.OpenApi</Title>
         <PackageId>Microsoft.OpenApi</PackageId>
-        <Version>1.4.1</Version>
+        <Version>1.4.2</Version>
         <Description>.NET models with JSON and YAML writers for OpenAPI specification</Description>
         <Copyright>© Microsoft Corporation. All rights reserved.</Copyright>
         <PackageTags>OpenAPI .NET</PackageTags>

src/Microsoft.OpenApi/Models/OpenApiDocument.cs

@@ -1,4 +1,4 @@
-// Copyright (c) Microsoft Corporation. All rights reserved.
+// Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT license. 
 
 using System;

src/Microsoft.OpenApi/Models/OpenApiExtensibleDictionary.cs

@@ -2,7 +2,6 @@
 // Licensed under the MIT license. 
 
 using System.Collections.Generic;
-using Microsoft.OpenApi.Any;
 using Microsoft.OpenApi.Interfaces;
 using Microsoft.OpenApi.Writers;
 
@@ -17,6 +16,23 @@ public abstract class OpenApiExtensibleDictionary<T> : Dictionary<string, T>,
         IOpenApiExtensible
         where T : IOpenApiSerializable
     {
+        /// <summary>
+        /// Parameterless constructor
+        /// </summary>
+        protected OpenApiExtensibleDictionary() { }
+
+        /// <summary>
+        /// Initializes a copy of <see cref="OpenApiExtensibleDictionary{T}"/> class.
+        /// </summary>
+        /// <param name="dictionary">The generic dictionary.</param>
+        /// <param name="extensions">The dictionary of <see cref="IOpenApiExtension"/>.</param>
+        protected OpenApiExtensibleDictionary(
+            Dictionary<string, T> dictionary = null,
+            IDictionary<string, IOpenApiExtension> extensions = null) : base (dictionary)
+        {
+            Extensions = extensions != null ? new Dictionary<string, IOpenApiExtension>(extensions) : null;
+        }       
+
         /// <summary>
         /// This object MAY be extended with Specification Extensions.
         /// </summary>

src/Microsoft.OpenApi/Models/OpenApiPaths.cs

@@ -16,7 +16,7 @@ public OpenApiPaths() {}
         /// <summary>
         /// Initializes a copy of <see cref="OpenApiPaths"/> object
         /// </summary>
-        public OpenApiPaths(OpenApiPaths paths) {}
-
+        /// <param name="paths">The <see cref="OpenApiPaths"/>.</param>
+        public OpenApiPaths(OpenApiPaths paths) : base(dictionary: paths) {}
     }
 }

test/Microsoft.OpenApi.Tests/Models/OpenApiDocumentTests.cs

@@ -1342,5 +1342,19 @@ private static OpenApiDocument ParseInputFile(string filePath)
 
             return openApiDoc;
         }
+
+        [Fact]
+        public void CopyConstructorForAdvancedDocumentWorks()
+        {
+            // Arrange & Act
+            var doc = new OpenApiDocument(AdvancedDocument);
+
+            // Assert
+            Assert.NotNull(doc.Info);
+            Assert.NotNull(doc.Servers);
+            Assert.NotNull(doc.Paths);
+            Assert.Equal(2, doc.Paths.Count);
+            Assert.NotNull(doc.Components);
+        }
     }
 }

test/Microsoft.OpenApi.Tests/PublicApi/PublicApi.approved.txt

@@ -581,6 +581,7 @@ namespace Microsoft.OpenApi.Models
         where T : Microsoft.OpenApi.Interfaces.IOpenApiSerializable
     {
         protected OpenApiExtensibleDictionary() { }
+        protected OpenApiExtensibleDictionary(System.Collections.Generic.Dictionary<string, T> dictionary = null, System.Collections.Generic.IDictionary<string, Microsoft.OpenApi.Interfaces.IOpenApiExtension> extensions = null) { }
         public System.Collections.Generic.IDictionary<string, Microsoft.OpenApi.Interfaces.IOpenApiExtension> Extensions { get; set; }
         public void SerializeAsV2(Microsoft.OpenApi.Writers.IOpenApiWriter writer) { }
         public void SerializeAsV3(Microsoft.OpenApi.Writers.IOpenApiWriter writer) { }