Cleanup tests for handling special characters

json-api-dotnet · Dec 3, 2021 · 5c1648a · 5c1648a
1 parent fec1973
commit 5c1648a
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/test/JsonApiDotNetCoreTests/IntegrationTests/QueryStrings/Filtering/FilterDataTypeTests.cs b/test/JsonApiDotNetCoreTests/IntegrationTests/QueryStrings/Filtering/FilterDataTypeTests.cs
@@ -5,6 +5,7 @@
 using System.Reflection;
 using System.Text.Json.Serialization;
 using System.Threading.Tasks;
+using System.Web;
 using FluentAssertions;
 using FluentAssertions.Extensions;
 using Humanizer;
@@ -208,7 +209,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
                 await dbContext.SaveChangesAsync();
             });
 
-            string route = $"/filterableResources?filter=equals(someDateTimeOffset,'{WebUtility.UrlEncode(resource.SomeDateTimeOffset.ToString("O"))}')";
+            string route = $"/filterableResources?filter=equals(someDateTimeOffset,'{HttpUtility.UrlEncode(resource.SomeDateTimeOffset.ToString("O"))}')";
 
             // Act
             (HttpResponseMessage httpResponse, Document responseDocument) = await _testContext.ExecuteGetAsync<Document>(route);

diff --git a/test/JsonApiDotNetCoreTests/IntegrationTests/QueryStrings/Filtering/FilterOperatorTests.cs b/test/JsonApiDotNetCoreTests/IntegrationTests/QueryStrings/Filtering/FilterOperatorTests.cs
@@ -36,7 +36,7 @@ public async Task Can_filter_equality_on_special_characters()
             // Arrange
             var resource = new FilterableResource
             {
-                SomeString = "This, that & more"
+                SomeString = "This, that & more + some"
             };
 
             await _testContext.RunOnDatabaseAsync(async dbContext =>

diff --git a/...DotNetCoreTests/IntegrationTests/QueryStrings/Pagination/PaginationWithTotalCountTests.cs b/...DotNetCoreTests/IntegrationTests/QueryStrings/Pagination/PaginationWithTotalCountTests.cs
@@ -582,7 +582,6 @@ public async Task Renders_correct_top_level_links_for_page_number(int pageNumber
         {
             // Arrange
             WebAccount account = _fakers.WebAccount.Generate();
-            account.UserName = $"&{account.UserName}";
 
             const int totalCount = 3 * DefaultPageSize + 3;
             List<BlogPost> posts = _fakers.BlogPost.Generate(totalCount);
@@ -599,7 +598,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
                 await dbContext.SaveChangesAsync();
             });
 
-            string routePrefix = $"/blogPosts?filter=equals(author.userName,'{WebUtility.UrlEncode(account.UserName)}')" +
+            string routePrefix = $"/blogPosts?filter=equals(author.userName,'{account.UserName}')" +
                 "&fields[webAccounts]=userName&include=author&sort=id&foo=bar,baz";
 
             string route = $"{routePrefix}&page[number]={pageNumber}";