Skip to content

Commit

Permalink
Fix formatting
Browse files Browse the repository at this point in the history
  • Loading branch information
tvdias committed Aug 21, 2023
1 parent 387ab1f commit 352f278
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
11 changes: 7 additions & 4 deletions Minio.Tests/HttpRequestMessageBuilderTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -19,23 +19,25 @@
namespace Minio.Tests;

/// <summary>
/// HttpRequestMessageBuilder Tests
/// HttpRequestMessageBuilder Tests
/// </summary>
[TestClass]
public class HttpRequestMessageBuilderTests
{
[TestMethod]
public void RequestGetter_NoQueryParametersAdded_ReturnsUrisQueryParameters()
{
var requestBuilder = new HttpRequestMessageBuilder(HttpMethod.Get, "http://localhost:9000/bucketname/objectname?query1=one&query2=two");
var requestBuilder = new HttpRequestMessageBuilder(HttpMethod.Get,
"http://localhost:9000/bucketname/objectname?query1=one&query2=two");

Assert.AreEqual("?query1=one&query2=two", requestBuilder.Request.RequestUri.Query);
}

[TestMethod]
public void RequestGetter_AditionalQueryParameters_ReturnsAllQueryParameters()
{
var requestBuilder = new HttpRequestMessageBuilder(HttpMethod.Get, "http://localhost:9000/bucketname/objectname?query1=one&query2=two");
var requestBuilder = new HttpRequestMessageBuilder(HttpMethod.Get,
"http://localhost:9000/bucketname/objectname?query1=one&query2=two");
requestBuilder.QueryParameters.Add("query3", "three");

Assert.AreEqual("?query1=one&query2=two&query3=three", requestBuilder.Request.RequestUri.Query);
Expand All @@ -44,7 +46,8 @@ public void RequestGetter_AditionalQueryParameters_ReturnsAllQueryParameters()
[TestMethod]
public void RequestGetter_DuplicatedQueryParameters_ReturnsAllQueryParameters()
{
var requestBuilder = new HttpRequestMessageBuilder(HttpMethod.Get, "http://localhost:9000/bucketname/objectname?query1=one&query2=two");
var requestBuilder = new HttpRequestMessageBuilder(HttpMethod.Get,
"http://localhost:9000/bucketname/objectname?query1=one&query2=two");
requestBuilder.QueryParameters.Add("query2", "new_value");

Assert.AreEqual("?query1=one&query2=two&query2=new_value", requestBuilder.Request.RequestUri.Query);
Expand Down
2 changes: 0 additions & 2 deletions Minio/HttpRequestMessageBuilder.cs
Original file line number Diff line number Diff line change
Expand Up @@ -66,9 +66,7 @@ public HttpRequestMessage Request
{
var query = HttpUtility.ParseQueryString(requestUriBuilder.Query);
foreach (var queryParameter in QueryParameters)
{
query.Add(HttpUtility.UrlEncode(queryParameter.Key), HttpUtility.UrlEncode(queryParameter.Value));
}
requestUriBuilder.Query = query.ToString();
}

Expand Down

0 comments on commit 352f278

Please sign in to comment.