Fix CA1032 warnings (#2194)

Resolve CA1032 warnings.

src/Polly/Polly.csproj

@@ -7,7 +7,7 @@
     <ProjectType>Library</ProjectType>
     <MutationScore>70</MutationScore>
     <IncludePollyUsings>true</IncludePollyUsings>
-    <NoWarn>$(NoWarn);CA1010;CA1031;CA1032;CA1051;CA1062;CA1063;CA1064;CA1710;CA1716;CA1724;CA1805;CA1815;CA1816;CA2211</NoWarn>
+    <NoWarn>$(NoWarn);CA1010;CA1031;CA1051;CA1062;CA1063;CA1064;CA1710;CA1716;CA1724;CA1805;CA1815;CA1816;CA2211</NoWarn>
     <NoWarn>$(NoWarn);S2223;S3215;S3246;S3971;S4039;S4457</NoWarn>
     <!--Public API Analyzers: We do not need to fix these as it would break compatibility with released Polly versions-->
     <NoWarn>$(NoWarn);RS0037;</NoWarn>

src/Polly/RateLimit/RateLimitRejectedException.cs

@@ -21,6 +21,33 @@ public class RateLimitRejectedException : ExecutionRejectedException
     /// </summary>
     public TimeSpan RetryAfter { get; private set; }
 
+    /// <summary>
+    /// Initializes a new instance of the <see cref="RateLimitRejectedException"/> class.
+    /// </summary>
+    public RateLimitRejectedException()
+        : base("The operation could not be executed because it was rejected by the rate limit.")
+    {
+    }
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="RateLimitRejectedException"/> class.
+    /// </summary>
+    /// <param name="message">The message that describes the error.</param>
+    public RateLimitRejectedException(string message)
+        : base(message)
+    {
+    }
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="RateLimitRejectedException"/> class.
+    /// </summary>
+    /// <param name="message">The message that describes the error.</param>
+    /// <param name="inner">The inner exception.</param>
+    public RateLimitRejectedException(string message, Exception inner)
+        : base(message, inner)
+    {
+    }
+
     /// <summary>
     /// Initializes a new instance of the <see cref="RateLimitRejectedException"/> class.
     /// </summary>

src/Polly/Utilities/TimedLock.cs

@@ -85,8 +85,30 @@ private class Sentinel
 
 internal class LockTimeoutException : Exception
 {
+    /// <summary>
+    /// Initializes a new instance of the <see cref="LockTimeoutException"/> class.
+    /// </summary>
     public LockTimeoutException()
         : base("Timeout waiting for lock")
     {
     }
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="LockTimeoutException"/> class.
+    /// </summary>
+    /// <param name="message">The message that describes the error.</param>
+    public LockTimeoutException(string message)
+        : base(message)
+    {
+    }
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="LockTimeoutException"/> class.
+    /// </summary>
+    /// <param name="message">The message that describes the error.</param>
+    /// <param name="innerException">The inner exception.</param>
+    public LockTimeoutException(string message, Exception innerException)
+        : base(message, innerException)
+    {
+    }
 }

test/Polly.Specs/RateLimit/RateLimitRejectedExceptionTests.cs

@@ -0,0 +1,35 @@
+namespace Polly.Specs.RateLimit;
+
+public class RateLimitRejectedExceptionTests
+{
+    [Fact]
+    public void Ctor_Ok()
+    {
+        const string Dummy = "dummy";
+        var exception = new InvalidOperationException();
+        var retryAfter = TimeSpan.FromSeconds(4);
+
+        new RateLimitRejectedException().Message.Should().Be("The operation could not be executed because it was rejected by the rate limit.");
+        new RateLimitRejectedException(Dummy).Message.Should().Be(Dummy);
+
+        var rate = new RateLimitRejectedException(Dummy, exception);
+        rate.Message.Should().Be(Dummy);
+        rate.InnerException.Should().Be(exception);
+
+        new RateLimitRejectedException(retryAfter).RetryAfter.Should().Be(retryAfter);
+        new RateLimitRejectedException(retryAfter).Message.Should().Be($"The operation has been rate-limited and should be retried after {retryAfter}");
+
+        rate = new RateLimitRejectedException(retryAfter, exception);
+        rate.RetryAfter.Should().Be(retryAfter);
+        rate.InnerException.Should().Be(exception);
+
+        rate = new RateLimitRejectedException(retryAfter, Dummy);
+        rate.RetryAfter.Should().Be(retryAfter);
+        rate.Message.Should().Be(Dummy);
+
+        rate = new RateLimitRejectedException(retryAfter, Dummy, exception);
+        rate.RetryAfter.Should().Be(retryAfter);
+        rate.Message.Should().Be(Dummy);
+        rate.InnerException.Should().Be(exception);
+    }
+}

test/Polly.Specs/Utilities/LockTimeoutExceptionTests.cs

@@ -0,0 +1,18 @@
+namespace Polly.Specs.Utilities;
+
+public class LockTimeoutExceptionTests
+{
+    [Fact]
+    public void Ctor_Ok()
+    {
+        const string Dummy = "dummy";
+        var exception = new InvalidOperationException();
+
+        new LockTimeoutException().Message.Should().Be("Timeout waiting for lock");
+        new LockTimeoutException(Dummy).Message.Should().Be(Dummy);
+
+        var rate = new LockTimeoutException(Dummy, exception);
+        rate.Message.Should().Be(Dummy);
+        rate.InnerException.Should().Be(exception);
+    }
+}