Author: spouliot
Date: 2008-02-15 15:29:45 -0500 (Fri, 15 Feb 2008)
New Revision: 95823

Modified:
   
trunk/mono-tools/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUncalledPrivateCodeTest.cs
   trunk/mono-tools/gendarme/rules/Gendarme.Rules.Performance/Test/ChangeLog
Log:
2008-02-15  Sebastien Pouliot  <[EMAIL PROTECTED]>

        * AvoidUncalledPrivateCodeTest.cs: Fix RuleResult checks in loops.



Modified: 
trunk/mono-tools/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUncalledPrivateCodeTest.cs
===================================================================
--- 
trunk/mono-tools/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUncalledPrivateCodeTest.cs
     2008-02-15 20:19:23 UTC (rev 95822)
+++ 
trunk/mono-tools/gendarme/rules/Gendarme.Rules.Performance/Test/AvoidUncalledPrivateCodeTest.cs
     2008-02-15 20:29:45 UTC (rev 95823)
@@ -469,10 +469,12 @@
                        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 @@
                                        // 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;
                                }
                        }

Modified: 
trunk/mono-tools/gendarme/rules/Gendarme.Rules.Performance/Test/ChangeLog
===================================================================
--- trunk/mono-tools/gendarme/rules/Gendarme.Rules.Performance/Test/ChangeLog   
2008-02-15 20:19:23 UTC (rev 95822)
+++ trunk/mono-tools/gendarme/rules/Gendarme.Rules.Performance/Test/ChangeLog   
2008-02-15 20:29:45 UTC (rev 95823)
@@ -1,3 +1,7 @@
+2008-02-15  Sebastien Pouliot  <[EMAIL PROTECTED]>
+
+       * AvoidUncalledPrivateCodeTest.cs: Fix RuleResult checks in loops.
+
 2008-02-06  Sebastien Pouliot  <[EMAIL PROTECTED]>
 
        * AvoidUnsealedUninheritedInternalClassesTest.cs: New. Unit tests

_______________________________________________
Mono-patches maillist  -  [email protected]
http://lists.ximian.com/mailman/listinfo/mono-patches

Reply via email to