Update properties to be virtual on DefaultOpenApiConfigurationOptions (…

…Azure#392)
Derich367 · Jan 30, 2023 · 8d9003e · 8d9003e
1 parent 00df753
commit 8d9003e
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 3 deletions.
diff --git a/...zure.WebJobs.Extensions.OpenApi.Core/Configurations/DefaultOpenApiConfigurationOptions.cs b/...zure.WebJobs.Extensions.OpenApi.Core/Configurations/DefaultOpenApiConfigurationOptions.cs
@@ -39,13 +39,13 @@ public class DefaultOpenApiConfigurationOptions : IOpenApiConfigurationOptions
         public virtual bool IncludeRequestingHostName { get; set; } = IsFunctionsRuntimeEnvironmentDevelopment();
 
         /// <inheritdoc />
-        public bool ForceHttp { get; set; } = IsHttpForced();
+        public virtual bool ForceHttp { get; set; } = IsHttpForced();
 
         /// <inheritdoc />
-        public bool ForceHttps { get; set; } = IsHttpsForced();
+        public virtual bool ForceHttps { get; set; } = IsHttpsForced();
 
         /// <inheritdoc />
-        public List<IDocumentFilter> DocumentFilters { get; set; } = new List<IDocumentFilter>();
+        public virtual List<IDocumentFilter> DocumentFilters { get; set; } = new List<IDocumentFilter>();
 
         /// <summary>
         /// Gets the OpenAPI document version.

diff --git a/...s.Extensions.OpenApi.Core.Tests/Configurations/DefaultOpenApiConfigurationOptionsTests.cs b/...s.Extensions.OpenApi.Core.Tests/Configurations/DefaultOpenApiConfigurationOptionsTests.cs
@@ -223,5 +223,13 @@ public void Given_EnvironmentVariable_When_IsHttpsForced_Invoked_Then_It_Should_
             result.Should().BeOfType<bool>();
             ((bool)result).Should().Be(expected);
         }
+
+        public void Given_Type_When_Instantiated_Then_It_Should_Return_EmptyListOfDocumentFilters()
+        {
+            var options = new DefaultOpenApiConfigurationOptions();
+
+            options.DocumentFilters.Should().NotBeNull();
+            options.DocumentFilters.Should().BeEmpty();
+        }
     }
 }