PR comments and attempt to fix code coverage

dotnet · Jun 27, 2023 · cd35a19 · cd35a19
1 parent c6e88aa
commit cd35a19
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/...FaultInjection/PolicyContextExtensions.cs → ...ience/FaultInjection/ContextExtensions.cs b/...FaultInjection/PolicyContextExtensions.cs → ...ience/FaultInjection/ContextExtensions.cs
@@ -12,7 +12,7 @@ namespace Microsoft.Extensions.Http.Resilience.FaultInjection;
 /// Provides extension methods for <see cref="Context"/>.
 /// </summary>
 [Experimental]
-public static class PolicyContextExtensions
+public static class ContextExtensions
 {
     private const string CallingRequestMessage = "CallingRequestMessage";
 

diff --git a/...Microsoft.Extensions.Http.Resilience/Resilience/HttpClientBuilderExtensions.Resilience.cs b/...Microsoft.Extensions.Http.Resilience/Resilience/HttpClientBuilderExtensions.Resilience.cs
@@ -37,11 +37,9 @@ public static IHttpResilienceStrategyBuilder AddResilienceHandler(
         _ = Throw.IfNullOrEmpty(strategyName);
         _ = Throw.IfNull(configure);
 
-        var resilienceBuilder = builder.AddResilienceHandler(
-            strategyName,
-            (builder, _) => configure(builder));
+        return builder.AddResilienceHandler(strategyName, ConfigureBuilder);
 
-        return resilienceBuilder;
+        void ConfigureBuilder(ResilienceStrategyBuilder<HttpResponseMessage> builder, ResilienceHandlerContext context) => configure(builder);
     }
 
     /// <summary>

diff --git a/...tInjection/PolicyContextExtensionsTest.cs → ...s/FaultInjection/ContextExtensionsTest.cs b/...tInjection/PolicyContextExtensionsTest.cs → ...s/FaultInjection/ContextExtensionsTest.cs
@@ -8,7 +8,7 @@
 
 namespace Microsoft.Extensions.Http.Resilience.FaultInjection.Test;
 
-public class PolicyContextExtensionsTest
+public class ContextExtensionsTest
 {
     [Fact]
     public void WithCallingRequestMessage_ShouldSetRequestMessageInContext()