diff options
author | Sebastien Pouliot <sebastien@ximian.com> | 2010-06-10 16:31:51 +0400 |
---|---|---|
committer | Sebastien Pouliot <sebastien@ximian.com> | 2010-06-10 16:31:51 +0400 |
commit | 597de5c95def08da8d5a3703f7c62486542af998 (patch) | |
tree | 33793476b99a2a99b2455e9072c1cb33429a3916 /gendarme/rules/Gendarme.Rules.Smells/Test | |
parent | 9275ea4563265111efdd6b9156be9df3ae0741e7 (diff) |
In Test:
2010-06-10 Sebastien Pouliot <sebastien@ximian.com>
* AvoidLargeClassesTest.cs: Adjust defects count for prefixes
In .:
2010-06-10 Sebastien Pouliot <sebastien@ximian.com>
* AvoidLargeClassesRule.cs: Be more accurate when reporting. E.g.
report all common prefixes with a count
svn path=/trunk/mono-tools/; revision=158786
Diffstat (limited to 'gendarme/rules/Gendarme.Rules.Smells/Test')
-rw-r--r-- | gendarme/rules/Gendarme.Rules.Smells/Test/AvoidLargeClassesTest.cs | 3 | ||||
-rw-r--r-- | gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog | 4 |
2 files changed, 6 insertions, 1 deletions
diff --git a/gendarme/rules/Gendarme.Rules.Smells/Test/AvoidLargeClassesTest.cs b/gendarme/rules/Gendarme.Rules.Smells/Test/AvoidLargeClassesTest.cs index a767ef54..cc20d62d 100644 --- a/gendarme/rules/Gendarme.Rules.Smells/Test/AvoidLargeClassesTest.cs +++ b/gendarme/rules/Gendarme.Rules.Smells/Test/AvoidLargeClassesTest.cs @@ -134,7 +134,8 @@ namespace Test.Rules.Smells { { type = assembly.MainModule.Types["Test.Rules.Smells.LargeClass"]; Assert.AreEqual (RuleResult.Failure, runner.CheckType (type)); - Assert.AreEqual (2, runner.Defects.Count); + // 'x', 'foo', 'bar', 'f', 'c' and 's' are reported as prefixes + Assert.AreEqual (6, runner.Defects.Count); } [Test] diff --git a/gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog b/gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog index bedf995a..7746e6ce 100644 --- a/gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog +++ b/gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog @@ -1,3 +1,7 @@ +2010-06-10 Sebastien Pouliot <sebastien@ximian.com> + + * AvoidLargeClassesTest.cs: Adjust defects count for prefixes + 2010-06-10 Sebastien Pouliot <sebastien@ximian.com> * AvoidCodeDuplicatedInSameClassTest.cs: Add test case for |