Allow empty uri in RestRequest.

spoleto-software · Jun 26, 2024 · 55b1787 · 55b1787
1 parent c327ace
commit 55b1787
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 5 deletions.
diff --git a/src/Spoleto.RestClient/RestRequest/RestRequest.cs b/src/Spoleto.RestClient/RestRequest/RestRequest.cs
@@ -8,13 +8,21 @@ public record RestRequest : IRestRequest
 
         private readonly Dictionary<string, string> _headers = [];
 
+        public RestRequest(RestHttpMethod httpMethod)
+            : this(httpMethod, string.Empty, null)
+        {
+        }
+
         public RestRequest(RestHttpMethod httpMethod, string uri)
             : this(httpMethod, uri, null)
         {
         }
 
         public RestRequest(RestHttpMethod httpMethod, string uri, HttpContent? content = null)
         {
+            if (uri == null)
+                throw new ArgumentNullException(nameof(uri));
+
             Method = httpMethod;
             Uri = uri;
             _content = content;

diff --git a/src/Spoleto.RestClient/RestRequest/RestRequestFactory.cs b/src/Spoleto.RestClient/RestRequest/RestRequestFactory.cs
@@ -17,11 +17,22 @@ public class RestRequestFactory
         private HttpContent? _content;
         private readonly Dictionary<string, string> _headers = [];
 
+        /// <summary>
+        /// Constructor with parameter.
+        /// </summary>
+        public RestRequestFactory(RestHttpMethod method)
+            : this(method, string.Empty)
+        {
+        }
+
         /// <summary>
         /// Constructor with parameters.
         /// </summary>
         public RestRequestFactory(RestHttpMethod method, string requestUri)
         {
+            if (requestUri == null)
+                throw new ArgumentNullException(nameof(requestUri), "Request URI is required");
+
             _method = method;
             _requestUri = requestUri;
         }
@@ -180,11 +191,6 @@ public RestRequestFactory WithBearerToken(string token)
 
         public RestRequest Build()
         {
-            if (string.IsNullOrEmpty(_requestUri))
-            {
-                throw new InvalidOperationException("Request URI is required");
-            }
-
             var uri = _requestUri;
             if (!string.IsNullOrEmpty(_query))
             {