bug: stop removing trailing single chars from query string

Closes #2044

Author: baynezy

Refit.Tests/RequestBuilder.cs

@@ -1770,6 +1770,9 @@ Task<string> FetchSomeStuffWithHardcodedAndOtherQueryParameters(
         [Get("/foo/bar?param=first {id} and second {id}")]
         Task<string> FetchSomeStuffWithTheIdInAParameterMultipleTimes(int id);
 
+        [Get("/foo?q=app_metadata.id:\"{id}\"")]
+        Task<string> FetchSomeStuffWithDoubleQuotesInUrl(int id);
+
         [Post("/foo/bar/{id}")]
         [Headers("Content-Type: literally/anything")]
         Task<string> PostSomeStuffWithHardCodedContentTypeHeader(int id, [Body] string content);
@@ -2594,6 +2597,20 @@ public void QueryParamWithPathDelimiterShouldBeEncoded()
             );
         }
 
+        [Fact]
+        public void QueryParamWhichEndsInDoubleQuotesShouldNotBeTruncated()
+        {
+            var fixture = new RequestBuilderImplementation<IDummyHttpApi>();
+            var factory = fixture.BuildRequestFactoryForMethod(
+                "FetchSomeStuffWithDoubleQuotesInUrl"
+            );
+            var output = factory(new object[] { 42 });
+
+            var uri = new Uri(new Uri("http://api"), output.RequestUri);
+
+            Assert.Equal("/foo?q=app_metadata.id%3A%2242%22", uri.PathAndQuery);
+        }
+
         [Fact]
         public void ParameterizedQueryParamsShouldBeInUrlAndValuesEncodedWhenMixedReplacementAndQueryBadId()
         {

Refit/RestMethodInfo.cs

@@ -402,7 +402,7 @@ ParameterInfo[] parameterInfo
             }
 
             // add trailing string
-            if (index < relativePath.Length - 1)
+            if (index < relativePath.Length)
             {
                 var trailingConstant = relativePath.Substring(index, relativePath.Length - index);
                 fragmentList.Add(ParameterFragment.Constant(trailingConstant));