Skip to content

Commit be119a8

Browse files
committed
Fix MSTEST0037 analyzer warnings
Refactored assertions to use local variables for actual values instead of inline method calls. This avoids triggering the MSTEST0037 analyzer which was confused by the inline method call pattern. Changes: - Store GetLevelFromReputation/GetLevelFromDignity results in local variables - Use descriptive variable names (actualFromMin, actualFromMax) to clarify intent - Removed NoWarn suppression now that the issue is properly fixed
1 parent 61c7e89 commit be119a8

2 files changed

Lines changed: 8 additions & 5 deletions

File tree

test/NosCore.Algorithm.Tests/DocumentationTest.cs

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -165,8 +165,10 @@ public void ReputationDocumentation()
165165
resultBuilder.AppendLine($"- {(byte)reput,2} {reput.ToString().PadRight(16)} - Min: {result.Item1} Max: {result.Item2}");
166166
if (reput < ReputationType.GreenLegend)
167167
{
168-
Assert.AreEqual(reput, reputationService.GetLevelFromReputation(result.Item1));
169-
Assert.AreEqual(reput, reputationService.GetLevelFromReputation(result.Item2));
168+
var actualFromMin = reputationService.GetLevelFromReputation(result.Item1);
169+
var actualFromMax = reputationService.GetLevelFromReputation(result.Item2);
170+
Assert.AreEqual(reput, actualFromMin);
171+
Assert.AreEqual(reput, actualFromMax);
170172
}
171173
}
172174

@@ -184,8 +186,10 @@ public void DignityDocumentation()
184186
{
185187
var result = dignityService.GetDignity(dignity);
186188
resultBuilder.AppendLine($"- {(byte)dignity,2} {dignity.ToString().PadRight(11)} - Max: {result.Item1} Min: {result.Item2}");
187-
Assert.AreEqual(dignity, dignityService.GetLevelFromDignity(result.Item1));
188-
Assert.AreEqual(dignity, dignityService.GetLevelFromDignity(result.Item2));
189+
var actualFromMax = dignityService.GetLevelFromDignity(result.Item1);
190+
var actualFromMin = dignityService.GetLevelFromDignity(result.Item2);
191+
Assert.AreEqual(dignity, actualFromMax);
192+
Assert.AreEqual(dignity, actualFromMin);
189193
}
190194

191195
Approvals.Verify(WriterFactory.CreateTextWriter(resultBuilder.ToString(), "md"));

test/NosCore.Algorithm.Tests/NosCore.Algorithm.Tests.csproj

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55
<TreatWarningsAsErrors>true</TreatWarningsAsErrors>
66
<IsPackable>false</IsPackable>
77
<Nullable>enable</Nullable>
8-
<NoWarn>MSTEST0037</NoWarn>
98
</PropertyGroup>
109

1110
<PropertyGroup Condition="'$(Configuration)|$(Platform)'=='Debug|AnyCPU'">

0 commit comments

Comments
 (0)