From 1a9b59efb44e69d39727885531c75430e7f2b12a Mon Sep 17 00:00:00 2001 From: Blank Date: Wed, 12 Aug 2026 22:26:47 +0800 Subject: [PATCH] =?UTF-8?q?test(options):=20=E6=B6=88=E9=99=A4=20FlagOptio?= =?UTF-8?q?nAttributeTests=20=E7=9A=84=20Sonar=20S2701=20=E5=AD=97?= =?UTF-8?q?=E9=9D=A2=E9=87=8F=E5=B8=83=E5=B0=94=E6=96=AD=E8=A8=80?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 将 5 处 Assert.Equal(false, attr.DefaultValue)(DefaultValue 为 object 装箱 false)改写为 Assert.False((bool)attr.DefaultValue),消除 csharpsquid:S2701 违规,断言语义不变。 Linear: GFX-639 --- .../Options/FlagOptionAttributeTests.cs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/GameFrameX.Foundation.Tests/Options/FlagOptionAttributeTests.cs b/GameFrameX.Foundation.Tests/Options/FlagOptionAttributeTests.cs index 4505aa8..3367e16 100644 --- a/GameFrameX.Foundation.Tests/Options/FlagOptionAttributeTests.cs +++ b/GameFrameX.Foundation.Tests/Options/FlagOptionAttributeTests.cs @@ -21,7 +21,7 @@ public class FlagOptionAttributeTests public void DefaultConstructor_ShouldSetDefaultValueToFalse() { var attr = new FlagOptionAttribute(); - Assert.Equal(false, attr.DefaultValue); + Assert.False((bool)attr.DefaultValue); } [Fact] @@ -29,7 +29,7 @@ public void ConstructorWithLongName_ShouldSetLongNameAndDefaultValueToFalse() { var attr = new FlagOptionAttribute("verbose"); Assert.Equal("verbose", attr.LongName); - Assert.Equal(false, attr.DefaultValue); + Assert.False((bool)attr.DefaultValue); } [Fact] @@ -37,7 +37,7 @@ public void ConstructorWithNullLongName_ShouldSetNullLongNameAndDefaultValueToFa { var attr = new FlagOptionAttribute(null); Assert.Null(attr.LongName); - Assert.Equal(false, attr.DefaultValue); + Assert.False((bool)attr.DefaultValue); } [Fact] @@ -45,7 +45,7 @@ public void ConstructorWithEmptyLongName_ShouldSetEmptyLongName() { var attr = new FlagOptionAttribute(""); Assert.Equal("", attr.LongName); - Assert.Equal(false, attr.DefaultValue); + Assert.False((bool)attr.DefaultValue); } // ============================================================ @@ -73,7 +73,7 @@ public void FlagOptionAttribute_ShouldAccessInheritedProperties() Assert.Equal("Enable debug mode", attr.Description); Assert.Equal("DEBUG_FLAG", attr.EnvironmentVariable); Assert.False(attr.Sensitive); - Assert.Equal(false, attr.DefaultValue); + Assert.False((bool)attr.DefaultValue); } // ============================================================