Merge pull request #1022 from microsoft/mk/copy-responses-property

Copy OpenApiResponses object from source OpenApiOperation
microsoft · Sep 26, 2022 · e81a1c6 · e81a1c6
2 parents b645cb5 + 194ef81
commit e81a1c6
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 3 deletions.
diff --git a/src/Microsoft.OpenApi/Microsoft.OpenApi.csproj b/src/Microsoft.OpenApi/Microsoft.OpenApi.csproj
@@ -11,7 +11,7 @@
         <Company>Microsoft</Company>
         <Title>Microsoft.OpenApi</Title>
         <PackageId>Microsoft.OpenApi</PackageId>
-        <Version>1.4.2</Version>
+        <Version>1.4.3</Version>
         <Description>.NET models with JSON and YAML writers for OpenAPI specification</Description>
         <Copyright>© Microsoft Corporation. All rights reserved.</Copyright>
         <PackageTags>OpenAPI .NET</PackageTags>

diff --git a/src/Microsoft.OpenApi/Models/OpenApiResponses.cs b/src/Microsoft.OpenApi/Models/OpenApiResponses.cs
@@ -16,7 +16,7 @@ public OpenApiResponses() { }
         /// <summary>
         /// Initializes a copy of <see cref="OpenApiResponses"/> object
         /// </summary>
-        public OpenApiResponses(OpenApiResponses openApiResponses) { }
-
+        /// <param name="openApiResponses">The <see cref="OpenApiResponses"/></param>
+        public OpenApiResponses(OpenApiResponses openApiResponses) : base(dictionary: openApiResponses) {}
     }
 }
diff --git a/test/Microsoft.OpenApi.Tests/Models/OpenApiOperationTests.cs b/test/Microsoft.OpenApi.Tests/Models/OpenApiOperationTests.cs
@@ -787,5 +787,16 @@ public void SerializeOperationWithNullCollectionAsV2JsonWorks()
             expected = expected.MakeLineBreaksEnvironmentNeutral();
             actual.Should().Be(expected);
         }
+
+        [Fact]
+        public void EnsureOpenApiOperationCopyConstructorCopiesResponsesObject()
+        {
+            // Arrange and act
+            var operation = new OpenApiOperation(_operationWithBody);
+
+            // Assert
+            Assert.NotNull(operation.Responses);
+            Assert.Equal(2, operation.Responses.Count);
+        }
     }
 }