diff --git a/.ci/files/all-java.xml b/.ci/files/all-java.xml index 83ecc15f9b..d149f2de65 100644 --- a/.ci/files/all-java.xml +++ b/.ci/files/all-java.xml @@ -129,7 +129,7 @@ - + diff --git a/pmd-java/src/main/resources/category/java/design.xml b/pmd-java/src/main/resources/category/java/design.xml index 1a73968171..c3c579c464 100644 --- a/pmd-java/src/main/resources/category/java/design.xml +++ b/pmd-java/src/main/resources/category/java/design.xml @@ -286,8 +286,8 @@ public class Foo { class="net.sourceforge.pmd.lang.rule.XPathRule" externalInfoUrl="${pmd.website.baseurl}/pmd_rules_java_design.html#avoiduncheckedexceptionsinsignatures"> -A method or constructor should not explicitly declare unchecked exceptions in its -`throws` clause. Java doesn't force the caller to handle an unchecked exception, +Reports unchecked exceptions in the `throws` clause of a method or constructor. +Java doesn't force the caller to handle an unchecked exception, so it's unnecessary except for documentation. A better practice is to document the exceptional cases with a `@throws` Javadoc tag, which allows being more descriptive. @@ -296,9 +296,7 @@ exceptional cases with a `@throws` Javadoc tag, which allows being more descript diff --git a/pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/design/AvoidUncheckedExceptionsInSignaturesTest.java b/pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/design/AvoidUncheckedExceptionsInSignaturesTest.java index 9e2cc6b138..62c9c4e7e8 100644 --- a/pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/design/AvoidUncheckedExceptionsInSignaturesTest.java +++ b/pmd-java/src/test/java/net/sourceforge/pmd/lang/java/rule/design/AvoidUncheckedExceptionsInSignaturesTest.java @@ -6,7 +6,6 @@ package net.sourceforge.pmd.lang.java.rule.design; import net.sourceforge.pmd.testframework.PmdRuleTst; -@org.junit.Ignore("Rule has not been updated yet") public class AvoidUncheckedExceptionsInSignaturesTest extends PmdRuleTst { // no additional unit tests }