fix(PollyTests): bug fix for the test IsBackOffFalse and small correction.

Author: Ying Feifan

src/Liquid.Runtime/Polly/Polly.cs

@@ -17,20 +17,18 @@ public override async Task<HttpResponseMessage> ResilientRequest(string url, Htt
             PollyConfiguration pollyConfig = (PollyConfiguration)pollyconfiguration;
             AsyncRetryPolicy<HttpResponseMessage> retryPolicy = null;
 
-            // TODO: Please let's remove this ternary
             if (pollyConfig.IsBackOff == true)
             {
-                retryPolicy = Policy.HandleResult<HttpResponseMessage>(r => r.StatusCode.Equals(HttpStatusCode.InternalServerError))
-                .Or<WebException>().Or<HttpRequestException>()
+                retryPolicy = Policy.HandleResult<HttpResponseMessage>(r => r.StatusCode.Equals(HttpStatusCode.InternalServerError)).Or<WebException>().Or<HttpRequestException>()
                     .WaitAndRetryAsync(pollyConfig.Retry, retryAttempt => TimeSpan.FromSeconds(Math.Pow(2, retryAttempt)), (result, timeSpan, retryCount, context) =>
                     {
                         CallbackError<HttpResponseMessage>(result, timeSpan, retryCount);
                     });
             }
             else
             {
-                Policy.HandleResult<HttpResponseMessage>(r => r.StatusCode.Equals(HttpStatusCode.InternalServerError)).Or<WebException>().Or<HttpRequestException>()
-                .WaitAndRetryAsync(pollyConfig.Retry, retryAttempt => TimeSpan.FromSeconds(pollyConfig.Wait), (result, timeSpan, retryCount, context) =>
+                retryPolicy = Policy.HandleResult<HttpResponseMessage>(r => r.StatusCode.Equals(HttpStatusCode.InternalServerError)).Or<WebException>().Or<HttpRequestException>()
+                    .WaitAndRetryAsync(pollyConfig.Retry, retryAttempt => TimeSpan.FromSeconds(pollyConfig.Wait), (result, timeSpan, retryCount, context) =>
                     {
                         CallbackError<HttpResponseMessage>(result, timeSpan, retryCount);
                     });

test/Liquid.Runtime.Tests/PollyTest.cs

@@ -0,0 +1,88 @@
+using System.Net.Http;
+using System.Threading.Tasks;
+using Liquid.Runtime.Polly;
+using Xunit;
+
+
+
+
+
+namespace Liquid.Runtime.Tests
+{
+    public class PollyTests
+    {
+        private readonly LightPolly _sut;
+        private HttpClient _httpClient;
+        private WorkBenchServiceHttp _http;
+
+
+
+
+        public PollyTests()
+        {
+            _sut = new Polly.Polly();
+            _httpClient = new HttpClient();
+            _http = WorkBenchServiceHttp.GET;
+        }
+
+
+
+
+        [Fact]
+        public async Task ResilientRequestWhenInvalidUrlAfterRetriedConfiguredAttempsThrowHttpRequest()
+        {
+            //Arrange
+            PollyConfiguration _pollyConfiguration = new PollyConfiguration { IsBackOff = true, Retry = 3, Wait = 5 };
+            var url = "https://www.aipdhapsidh.com";
+            HttpResponseMessage result;
+
+
+
+            //Action/Assert
+            await Assert.ThrowsAnyAsync<HttpRequestException>(async () => result = await _sut.ResilientRequest(url, _httpClient, _http, _pollyConfiguration).ConfigureAwait(true));
+        }
+
+
+
+
+        [Fact]
+        public async Task ResilientRequestShouldUseMathPowAsRetryAttemptWhenIsBackOffTrue()
+        {
+
+
+
+            //Arrange
+            PollyConfiguration _pollyConfiguration = new PollyConfiguration { IsBackOff = true, Retry = 3, Wait = 5 };
+            var url = "https://www.google.com";
+
+
+
+            //Action
+            using (HttpResponseMessage result = await _sut.ResilientRequest(url, _httpClient, _http, _pollyConfiguration).ConfigureAwait(true))
+            {
+                //Assert
+                Assert.True(result.IsSuccessStatusCode);
+            }
+        }
+
+
+
+
+        [Fact]
+        public async Task ResilientRequestShouldUseDefaultPollyConfigAsRetryAttemptWhenIsBackOffFalse()
+        {
+
+
+
+            //Arrange
+            PollyConfiguration _pollyConfiguration = new PollyConfiguration { IsBackOff = false, Retry = 3, Wait = 5 };
+            var url = "https://www.google.com";
+            //Action
+            using (HttpResponseMessage result = await _sut.ResilientRequest(url, _httpClient, _http, _pollyConfiguration).ConfigureAwait(false))
+            {
+                //Assert
+                Assert.True(result.IsSuccessStatusCode);
+            }
+        }
+    }
+}