Fix StyleCop warnings

ThreeMammals · Nov 1, 2023 · 8950350 · 8950350
1 parent 2989312
commit 8950350
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/src/Ocelot.Provider.Polly/CircuitBreaker.cs b/src/Ocelot.Provider.Polly/CircuitBreaker.cs
@@ -7,7 +7,7 @@ public class CircuitBreaker<TResult>
     /// Initializes a new instance of the <see cref="CircuitBreaker{TResult}"/> class.
     /// We expect at least one policy to be passed in, default can't be null.
     /// </summary>
-    /// <param name="policies">the policies with at least a timeout policy</param>
+    /// <param name="policies">The policies with at least a <see cref="Policy.Timeout(int)"/> policy.</param>
     public CircuitBreaker(params IAsyncPolicy<TResult>[] policies)
     {
         var allPolicies = policies.Where(p => p != null).ToArray();

diff --git a/test/Ocelot.UnitTests/Polly/PollyCircuitBreakingDelegatingHandlerTests.cs b/test/Ocelot.UnitTests/Polly/PollyCircuitBreakingDelegatingHandlerTests.cs
@@ -54,6 +54,7 @@ public async void SendAsync_OnePolicy_NoWrapping()
             .Returns(_pollyQoSProviderMock.Object);
 
         _contextAccessorMock.Setup(x => x.HttpContext).Returns(httpContext.Object);
+
         // Act
         var actual = await InvokeAsync("SendAsync");
 
@@ -94,7 +95,7 @@ public async void SendAsync_TwoPolicies_HaveWrapped()
         ShouldBeWrappedBy(policy2, typeof(AsyncPolicy).FullName);
     }
 
-    private DownstreamRoute DownstreamRouteFactory()
+    private static DownstreamRoute DownstreamRouteFactory()
     {
         var options = new QoSOptionsBuilder()
             .WithTimeoutValue(100)
@@ -179,7 +180,7 @@ protected override async Task<TResult> ImplementationAsync(Func<Context, Cancell
             return result;
         }
 
-        public IAsyncPolicy WithPolicyKey(string policyKey)
+        public new IAsyncPolicy WithPolicyKey(string policyKey)
         {
             throw new NotImplementedException();
         }