diff --git a/pmd/regress/test/net/sourceforge/pmd/AbstractRuleTest.java b/pmd/regress/test/net/sourceforge/pmd/AbstractRuleTest.java index 61521eb4e9..5b26538e22 100644 --- a/pmd/regress/test/net/sourceforge/pmd/AbstractRuleTest.java +++ b/pmd/regress/test/net/sourceforge/pmd/AbstractRuleTest.java @@ -197,7 +197,7 @@ public class AbstractRuleTest { MyRule r2 = new MyRule(); r2.setMessage("another message"); assertEquals("Rules with different messages are still equal", r1, r2); - assertSame("Rules that are equal must have the same hashcode", r1.hashCode(), r2.hashCode()); + assertEquals("Rules that are equal must have the an equal hashcode", r1.hashCode(), r2.hashCode()); } diff --git a/pmd/regress/test/net/sourceforge/pmd/lang/java/rule/typeresolution/xml/MyTestCase.java b/pmd/regress/test/net/sourceforge/pmd/lang/java/rule/typeresolution/xml/MyTestCase.java index f77b7861c7..9739ddd493 100644 --- a/pmd/regress/test/net/sourceforge/pmd/lang/java/rule/typeresolution/xml/MyTestCase.java +++ b/pmd/regress/test/net/sourceforge/pmd/lang/java/rule/typeresolution/xml/MyTestCase.java @@ -3,9 +3,9 @@ package test.net.sourceforge.pmd.lang.java.rule.typeresolution.xml; import junit.framework.TestCase; /** - * Warning, this class ARE not useless. - * It is used by the some of regression - * tests. + * Warning, this class IS NOT useless. + * It is used by the some regression tests. + * * See file: SignatureDeclareThrowsException.xml * * The file is already excluded from maven/surefire. diff --git a/pmd/rulesets/java/unnecessary.xml b/pmd/rulesets/java/unnecessary.xml index aee0a7c21d..7c2e8824a8 100644 --- a/pmd/rulesets/java/unnecessary.xml +++ b/pmd/rulesets/java/unnecessary.xml @@ -157,7 +157,7 @@ After checking an object reference for null, you should invoke equals() on that