Trim trailing / from hostUrl

reactiveui · Feb 17, 2019 · 69bea73 · 69bea73
1 parent d835e88
commit 69bea73
Show file tree

Hide file tree

Showing 5 changed files with 85 additions and 2 deletions.
diff --git a/Refit.Tests/InterfaceStubGenerator.cs b/Refit.Tests/InterfaceStubGenerator.cs
@@ -123,7 +123,7 @@ public void GenerateTemplateInfoForInterfaceListSmokeTest()
                 .ToList();
 
             var result = fixture.GenerateTemplateInfoForInterfaceList(input);
-            Assert.Equal(10, result.ClassList.Count);
+            Assert.Equal(11, result.ClassList.Count);
         }
 
         [Fact]

diff --git a/Refit.Tests/RefitStubs.Net46.cs b/Refit.Tests/RefitStubs.Net46.cs
@@ -1083,6 +1083,38 @@ public virtual Task<ApiResponse<Stream>> GetRemoteFileWithMetadata(string filena
     }
 }
 
+namespace Refit.Tests
+{
+    using Refit.Tests.RefitInternalGenerated;
+
+    /// <inheritdoc />
+    [global::System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
+    [global::System.Diagnostics.DebuggerNonUserCode]
+    [Preserve]
+    [global::System.Reflection.Obfuscation(Exclude=true)]
+    partial class AutoGeneratedITrimTrailingForwardSlashApi : ITrimTrailingForwardSlashApi
+    {
+        /// <inheritdoc />
+        public HttpClient Client { get; protected set; }
+        readonly IRequestBuilder requestBuilder;
+
+        /// <inheritdoc />
+        public AutoGeneratedITrimTrailingForwardSlashApi(HttpClient client, IRequestBuilder requestBuilder)
+        {
+            Client = client;
+            this.requestBuilder = requestBuilder;
+        }
+
+        /// <inheritdoc />
+        public virtual Task Get()
+        {
+            var arguments = new object[] {  };
+            var func = requestBuilder.BuildRestResultFuncForMethod("Get", new Type[] {  });
+            return (Task)func(Client, arguments);
+        }
+    }
+}
+
 namespace Refit.Tests
 {
     using Refit.Tests.RefitInternalGenerated;

diff --git a/Refit.Tests/RefitStubs.NetCore2.cs b/Refit.Tests/RefitStubs.NetCore2.cs
@@ -1083,6 +1083,38 @@ public virtual Task<ApiResponse<Stream>> GetRemoteFileWithMetadata(string filena
     }
 }
 
+namespace Refit.Tests
+{
+    using Refit.Tests.RefitInternalGenerated;
+
+    /// <inheritdoc />
+    [global::System.Diagnostics.CodeAnalysis.ExcludeFromCodeCoverage]
+    [global::System.Diagnostics.DebuggerNonUserCode]
+    [Preserve]
+    [global::System.Reflection.Obfuscation(Exclude=true)]
+    partial class AutoGeneratedITrimTrailingForwardSlashApi : ITrimTrailingForwardSlashApi
+    {
+        /// <inheritdoc />
+        public HttpClient Client { get; protected set; }
+        readonly IRequestBuilder requestBuilder;
+
+        /// <inheritdoc />
+        public AutoGeneratedITrimTrailingForwardSlashApi(HttpClient client, IRequestBuilder requestBuilder)
+        {
+            Client = client;
+            this.requestBuilder = requestBuilder;
+        }
+
+        /// <inheritdoc />
+        public virtual Task Get()
+        {
+            var arguments = new object[] {  };
+            var func = requestBuilder.BuildRestResultFuncForMethod("Get", new Type[] {  });
+            return (Task)func(Client, arguments);
+        }
+    }
+}
+
 namespace Refit.Tests
 {
     using Refit.Tests.RefitInternalGenerated;

diff --git a/Refit.Tests/RestService.cs b/Refit.Tests/RestService.cs
@@ -131,6 +131,14 @@ public interface IBodylessApi
         Task Head();
     }
 
+    public interface ITrimTrailingForwardSlashApi
+    {
+        HttpClient Client { get; }
+
+        [Get("/someendpoint")]
+        Task Get();
+    }
+
     public class HttpBinGet
     {
         public Dictionary<string, object> Args { get; set; }
@@ -1187,5 +1195,16 @@ public async Task CanSerializeContentAsXml()
 
             mockHttp.VerifyNoOutstandingExpectation();
         }
+
+        [Fact]
+        public void ShouldTrimTrailingForwardSlashFromBaseUrl()
+        {
+            var expectedBaseAddress = "http://example.com/api";
+            var inputBaseAddress = "http://example.com/api/";
+
+            var fixture = RestService.For<ITrimTrailingForwardSlashApi>(inputBaseAddress);
+
+            Assert.Equal(fixture.Client.BaseAddress.AbsoluteUri, expectedBaseAddress);
+        }
     }
 }
diff --git a/Refit/RestService.cs b/Refit/RestService.cs
@@ -49,7 +49,7 @@ public static T For<T>(string hostUrl, RefitSettings settings)
                 }
             }
 
-            var client = new HttpClient(innerHandler ?? new HttpClientHandler()) { BaseAddress = new Uri(hostUrl) };
+            var client = new HttpClient(innerHandler ?? new HttpClientHandler()) { BaseAddress = new Uri(hostUrl.TrimEnd('/')) };
             return For<T>(client, settings);
         }