[Core] Don't throw from Response.Content if memory stream is private

sdk/core/Azure.Core/CHANGELOG.md

@@ -6,6 +6,10 @@
 
 - Types to represent `GeoJson` primitives.
 
+### Changed
+
+- `Response.Content` no longer throws `InvalidOperationException` when the response is backed by a `MemoryStream` with a non publicly visible buffer.
+
 ## 1.14.0 (2021-05-11)
 
 ### Features Added

sdk/core/Azure.Core/src/Response.cs

@@ -72,7 +72,7 @@ public virtual BinaryData Content
                 }
                 else
                 {
-                    throw new InvalidOperationException($"The {nameof(ContentStream)}'s internal buffer cannot be accessed.");
+                    return new BinaryData(memoryContent.ToArray());
                 }
             }
         }

sdk/core/Azure.Core/tests/ResponseTests.cs

@@ -123,12 +123,15 @@ public void ContentPropertyThrowsForNonMemoryStream()
             Assert.Throws<InvalidOperationException>(() => { BinaryData d = response.Content; });
         }
 
-        public void ContentPropertyThrowsForNotExposableMemoryStream()
+        [Test]
+        public void ContentPropertyWorksForMemoryStreamsWithPrivateBuffers()
         {
             var response = new MockResponse(200);
-            response.ContentStream = new MemoryStream(new byte[100], 0, 100, writable: false, publiclyVisible: false);
+            var responseBody = new byte[100];
+            response.ContentStream = new MemoryStream(responseBody, 0, responseBody.Length, writable: false, publiclyVisible: false);
 
-            Assert.Throws<InvalidOperationException>(() => { BinaryData d = response.Content; });
+            Assert.DoesNotThrow(() => { BinaryData d = response.Content; });
+            CollectionAssert.AreEqual(responseBody, response.Content.ToArray());
         }
 
         internal class TestPayload