Skip to content

Commit 54d4e4b

Browse files
committed
fix for test
1 parent b928b31 commit 54d4e4b

File tree

3 files changed

+24
-9
lines changed

3 files changed

+24
-9
lines changed

src/WebSerializer/WebSerializerWriter.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ public void AppendRaw(string value)
7878
stringBuilder.Append(value);
7979
}
8080

81-
/// <summary>Append '&'.</summary>
81+
/// <summary>Append and.</summary>
8282
[MethodImpl(MethodImplOptions.AggressiveInlining)]
8383
public void AppendConcatenate()
8484
{

tests/WebSerializer.Tests/CollectionTest.cs

Lines changed: 17 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33
using System.Globalization;
44
using System.Linq;
55
using System.Text;
6+
using System.Text.Encodings.Web;
67
using System.Threading.Tasks;
78

89
namespace WebSerializerTests;
@@ -29,14 +30,22 @@ public void Dict()
2930
{"DEAR", true }
3031
};
3132

32-
var newConfig = WebSerializerOptions.Default with
33-
{
34-
CultureInfo = CultureInfo.CurrentCulture,
35-
CollectionSeparator = " ",
36-
Provider = WebSerializerProvider.Create(
37-
new[] { new BoolZeroOneSerializer() },
38-
new[] { WebSerializerProvider.Default })
39-
};
33+
var newConfig = WebSerializerOptions.Default with
34+
{
35+
CultureInfo = CultureInfo.CurrentCulture,
36+
CollectionSeparator = " ",
37+
Provider = WebSerializerProvider.Create(
38+
new[] { new BoolZeroOneSerializer() },
39+
new[] { WebSerializerProvider.Default })
40+
};
41+
42+
var sb = new StringBuilder();
43+
foreach (var item in foo)
44+
{
45+
if (sb.Length != 0) sb.Append('&');
46+
sb.Append(item.Key + "=" + UrlEncoder.Default.Encode(item.Value.ToString() ?? ""));
47+
}
48+
4049
WebSerializer.ToQueryString(foo, newConfig).Should().Be("Yeah=100&Kuooo=nano%20yo&DEAR=0");
4150
}
4251

tests/WebSerializer.Tests/StandardTest.cs

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
using System.Runtime.Serialization;
2+
13
namespace WebSerializerTests;
24

35
public class StandardTest
@@ -67,8 +69,12 @@ public void CheckRecursive()
6769

6870
public class PagingRequest
6971
{
72+
73+
[DataMember(Order = 1)]
7074
public string? SortBy { get; init; }
75+
[DataMember(Order = 2)]
7176
public SortDirection SortDirection { get; init; }
77+
[DataMember(Order = 0)]
7278
public int CurrentPage { get; init; } = 1;
7379
}
7480

0 commit comments

Comments
 (0)