diff options
author | Sebastien Pouliot <sebastien@ximian.com> | 2008-02-15 23:29:45 +0300 |
---|---|---|
committer | Sebastien Pouliot <sebastien@ximian.com> | 2008-02-15 23:29:45 +0300 |
commit | 930f1c00565c9c0f1969633b29744c1c86fffa6e (patch) | |
tree | 6a26ca4260ae61246f0747800391733fd64e55f3 /gendarme/rules | |
parent | fe1f50a4be920ac42bbd5f86f27bcce793963d70 (diff) |
2008-02-15 Sebastien Pouliot <sebastien@ximian.com>
* AvoidUncalledPrivateCodeTest.cs: Fix RuleResult checks in loops.
svn path=/trunk/mono-tools/; revision=95823
Diffstat (limited to 'gendarme/rules')
-rw-r--r-- | gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUncalledPrivateCodeTest.cs | 6 | ||||
-rw-r--r-- | gendarme/rules/Gendarme.Rules.Performance/Test/ChangeLog | 4 |
2 files changed, 8 insertions, 2 deletions
diff --git a/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUncalledPrivateCodeTest.cs b/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUncalledPrivateCodeTest.cs index 8baabd3e..af5b3f1c 100644 --- a/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUncalledPrivateCodeTest.cs +++ b/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUncalledPrivateCodeTest.cs @@ -469,10 +469,12 @@ namespace Test.Rules.Performance { foreach (MethodDefinition method in type.Methods) { switch (method.Name) { case "Main": + Assert.AreEqual (RuleResult.DoesNotApply, runner.CheckMethod (method), method.Name); + break; case "Test.Rules.Performance.AvoidUncalledPrivateCodeTest.Iface1.IfaceMethod1": // mono bug #343465 case "Test.Rules.Performance.AvoidUncalledPrivateCodeTest+Iface1.IfaceMethod1": - Assert.AreEqual (RuleResult.DoesNotApply, runner.CheckMethod (method), method.Name); + Assert.AreEqual (RuleResult.Success, runner.CheckMethod (method), method.Name); break; case "Test.Rules.Performance.AvoidUncalledPrivateCodeTest.Iface2.IfaceMethod2": // mono bug #343465 @@ -596,7 +598,7 @@ namespace Test.Rules.Performance { // this isn't part of the test (but included with CSC) break; default: - Assert.AreEqual (RuleResult.Success, runner.CheckMethod (method)); + Assert.AreEqual (RuleResult.DoesNotApply, runner.CheckMethod (method)); break; } } diff --git a/gendarme/rules/Gendarme.Rules.Performance/Test/ChangeLog b/gendarme/rules/Gendarme.Rules.Performance/Test/ChangeLog index 659bb166..5998a861 100644 --- a/gendarme/rules/Gendarme.Rules.Performance/Test/ChangeLog +++ b/gendarme/rules/Gendarme.Rules.Performance/Test/ChangeLog @@ -1,3 +1,7 @@ +2008-02-15 Sebastien Pouliot <sebastien@ximian.com> + + * AvoidUncalledPrivateCodeTest.cs: Fix RuleResult checks in loops. + 2008-02-06 Sebastien Pouliot <sebastien@ximian.com> * AvoidUnsealedUninheritedInternalClassesTest.cs: New. Unit tests |