Skip to content

Commit accd090

Browse files
committed
improve and clean up test codes
1 parent 65a94e9 commit accd090

File tree

1 file changed

+19
-4
lines changed

1 file changed

+19
-4
lines changed

tests/Smdn.Net.SmartMeter/Smdn.Net.SmartMeter/SmartMeterDataAggregatorServiceCollectionExtensions.cs

Lines changed: 19 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,15 @@ private static void ConfigureNothing(
2121
AddResiliencePipelineContext<string> context
2222
)
2323
{
24+
// do nothing
2425
}
2526

2627
private static void ConfigureNothing(
2728
ResiliencePipelineBuilder builder,
2829
AddResiliencePipelineContext<SmartMeterDataAggregator.ResiliencePipelineKeyPair<string>> context
2930
)
3031
{
32+
// do nothing
3133
}
3234

3335
private static void AssertResiliencePipelineRegistered(
@@ -42,6 +44,12 @@ string pipelineKey
4244
() => pipelineProvider.GetPipeline(pipelineKey),
4345
Throws.Nothing
4446
);
47+
Assert.That(
48+
pipelineProvider.GetPipeline(pipelineKey),
49+
Is.SameAs(
50+
pipelineProvider.GetPipeline(pipelineKey)
51+
)
52+
);
4553
}
4654

4755
private static void AssertResiliencePipelineRegistered<TServiceKey>(
@@ -50,17 +58,24 @@ private static void AssertResiliencePipelineRegistered<TServiceKey>(
5058
string pipelineKey
5159
)
5260
{
53-
foreach (var desc in services) {
54-
Console.WriteLine(desc);
55-
}
56-
5761
var serviceProvider = services.BuildServiceProvider();
5862
var pipelineProvider = serviceProvider.GetRequiredKeyedService<ResiliencePipelineProvider<string>>(serviceKey: serviceKey);
5963

64+
Assert.That(
65+
serviceProvider.GetRequiredKeyedService<ResiliencePipelineProvider<string>>(serviceKey: serviceKey),
66+
Is.SameAs(pipelineProvider)
67+
);
68+
6069
Assert.That(
6170
() => pipelineProvider.GetPipeline(pipelineKey),
6271
Throws.Nothing
6372
);
73+
Assert.That(
74+
pipelineProvider.GetPipeline(pipelineKey),
75+
Is.SameAs(
76+
pipelineProvider.GetPipeline(pipelineKey)
77+
)
78+
);
6479
}
6580

6681
[Test]

0 commit comments

Comments
 (0)