Skip to content

Commit

Permalink
Minor refactor: Adjust test cases naming
Browse files Browse the repository at this point in the history
  • Loading branch information
andreise committed Jun 8, 2024
1 parent d4ac5b2 commit 698c85e
Show file tree
Hide file tree
Showing 4 changed files with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,7 @@ public static void DeserializeDtoWithArrayToDtoWithUnitFromObject_ExpectNoExcept

foreach (var array in arrays)
{
foreach (var options in BuildJsonSerializerOptionsCollection())
foreach (var options in EnumerateJsonSerializerOptionsCases())
{
result.Add(array, options);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,7 @@ public static void DeserializeDtoWithNonEmptyObjectToDtoWithUnitFromObject_Expec

foreach (var obj in objects)
{
foreach (var options in BuildJsonSerializerOptionsCollection())
foreach (var options in EnumerateJsonSerializerOptionsCases())
{
result.Add(obj, options);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,7 @@ public static void DeserializeDtoWithValueToDtoWithUnitFromObject_ExpectNoExcept

foreach (var value in values)
{
foreach (var options in BuildJsonSerializerOptionsCollection())
foreach (var options in EnumerateJsonSerializerOptionsCases())
{
result.Add(value, options);
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,9 @@ public static partial class UnitSerializationTests

public static TheoryData<JsonSerializerOptions?> JsonSerializerOptionsCases
=>
new(BuildJsonSerializerOptionsCollection().ToArray());
new(EnumerateJsonSerializerOptionsCases().ToArray());

private static IEnumerable<JsonSerializerOptions?> BuildJsonSerializerOptionsCollection()
private static IEnumerable<JsonSerializerOptions?> EnumerateJsonSerializerOptionsCases()
{
yield return null;
yield return JsonSerializerOptions.Default;
Expand Down

0 comments on commit 698c85e

Please sign in to comment.