Expose request URI in exception in case of failed requests

reportportal · Jul 25, 2018 · 6de087e · 6de087e
1 parent fc8e0be
commit 6de087e
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/ReportPortal.Client.Tests/LaunchItem/LaunchItem.cs b/ReportPortal.Client.Tests/LaunchItem/LaunchItem.cs
@@ -7,12 +7,12 @@
 using ReportPortal.Client.Requests;
 using System.Threading.Tasks;
 using System.Net.Http;
-using Xunit;  
+using Xunit;
 
 namespace ReportPortal.Client.Tests.LaunchItem
-{ 
+{
     public class LaunchItemFixture : BaseFixture
-    { 
+    {
         [Fact]
         public async Task GetInvalidLaunch()
         {
@@ -29,7 +29,7 @@ public async Task GetLaunches()
 
         [Fact]
         public async Task GetDebugLaunches()
-        { 
+        {
             var launches = await Service.GetLaunchesAsync(debug: true);
             launches.Launches.ForEach((l) => Assert.Equal(LaunchMode.Debug, l.Mode));
         }

diff --git a/ReportPortal.Client/Extentions/HttpResponseMessageExtension.cs b/ReportPortal.Client/Extentions/HttpResponseMessageExtension.cs
@@ -7,6 +7,7 @@ static class HttpResponseMessageExtension
     {
         public static void VerifySuccessStatusCode(this HttpResponseMessage httpResponseMessage)
         {
+            var requestUri = httpResponseMessage.RequestMessage.RequestUri;
             var body = httpResponseMessage.Content.ReadAsStringAsync().Result;
 
             try
@@ -15,7 +16,7 @@ public static void VerifySuccessStatusCode(this HttpResponseMessage httpResponse
             }
             catch(Exception exp)
             {
-                throw new HttpRequestException(body, exp);
+                throw new HttpRequestException($"Unexpected response status code. Request URI: {requestUri}{Environment.NewLine}Response Body: {body}", exp);
             }
         }
     }