diff --git a/APIMatic.Core.Test/APIMatic.Core.Test.csproj b/APIMatic.Core.Test/APIMatic.Core.Test.csproj index f72cd836..6f35c45a 100644 --- a/APIMatic.Core.Test/APIMatic.Core.Test.csproj +++ b/APIMatic.Core.Test/APIMatic.Core.Test.csproj @@ -19,7 +19,7 @@ - + diff --git a/APIMatic.Core.Test/TypesTest.cs b/APIMatic.Core.Test/TypesTest.cs index 83a48ea1..5ea4bd51 100644 --- a/APIMatic.Core.Test/TypesTest.cs +++ b/APIMatic.Core.Test/TypesTest.cs @@ -62,34 +62,6 @@ public void ApiException_CheckResponseCode_WithResponseOfEmptyStream() Assert.AreEqual(200, exception.ResponseCode); } - [Test] - public void HttpContext_String_Representation() - { - var response = new HttpResponse(200, new Dictionary(), new MemoryStream(Encoding.UTF8.GetBytes("")), "Test body"); - var request = new HttpRequest(HttpMethod.Get, "https://myurl.com"); - - request.AddHeaders(new Dictionary - { - { "keyA1", "value A1"} - }); - request.AddHeaders(new Dictionary - { - { "keyA2", "value A2"} - }); - request.AddQueryParameters(new Dictionary - { - { "queryA1", "value A1"}, - }); - request.AddQueryParameters(new Dictionary - { - { "queryA2", "value A2"} - }); - var context = new HttpContext(request, response); - - var expected = " Request = HttpMethod = GET, QueryUrl = https://myurl.com, QueryParameters = {\"queryA1\":\"value A1\",\"queryA2\":\"value A2\"}, Headers = {\"keyA1\":\"value A1\",\"keyA2\":\"value A2\"}, FormParameters = , Body = , Username = , Password = , Response = StatusCode = 200, Headers = {} RawBody = System.IO.MemoryStream"; - Assert.AreEqual(expected, context.ToString()); - } - [Test] public void JsonObject_String_Representation() { @@ -124,5 +96,45 @@ public void JsonValue_String_Representation() var actualDeserialized = CoreHelper.JsonDeserialize(expectedString); Assert.AreEqual(jsonValue.ToString(), actualDeserialized.ToString()); } + + [Test] + public void AddHeaders_ShouldAddHeaders_WhenHeadersNotNull() + { + // Arrange + var coreRequest = new HttpRequest(HttpMethod.Get, "https://myurl.com"); + var headersToAdd = new Dictionary + { + { "Content-Type", "application/json" }, + { "Authorization", "Bearer token" } + }; + + // Act + var result = coreRequest.AddHeaders(headersToAdd); + + // Assert + Assert.AreEqual(2, result.Count); + Assert.AreEqual("application/json", result["Content-Type"]); + Assert.AreEqual("Bearer token", result["Authorization"]); + } + + [Test] + public void AddQueryParameters_ShouldAddQueryParameters_WhenQueryParametersAreNotNull() + { + // Arrange + var coreRequest = new HttpRequest(HttpMethod.Get, "https://myurl.com"); + var queryParametersToAdd = new Dictionary + { + { "search", "test" }, + { "limit", 10 } + }; + + // Act + coreRequest.AddQueryParameters(queryParametersToAdd); + + // Assert + Assert.AreEqual(2, coreRequest.QueryParameters.Count); + Assert.AreEqual("test", coreRequest.QueryParameters["search"]); + Assert.AreEqual(10, coreRequest.QueryParameters["limit"]); + } } } diff --git a/APIMatic.Core/Types/Sdk/CoreContext.cs b/APIMatic.Core/Types/Sdk/CoreContext.cs index df328c00..52b11b85 100644 --- a/APIMatic.Core/Types/Sdk/CoreContext.cs +++ b/APIMatic.Core/Types/Sdk/CoreContext.cs @@ -31,11 +31,5 @@ public class CoreContext public Res Response { get; } internal bool IsFailure() => (Response.StatusCode < 200) || (Response.StatusCode > 208); - - /// - public override string ToString() - { - return $" Request = {Request}, Response = {Response}"; - } } } diff --git a/APIMatic.Core/Types/Sdk/CoreRequest.cs b/APIMatic.Core/Types/Sdk/CoreRequest.cs index e0ceb76e..2eae0f68 100644 --- a/APIMatic.Core/Types/Sdk/CoreRequest.cs +++ b/APIMatic.Core/Types/Sdk/CoreRequest.cs @@ -128,18 +128,5 @@ internal string GetContentType() => Headers?.Where(p => p.Key.EqualsIgnoreCase(" /// otherwise, returns an empty string. /// internal string GetBodyAsString() => Body == null ? string.Empty : Body.ToString(); - - /// - public override string ToString() - { - return $" HttpMethod = {HttpMethod}, " + - $" QueryUrl = {QueryUrl}, " + - $" QueryParameters = {CoreHelper.JsonSerialize(QueryParameters)}, " + - $" Headers = {CoreHelper.JsonSerialize(Headers)}, " + - $" FormParameters = {CoreHelper.JsonSerialize(FormParameters)}, " + - $" Body = {Body}, " + - $" Username = {Username}, " + - $" Password = {Password}"; - } } } diff --git a/APIMatic.Core/Types/Sdk/CoreResponse.cs b/APIMatic.Core/Types/Sdk/CoreResponse.cs index 7e8d2942..0f688d3f 100644 --- a/APIMatic.Core/Types/Sdk/CoreResponse.cs +++ b/APIMatic.Core/Types/Sdk/CoreResponse.cs @@ -41,13 +41,5 @@ public CoreResponse(int statusCode, Dictionary headers, Stream r /// Gets the raw string body of the http response. /// public string Body { get; } - - /// - public override string ToString() - { - return $" StatusCode = {StatusCode}, " + - $" Headers = {CoreHelper.JsonSerialize(Headers)}" + - $" RawBody = {RawBody}"; - } } }