diff options
| author | Sebastien Pouliot <sebastien@ximian.com> | 2010-04-27 04:41:30 +0400 |
|---|---|---|
| committer | Sebastien Pouliot <sebastien@ximian.com> | 2010-04-27 04:41:30 +0400 |
| commit | 4916de15e83c44d23088eda070f43dba29555dbd (patch) | |
| tree | 042decb006a7408a9292a6f8b1e5e1a9296e715d /gendarme/rules/Gendarme.Rules.Smells/Test | |
| parent | 49bacea2068a21c1d3a134fdb9460ac76a66369c (diff) | |
In Test:
2010-01-26 Sebastien Pouliot <sebastien@ximian.com>
* AvoidMessageChainsTest.cs: Adjust test to new condition and
values.
In .:
2010-04-26 Sebastien Pouliot <sebastien@ximian.com>
* AvoidMessageChainsRule.cs: Change MaxChainLength from 5 to 4
and condition from >= to > (same effect) so the rule description
(and documentation) makes more sense.
svn path=/trunk/mono-tools/; revision=156153
Diffstat (limited to 'gendarme/rules/Gendarme.Rules.Smells/Test')
| -rw-r--r-- | gendarme/rules/Gendarme.Rules.Smells/Test/AvoidMessageChainsTest.cs | 4 | ||||
| -rw-r--r-- | gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog | 5 |
2 files changed, 7 insertions, 2 deletions
diff --git a/gendarme/rules/Gendarme.Rules.Smells/Test/AvoidMessageChainsTest.cs b/gendarme/rules/Gendarme.Rules.Smells/Test/AvoidMessageChainsTest.cs index 981b882c..c64432ae 100644 --- a/gendarme/rules/Gendarme.Rules.Smells/Test/AvoidMessageChainsTest.cs +++ b/gendarme/rules/Gendarme.Rules.Smells/Test/AvoidMessageChainsTest.cs @@ -103,12 +103,12 @@ namespace Test.Rules.Smells { int chain = Rule.MaxChainLength; try { // unless we lower the limit - Rule.MaxChainLength = 3; + Rule.MaxChainLength = 2; AssertRuleFailure<AvoidMessageChainsTest> ("MethodWithSmallChain", 1); AssertRuleFailure<AvoidMessageChainsTest> ("MethodWithArgumentsWithSmallChain", 1); } finally { - Rule.MaxChainLength = 5; + Rule.MaxChainLength = chain; } } diff --git a/gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog b/gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog index c396f1ff..5c50655b 100644 --- a/gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog +++ b/gendarme/rules/Gendarme.Rules.Smells/Test/ChangeLog @@ -1,3 +1,8 @@ +2010-01-26 Sebastien Pouliot <sebastien@ximian.com> + + * AvoidMessageChainsTest.cs: Adjust test to new condition and + values. + 2010-01-24 Sebastien Pouliot <sebastien@ximian.com> * AvoidSwitchStatementsTest.cs: Deal with different compiler |