Fix HttpStatusCode of exceptions being overwritten

PinSharp.Tests/Api/Exceptions/PinSharpExceptionsTest.cs

@@ -0,0 +1,39 @@
+using FluentAssertions;
+using PinSharp.Api.Exceptions;
+using Xunit;
+
+namespace PinSharp.Tests.Api.Exceptions
+{
+    public class PinSharpExceptionsTest
+    {
+        [Fact]
+        public void Create_HttpStatusCode_ShouldBeSetToPassedValue()
+        {
+            var expectedStatusCode = 500;
+
+            var exception = PinSharpException.Create<MockException>(null, null, null, expectedStatusCode);
+
+            exception.HttpStatusCode.Should().Be(expectedStatusCode);
+        }
+
+        [Fact]
+        public void Create_HttpStatusCode_NullValueShouldNotOverwriteExceptionValue()
+        {
+            var expectedStatusCode = MockException.DefaultStatusCode;
+
+            var exception = PinSharpException.Create<MockException>(null, null, null, null);
+
+            exception.HttpStatusCode.Should().Be(expectedStatusCode);
+        }
+
+        public class MockException : PinSharpException
+        {
+            public const int DefaultStatusCode = 400;
+
+            public MockException(string message) : base(message)
+            {
+                HttpStatusCode = DefaultStatusCode;
+            }
+        }
+    }
+}

PinSharp/Api/Exceptions/PinSharpException.cs

@@ -33,7 +33,10 @@ internal static T Create<T>(string message, string requestUrl, string responseCo
             var exception = (T) Activator.CreateInstance(typeof (T), message);
             exception.RequestUrl = requestUrl;
             exception.ResponseContent = responseContent;
-            exception.HttpStatusCode = httpStatusCode;
+
+            if (httpStatusCode != null)
+                exception.HttpStatusCode = httpStatusCode;
+
             return exception;
         }
     }