Skip to content

Commit 90bb92c

Browse files
author
Bart Koelman
committed
Cleanup tests for handling special characters
1 parent 03e321a commit 90bb92c

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

test/JsonApiDotNetCoreTests/IntegrationTests/QueryStrings/Filtering/FilterDataTypeTests.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
using System.Reflection;
66
using System.Text.Json.Serialization;
77
using System.Threading.Tasks;
8+
using System.Web;
89
using FluentAssertions;
910
using FluentAssertions.Extensions;
1011
using Humanizer;
@@ -208,7 +209,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
208209
await dbContext.SaveChangesAsync();
209210
});
210211

211-
string route = $"/filterableResources?filter=equals(someDateTimeOffset,'{WebUtility.UrlEncode(resource.SomeDateTimeOffset.ToString("O"))}')";
212+
string route = $"/filterableResources?filter=equals(someDateTimeOffset,'{HttpUtility.UrlEncode(resource.SomeDateTimeOffset.ToString("O"))}')";
212213

213214
// Act
214215
(HttpResponseMessage httpResponse, Document responseDocument) = await _testContext.ExecuteGetAsync<Document>(route);

test/JsonApiDotNetCoreTests/IntegrationTests/QueryStrings/Filtering/FilterOperatorTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ public async Task Can_filter_equality_on_special_characters()
3737
// Arrange
3838
var resource = new FilterableResource
3939
{
40-
SomeString = "This, that & more"
40+
SomeString = "This, that & more + some"
4141
};
4242

4343
await _testContext.RunOnDatabaseAsync(async dbContext =>

test/JsonApiDotNetCoreTests/IntegrationTests/QueryStrings/Pagination/PaginationWithTotalCountTests.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -582,7 +582,6 @@ public async Task Renders_correct_top_level_links_for_page_number(int pageNumber
582582
{
583583
// Arrange
584584
WebAccount account = _fakers.WebAccount.Generate();
585-
account.UserName = $"&{account.UserName}";
586585

587586
const int totalCount = 3 * DefaultPageSize + 3;
588587
List<BlogPost> posts = _fakers.BlogPost.Generate(totalCount);
@@ -599,7 +598,7 @@ await _testContext.RunOnDatabaseAsync(async dbContext =>
599598
await dbContext.SaveChangesAsync();
600599
});
601600

602-
string routePrefix = $"/blogPosts?filter=equals(author.userName,'{WebUtility.UrlEncode(account.UserName)}')" +
601+
string routePrefix = $"/blogPosts?filter=equals(author.userName,'{account.UserName}')" +
603602
"&fields[webAccounts]=userName&include=author&sort=id&foo=bar,baz";
604603

605604
string route = $"{routePrefix}&page[number]={pageNumber}";

0 commit comments

Comments
 (0)