[java-metadata-aggregator] branch master updated: JPAR-131 - Fix doclint issues in test javadoc

Ian Young ian at iay.org.uk
Fri Dec 20 13:05:15 EST 2019


This is an automated email from the git hooks/post-receive script.

iay pushed a commit to branch master
in repository java-metadata-aggregator.

View the commit online:
http://git.shibboleth.net/view/?p=java-metadata-aggregator.git;a=commit;h=d27bc8bbf0a61a170e98ad48f0a951b4793be8f9

The following commit(s) were added to refs/heads/master by this push:
       new  d27bc8b   JPAR-131 - Fix doclint issues in test javadoc
d27bc8b is described below

commit d27bc8bbf0a61a170e98ad48f0a951b4793be8f9
Author: Ian Young <ian at iay.org.uk>
AuthorDate: Fri Dec 20 18:05:09 2019 +0000

    JPAR-131 - Fix doclint issues in test javadoc
    
    https://issues.shibboleth.net/jira/browse/JPAR-131
---
 .../java/net/shibboleth/metadata/BaseTest.java     |  6 ++-
 .../net/shibboleth/metadata/dom/BaseDOMTest.java   |  8 +++-
 .../metadata/dom/XMLSignatureSigningStageTest.java | 12 ++++-
 .../dom/XMLSignatureValidationStageTest.java       | 15 +++++-
 .../metadata/dom/XSLValidationStageTest.java       |  8 +++-
 .../metadata/dom/ds/X509ValidationStageTest.java   |  1 -
 .../EntityDescriptorItemIdPopulationStageTest.java | 24 ++++++++--
 .../metadata/dom/saml/EntityFilterStageTest.java   | 34 ++++++++++++--
 .../dom/saml/EntityRoleFilterStageTest.java        | 38 +++++++++++++---
 .../metadata/dom/saml/GenerateIdStageTest.java     |  2 +
 .../dom/saml/PullUpCacheDurationStageTest.java     | 20 ++++++--
 .../dom/saml/PullUpValidUntilStageTest.java        | 20 ++++++--
 .../dom/saml/RemoveOrganizationStageTest.java      | 14 ++++--
 .../dom/saml/ValidateValidUntilStageTest.java      | 14 +++++-
 ...EntityRegistrationAuthorityFilterStageTest.java | 53 +++++++++++++++++-----
 .../metadata/pipeline/CountingStage.java           |  6 ++-
 .../pipeline/ItemMetadataFilterStageTest.java      | 20 ++++++--
 .../pipeline/ItemMetadataTerminationStageTest.java | 20 ++++++--
 .../metadata/pipeline/PipelineMergeStageTest.java  |  9 +++-
 .../metadata/pipeline/SplitMergeStageTest.java     | 14 ++++--
 .../metadata/pipeline/TerminatingStage.java        |  6 ++-
 .../validate/x509/X509ROCAValidatorTest.java       |  1 -
 .../x509/X509RSAExponentValidatorTest.java         |  1 -
 .../x509/X509RSAKeyLengthValidatorTest.java        |  1 -
 .../x509/X509RSAOpenSSLBlacklistValidatorTest.java |  1 -
 25 files changed, 282 insertions(+), 66 deletions(-)

diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/BaseTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/BaseTest.java
index 7e87b04..50a033f 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/BaseTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/BaseTest.java
@@ -42,7 +42,11 @@ public abstract class BaseTest {
      */
     private final String basePackagePath;
     
-    /** Constructor */
+    /**
+     * Constructor
+     * 
+     * @param clazz class under test
+     */
     protected BaseTest(final Class<?> clazz) {
         testingClass = clazz;
         baseClassPath = nameToPath(testingClass.getName());
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/BaseDOMTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/BaseDOMTest.java
index dd455bb..edfa449 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/BaseDOMTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/BaseDOMTest.java
@@ -63,7 +63,11 @@ public abstract class BaseDOMTest extends BaseTest {
      */
     private BasicParserPool parserPool;
 
-    /** Constructor */
+    /**
+     * Constructor
+     * 
+     * @param clazz class under test
+     */
     protected BaseDOMTest(final Class<?> clazz) {
         super(clazz);
     }
@@ -127,7 +131,7 @@ public abstract class BaseDOMTest extends BaseTest {
      * 
      * @return an {@link Item} wrapping the document representing the data file, never null
      * 
-     * @throws XMLParserExceptionthrown if the file does not exist or there is a problem parsing it
+     * @throws XMLParserException if the file does not exist or there is a problem parsing it
      */
     public Item<Element> readDOMItem(final String path) throws XMLParserException {
         final Element e = readXMLData(path);
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XMLSignatureSigningStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XMLSignatureSigningStageTest.java
index 1c3bb42..f657010 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XMLSignatureSigningStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XMLSignatureSigningStageTest.java
@@ -82,7 +82,11 @@ public class XMLSignatureSigningStageTest extends BaseDOMTest {
         return string.contains("
");
     }
 
-    /** Test signing with and verifying the result against a known good. */
+    /**
+     * Test signing with and verifying the result against a known good.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testSigning() throws Exception {
         Element testInput = readXMLData("input.xml");
@@ -115,7 +119,11 @@ public class XMLSignatureSigningStageTest extends BaseDOMTest {
         assertXMLIdentical(expected, result.unwrap());
     }
 
-    /** MDA-196: XMLSignatureSigningStage's includeX509SubjectName property causes exception */
+    /**
+     * MDA-196: XMLSignatureSigningStage's includeX509SubjectName property causes exception.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testMDA196() throws Exception {
         final Element testInput = readXMLData("input.xml");
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XMLSignatureValidationStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XMLSignatureValidationStageTest.java
index 11c7f94..979db81 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XMLSignatureValidationStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XMLSignatureValidationStageTest.java
@@ -40,7 +40,6 @@ public class XMLSignatureValidationStageTest extends BaseDOMTest {
     
     private final Certificate signingCert;
     
-    /** Constructor sets class under test. */
     public XMLSignatureValidationStageTest() throws IOException {
         super(XMLSignatureValidationStage.class);
         signingCert = CertUtil.readCertificate(XMLSignatureSigningStageTest.class
@@ -54,6 +53,8 @@ public class XMLSignatureValidationStageTest extends BaseDOMTest {
     
     /**
      * Tests verifying a file with a valid signature.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test
     public void testValidSignature() throws Exception {
@@ -84,6 +85,8 @@ public class XMLSignatureValidationStageTest extends BaseDOMTest {
 
     /**
      * Test that a metadata element with an invalid signature is labelled with an error.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test
     public void testInvalidSignature() throws Exception {
@@ -103,6 +106,8 @@ public class XMLSignatureValidationStageTest extends BaseDOMTest {
     /**
      * Test that metadata elements that do not contain signature are appropriately labelled when valid signatures
      * are required.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test
     public void testRequiredSignature() throws Exception {
@@ -140,6 +145,8 @@ public class XMLSignatureValidationStageTest extends BaseDOMTest {
     
     /**
      * Test digest algorithm blacklist.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test
     public void testDigestBlacklist() throws Exception {
@@ -171,6 +178,8 @@ public class XMLSignatureValidationStageTest extends BaseDOMTest {
 
     /**
      * Test signature method blacklist.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test
     public void testSignatureMethodBlacklist() throws Exception {
@@ -202,6 +211,8 @@ public class XMLSignatureValidationStageTest extends BaseDOMTest {
     
     /**
      * Test that a signature with an empty reference is permitted by default.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test
     public void testEmptyRefPermitted() throws Exception {
@@ -232,6 +243,8 @@ public class XMLSignatureValidationStageTest extends BaseDOMTest {
 
     /**
      * Test that a signature with an empty reference is not permitted if disallowed.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test
     public void testEmptyRefNotPermitted() throws Exception {
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XSLValidationStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XSLValidationStageTest.java
index 06cb6f4..05f173c 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XSLValidationStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/XSLValidationStageTest.java
@@ -43,7 +43,9 @@ public class XSLValidationStageTest extends BaseDOMTest {
     /**
      * Utility method to grab our standard input file and turn it into a {@link DOMElementItem}.
      * 
-     * @throws XMLParserException
+     * @return the parsed document wrapped in a {@link DOMElementItem}
+     * 
+     * @throws XMLParserException if the file does not exist or there is a problem parsing it
      */
     private DOMElementItem makeInput() throws XMLParserException {
         Element testInput = readXMLData("input.xml");
@@ -57,6 +59,8 @@ public class XSLValidationStageTest extends BaseDOMTest {
 
     /**
      * Test a validation transform.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test public void testValidation() throws Exception {
 
@@ -105,6 +109,8 @@ public class XSLValidationStageTest extends BaseDOMTest {
 
     /**
      * Test for MDA-45.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test public void testMDA45() throws Exception {
 
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/ds/X509ValidationStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/ds/X509ValidationStageTest.java
index 1e5063d..773cf43 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/ds/X509ValidationStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/ds/X509ValidationStageTest.java
@@ -40,7 +40,6 @@ import org.w3c.dom.Element;
 
 public class X509ValidationStageTest extends BaseDOMTest {
 
-    /** Constructor sets class under test. */
     public X509ValidationStageTest() throws Exception {
         super(X509ValidationStage.class);
     }
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityDescriptorItemIdPopulationStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityDescriptorItemIdPopulationStageTest.java
index 2b5fde7..d49037c 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityDescriptorItemIdPopulationStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityDescriptorItemIdPopulationStageTest.java
@@ -37,7 +37,11 @@ public class EntityDescriptorItemIdPopulationStageTest extends BaseDOMTest {
         super(EntityDescriptorItemIdPopulationStage.class);
     }
 
-    /** Tests running the stage on an empty collection. */
+    /**
+     * Tests running the stage on an empty collection.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testEmptyCollection() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -50,7 +54,11 @@ public class EntityDescriptorItemIdPopulationStageTest extends BaseDOMTest {
         Assert.assertTrue(metadataCollection.isEmpty());
     }
 
-    /** Tests running the stage on a collection that does not contain EntityDescriptors. */
+    /**
+     * Tests running the stage on a collection that does not contain EntityDescriptors.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testDifferentElement() throws Exception {
         final ArrayList<DOMElementItem> metadataCollection = new ArrayList<>();
@@ -64,7 +72,11 @@ public class EntityDescriptorItemIdPopulationStageTest extends BaseDOMTest {
         Assert.assertEquals(itemIds.size(), 0);
     }
 
-    /** Tests running the stage on a collection that contains a single EntityDescriptor. */
+    /**
+     * Tests running the stage on a collection that contains a single EntityDescriptor.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testSingleRecord() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -87,7 +99,11 @@ public class EntityDescriptorItemIdPopulationStageTest extends BaseDOMTest {
         Assert.assertEquals(itemId.getId(), entityDescriptor.getAttributeNS(null, "entityID"));
     }
 
-    /** Tests running the stage on a collection that contains multiple EntityDescriptors. */
+    /**
+     * Tests running the stage on a collection that contains multiple EntityDescriptors.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testMultipleRecords() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityFilterStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityFilterStageTest.java
index aebd1fd..bd45471 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityFilterStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityFilterStageTest.java
@@ -40,7 +40,11 @@ public class EntityFilterStageTest extends BaseDOMTest {
         super(EntityFilterStage.class);
     }
 
-    /** Test whitelisted entity is retained and ensure everything else is removed. */
+    /**
+     * Test whitelisted entity is retained and ensure everything else is removed.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test public void testEntityWhitelist() throws Exception {
         final EntityFilterStage stage = new EntityFilterStage();
         stage.setId("test");
@@ -54,7 +58,11 @@ public class EntityFilterStageTest extends BaseDOMTest {
         Assert.assertEquals(metadataCollection.size(), 1);
     }
 
-    /** Test blacklisted entity is remove and ensure everything else is retained. */
+    /**
+     * Test blacklisted entity is remove and ensure everything else is retained.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test public void testEntityBlacklist() throws Exception {
         final EntityFilterStage stage = new EntityFilterStage();
         stage.setId("test");
@@ -68,7 +76,11 @@ public class EntityFilterStageTest extends BaseDOMTest {
         Assert.assertEquals(metadataCollection.size(), 2);
     }
 
-    /** Test that filtering logic descends in to EntitiesDescriptors. */
+    /**
+     * Test that filtering logic descends in to EntitiesDescriptors.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test public void testEntitiesDescriptorFiltering() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
         metadataCollection.add(new DOMElementItem(readXMLData("in.xml")));
@@ -86,6 +98,8 @@ public class EntityFilterStageTest extends BaseDOMTest {
 
     /**
      * Test that EntitiesDescriptors that have had all their EntityDescriptor children remove are themselves removed.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test public void testRemoveEntitylessEntitiesDescriptor() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -105,6 +119,8 @@ public class EntityFilterStageTest extends BaseDOMTest {
     /**
      * Test that EntitiesDescriptors that have had all their EntityDescriptor children remove are not themselves removed
      * when {@link EntityFilterStage#isRemovingEntitylessEntitiesDescriptor()} is false.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test public void testDontRemoveEntitylessEntitiesDescriptor() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -124,6 +140,8 @@ public class EntityFilterStageTest extends BaseDOMTest {
 
     /**
      * Test that whitelisting an empty set of IDs removes everything from the collection.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test public void testWhitelistEmptySet() throws Exception {
         final var metadataCollection = buildMetadataCollection();
@@ -140,6 +158,8 @@ public class EntityFilterStageTest extends BaseDOMTest {
 
     /**
      * Test that blacklisting an empty set of IDs leaves everything in the collection.
+     * 
+     * @throws Exception if something bad happens
      */
     @Test public void testBlacklistEmptySet() throws Exception {
         final var metadataCollection = buildMetadataCollection();
@@ -154,7 +174,13 @@ public class EntityFilterStageTest extends BaseDOMTest {
         Assert.assertEquals(metadataCollection.size(), 3);
     }
     
-    /** Build up a metadata collection containing 3 EntityDescriptors. */
+    /**
+     * Build up a metadata collection containing three EntityDescriptors.
+     * 
+     * @return metadata collection
+     * 
+     * @throws Exception if something bad happens
+     */
     private Collection<Item<Element>> buildMetadataCollection() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
 
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityRoleFilterStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityRoleFilterStageTest.java
index 3a01b50..b5a329c 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityRoleFilterStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/EntityRoleFilterStageTest.java
@@ -44,8 +44,11 @@ public class EntityRoleFilterStageTest extends BaseDOMTest {
     /**
      * Test that whitelisted roles are retained and all other roles are removed. Also tests that roleless entities are
      * removed.
+     * 
+     * @throws Exception if something bad happens
      */
-    @Test public void testRoleWhitelist() throws Exception {
+    @Test
+    public void testRoleWhitelist() throws Exception {
         EntityRoleFilterStage stage = new EntityRoleFilterStage();
         stage.setId("test");
         stage.setDesignatedRoles(Collections.singletonList(EntityRoleFilterStage.IDP_SSO_DESCRIPTOR_NAME));
@@ -65,8 +68,11 @@ public class EntityRoleFilterStageTest extends BaseDOMTest {
     /**
      * Test that blacklisted roles are removed and all other roles are retained. Also tests that roleless entities are
      * removed.
+     * 
+     * @throws Exception if something bad happens
      */
-    @Test public void testRoleBlacklist() throws Exception {
+    @Test
+    public void testRoleBlacklist() throws Exception {
         EntityRoleFilterStage stage = new EntityRoleFilterStage();
         stage.setId("test");
         stage.setDesignatedRoles(Collections.singletonList(EntityRoleFilterStage.IDP_SSO_DESCRIPTOR_NAME));
@@ -90,8 +96,11 @@ public class EntityRoleFilterStageTest extends BaseDOMTest {
     /**
      * Test that EntityDescriptors that have had all their roles removed are not themselves removed if
      * {@link EntityRoleFilterStage#isRemovingRolelessEntities()} is false.
+     * 
+     * @throws Exception if something bad happens
      */
-    @Test public void testDontRemoveRolelessEntityDescriptor() throws Exception {
+    @Test
+    public void testDontRemoveRolelessEntityDescriptor() throws Exception {
         EntityRoleFilterStage stage = new EntityRoleFilterStage();
         stage.setId("test");
         stage.setDesignatedRoles(Collections.singletonList(EntityRoleFilterStage.IDP_SSO_DESCRIPTOR_NAME));
@@ -119,8 +128,11 @@ public class EntityRoleFilterStageTest extends BaseDOMTest {
 
     /**
      * Test that role filtering is performed on descendant elements of metadata collection elements.
+     * 
+     * @throws Exception if something bad happens
      */
-    @Test public void testEntitiesDescriptorFiltering() throws Exception {
+    @Test
+    public void testEntitiesDescriptorFiltering() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
         metadataCollection.add(new DOMElementItem(readXMLData("in.xml")));
 
@@ -146,8 +158,11 @@ public class EntityRoleFilterStageTest extends BaseDOMTest {
 
     /**
      * Test that EntitiesDescriptors that have had all their EntityDescriptor children remove are themselves removed.
+     * 
+     * @throws Exception if something bad happens
      */
-    @Test public void testRemoveEntitylessEntitiesDescriptor() throws Exception {
+    @Test
+    public void testRemoveEntitylessEntitiesDescriptor() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
         metadataCollection.add(new DOMElementItem(readXMLData("in.xml")));
 
@@ -166,8 +181,11 @@ public class EntityRoleFilterStageTest extends BaseDOMTest {
     /**
      * Test that EntitiesDescriptors that have had all their EntityDescriptor children remove are not themselves removed
      * when {@link EntityRoleFilterStage#isRemovingEntitylessEntitiesDescriptor()} is false.
+     * 
+     * @throws Exception if something bad happens
      */
-    @Test public void testDontRemoveEntitylessEntitiesDescriptor() throws Exception {
+    @Test
+    public void testDontRemoveEntitylessEntitiesDescriptor() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
         metadataCollection.add(new DOMElementItem(readXMLData("in.xml")));
 
@@ -184,7 +202,13 @@ public class EntityRoleFilterStageTest extends BaseDOMTest {
         Assert.assertEquals(metadataCollection.size(), 1);
     }
 
-    /** Build up a metadata collection containing 3 EntityDescriptors. */
+    /**
+     * Build up a metadata collection containing three EntityDescriptors.
+     * 
+     * @return metadata collection
+     * 
+     * @throws Exception if something bad happens
+     */
     private List<Item<Element>> buildMetadataCollection() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
 
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/GenerateIdStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/GenerateIdStageTest.java
index e8dfdbe..09df337 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/GenerateIdStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/GenerateIdStageTest.java
@@ -42,6 +42,8 @@ public class GenerateIdStageTest {
     /**
      * Test for MDA-90 (default identifier strategy used by GenerateIdStage
      * sometimes generates bad IDs).
+     * 
+     * @throws Exception if something bad happens
      */
     @Test
     public void mda90() throws Exception {
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/PullUpCacheDurationStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/PullUpCacheDurationStageTest.java
index f207a97..4a7bac7 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/PullUpCacheDurationStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/PullUpCacheDurationStageTest.java
@@ -40,7 +40,11 @@ public class PullUpCacheDurationStageTest extends BaseDOMTest {
         super(PullUpCacheDurationStage.class);
     }
 
-    /** Test that the shortest duration (1 hour) is pulled up to the EntitiesDescriptor. */
+    /**
+     * Test that the shortest duration (1 hour) is pulled up to the EntitiesDescriptor.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testPullCacheDuration() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -70,7 +74,11 @@ public class PullUpCacheDurationStageTest extends BaseDOMTest {
         }
     }
 
-    /** Test that the minimum cache duration is used when the shortest duration is less than it. */
+    /**
+     * Test that the minimum cache duration is used when the shortest duration is less than it.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testMinCacheDuration() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -92,7 +100,11 @@ public class PullUpCacheDurationStageTest extends BaseDOMTest {
         Assert.assertEquals(duration, Duration.ofHours(2));
     }
 
-    /** Test that the maximum cache duration is used when the shortest duration is greater than it. */
+    /**
+     * Test that the maximum cache duration is used when the shortest duration is greater than it.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testMaxCacheDuration() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -113,4 +125,4 @@ public class PullUpCacheDurationStageTest extends BaseDOMTest {
         final var duration = AttributeSupport.getDurationAttributeValue(durationAttr);
         Assert.assertEquals(duration, Duration.ofMinutes(30));
     }
-}
\ No newline at end of file
+}
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/PullUpValidUntilStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/PullUpValidUntilStageTest.java
index ada0260..7944bed 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/PullUpValidUntilStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/PullUpValidUntilStageTest.java
@@ -41,7 +41,11 @@ public class PullUpValidUntilStageTest extends BaseDOMTest {
         super(PullUpValidUntilStage.class);
     }
 
-    /** Test that the nearest validUntil is pulled up to the EntitiesDescriptor. */
+    /**
+     * Test that the nearest validUntil is pulled up to the EntitiesDescriptor.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testPullCacheDuration() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -71,7 +75,11 @@ public class PullUpValidUntilStageTest extends BaseDOMTest {
         }
     }
 
-    /** Test that the minimum validUntil is used when the nearest validUntil is earlier than min duration + now. */
+    /**
+     * Test that the minimum validUntil is used when the nearest validUntil is earlier than min duration + now.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testMinCacheDuration() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -95,7 +103,11 @@ public class PullUpValidUntilStageTest extends BaseDOMTest {
         Assert.assertTrue(validUntil.isAfter(Instant.now().plus(hundredYears).minus(Duration.ofMinutes(1))));
     }
 
-    /** Test that the maximum validUntil is used when the nearest validUntil is later than max duration + now. */
+    /**
+     * Test that the maximum validUntil is used when the nearest validUntil is later than max duration + now.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testMaxCacheDuration() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -122,4 +134,4 @@ public class PullUpValidUntilStageTest extends BaseDOMTest {
         Assert.assertTrue(validUntil.isBefore(twoYearsFromNow.plus(delta)));
         Assert.assertTrue(validUntil.isAfter(twoYearsFromNow.minus(delta)));
     }
-}
\ No newline at end of file
+}
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/RemoveOrganizationStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/RemoveOrganizationStageTest.java
index 6e0ccd2..9d714b3 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/RemoveOrganizationStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/RemoveOrganizationStageTest.java
@@ -37,7 +37,11 @@ public class RemoveOrganizationStageTest extends BaseDOMTest {
         super(RemoveOrganizationStage.class);
     }
 
-    /** Test the organization elements are removed from top level metadata elements. */
+    /**
+     * Test the organization elements are removed from top level metadata elements.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testRemoveOrganization() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -66,7 +70,11 @@ public class RemoveOrganizationStageTest extends BaseDOMTest {
         }
     }
 
-    /** Test that contact person elements are removed from children of top level metadata elements. */
+    /**
+     * Test that contact person elements are removed from children of top level metadata elements.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testRemoveOrganizationFromNestedElements() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
@@ -92,4 +100,4 @@ public class RemoveOrganizationStageTest extends BaseDOMTest {
                     "Organization").isEmpty());
         }
     }
-}
\ No newline at end of file
+}
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/ValidateValidUntilStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/ValidateValidUntilStageTest.java
index db55efe..89b16cc 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/ValidateValidUntilStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/ValidateValidUntilStageTest.java
@@ -39,7 +39,11 @@ public class ValidateValidUntilStageTest extends BaseDOMTest {
         super(ValidateValidUntilStage.class);
     }
 
-    /** Tests that optional check flag on the stage operates properly. */
+    /**
+     * Tests that optional check flag on the stage operates properly.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testOptionalCheck() throws Exception {
         ValidateValidUntilStage stage = new ValidateValidUntilStage();
@@ -61,7 +65,11 @@ public class ValidateValidUntilStageTest extends BaseDOMTest {
         Assert.assertTrue(item.getItemMetadata().containsKey(ErrorStatus.class));
     }
 
-    /** Tests that Items within range are not marked as in error and Items outside range are marked as in error. */
+    /**
+     * Tests that Items within range are not marked as in error and Items outside range are marked as in error.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testValidUntilRangeCheck() throws Exception {
         ValidateValidUntilStage stage = new ValidateValidUntilStage();
@@ -89,6 +97,8 @@ public class ValidateValidUntilStageTest extends BaseDOMTest {
      *            attribute
      * 
      * @return the created Item
+     * 
+     * @throws Exception if something bad happens
      */
     private DOMElementItem buildDomElementItem(@Nonnull final Duration validUntilInterval) throws Exception {
         Element descriptor = readXMLData("in.xml");
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/mdrpi/EntityRegistrationAuthorityFilterStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/mdrpi/EntityRegistrationAuthorityFilterStageTest.java
index 4193e5b..c4527ce 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/mdrpi/EntityRegistrationAuthorityFilterStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/dom/saml/mdrpi/EntityRegistrationAuthorityFilterStageTest.java
@@ -40,8 +40,13 @@ public class EntityRegistrationAuthorityFilterStageTest extends BaseDOMTest {
         super(EntityRegistrationAuthorityFilterStage.class);
     }
 
-    /** Tests filtering out Items based on an authority whitelist. */
-    @Test public void testAuthorityWhitelist() throws Exception {
+    /**
+     * Tests filtering out Items based on an authority whitelist.
+     * 
+     * @throws Exception if something bad happens
+     */
+    @Test
+    public void testAuthorityWhitelist() throws Exception {
         EntityRegistrationAuthorityFilterStage stage = new EntityRegistrationAuthorityFilterStage();
         stage.setId("test");
         stage.setRequiringRegistrationInformation(false);
@@ -56,8 +61,13 @@ public class EntityRegistrationAuthorityFilterStageTest extends BaseDOMTest {
         Assert.assertEquals(mdCollection.size(), 2);
     }
 
-    /** Tests filtering out Items based on an authority blacklist. */
-    @Test public void testAuthorityBlacklist() throws Exception {
+    /**
+     * Tests filtering out Items based on an authority blacklist.
+     * 
+     * @throws Exception if something bad happens
+     */
+    @Test
+    public void testAuthorityBlacklist() throws Exception {
         EntityRegistrationAuthorityFilterStage stage = new EntityRegistrationAuthorityFilterStage();
         stage.setId("test");
         stage.setWhitelistingRegistrationAuthorities(false);
@@ -72,8 +82,13 @@ public class EntityRegistrationAuthorityFilterStageTest extends BaseDOMTest {
         Assert.assertEquals(mdCollection.size(), 1);
     }
 
-    /** Tests filtering out Items that do not contain registration information. */
-    @Test public void testRequireRegistrationInfo() throws Exception {
+    /**
+     * Tests filtering out Items that do not contain registration information.
+     * 
+     * @throws Exception if something bad happens
+     */
+    @Test
+    public void testRequireRegistrationInfo() throws Exception {
         EntityRegistrationAuthorityFilterStage stage = new EntityRegistrationAuthorityFilterStage();
         stage.setId("test");
         stage.setRequiringRegistrationInformation(true);
@@ -90,8 +105,13 @@ public class EntityRegistrationAuthorityFilterStageTest extends BaseDOMTest {
         Assert.assertEquals(mdCollection.size(), 0);
     }
 
-    /** Tests removing EntitiesDescriptors that no longer contain any EntityDescriptors. */
-    @Test public void testRemoveEntitylessEntitiesDescriptor() throws Exception {
+    /**
+     * Tests removing EntitiesDescriptors that no longer contain any EntityDescriptors.
+     * 
+     * @throws Exception if something bad happens
+     */
+    @Test
+    public void testRemoveEntitylessEntitiesDescriptor() throws Exception {
         EntityRegistrationAuthorityFilterStage stage = new EntityRegistrationAuthorityFilterStage();
         stage.setId("test");
         stage.setRequiringRegistrationInformation(true);
@@ -109,8 +129,13 @@ public class EntityRegistrationAuthorityFilterStageTest extends BaseDOMTest {
         Assert.assertEquals(mdCollection.size(), 0);
     }
 
-    /** Tests not removing EntitiesDescriptors that no longer contain any EntityDescriptors. */
-    @Test public void testDontRemoveEntitylessEntitiesDescriptor() throws Exception {
+    /**
+     * Tests not removing EntitiesDescriptors that no longer contain any EntityDescriptors.
+     * 
+     * @throws Exception if something bad happens
+     */
+    @Test
+    public void testDontRemoveEntitylessEntitiesDescriptor() throws Exception {
         EntityRegistrationAuthorityFilterStage stage = new EntityRegistrationAuthorityFilterStage();
         stage.setId("test");
         stage.setRemovingEntitylessEntitiesDescriptor(false);
@@ -129,7 +154,13 @@ public class EntityRegistrationAuthorityFilterStageTest extends BaseDOMTest {
         Assert.assertEquals(ElementSupport.getChildElements(mdCollection.get(0).unwrap()).size(), 0);
     }
 
-    /** Build up a metadata collection containing 3 EntityDescriptors. */
+    /**
+     * Build up a metadata collection containing three EntityDescriptors.
+     * 
+     * @return metadata collection
+     * 
+     * @throws Exception if something bad happens
+     */
     private Collection<Item<Element>> buildMetadataCollection() throws Exception {
         final ArrayList<Item<Element>> metadataCollection = new ArrayList<>();
 
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/CountingStage.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/CountingStage.java
index 2a02de2..0ad8f09 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/CountingStage.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/CountingStage.java
@@ -21,7 +21,11 @@ import java.util.Collection;
 
 import net.shibboleth.metadata.Item;
 
-/** Stage that counts the number of times it was invoked and the number of items it has observed. */
+/**
+ * Stage that counts the number of times it was invoked and the number of items it has observed.
+ * 
+ * @param <T> type of {@link Item} being processed
+ */
 public class CountingStage<T> extends AbstractStage<T> {
 
     /** Number of times the stage was invoked. */
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemMetadataFilterStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemMetadataFilterStageTest.java
index 4c181e5..27592af 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemMetadataFilterStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemMetadataFilterStageTest.java
@@ -70,7 +70,11 @@ public class ItemMetadataFilterStageTest {
         metadataCollectionPrototype.add(md4);
     }
 
-    /** Tests a {@link ItemMetadataFilterStage} without any filter requirements. */
+    /**
+     * Tests a {@link ItemMetadataFilterStage} without any filter requirements.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testNoFilterRequirements() throws Exception {
         final Collection<Item<String>> metadataCollection = new ArrayList<>(metadataCollectionPrototype);
@@ -87,7 +91,11 @@ public class ItemMetadataFilterStageTest {
         Assert.assertTrue(metadataCollection.contains(md4));
     }
 
-    /** Tests a {@link ItemMetadataFilterStage} containing one filter requirement. */
+    /**
+     * Tests a {@link ItemMetadataFilterStage} containing one filter requirement.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testSingleFilterRequirement() throws Exception {
         final Collection<Item<String>> metadataCollection = new ArrayList<>(metadataCollectionPrototype);
@@ -108,7 +116,11 @@ public class ItemMetadataFilterStageTest {
         Assert.assertFalse(metadataCollection.contains(md4));
     }
 
-    /** Tests a {@link ItemMetadataFilterStage} containing multiple filter requirements. */
+    /**
+     * Tests a {@link ItemMetadataFilterStage} containing multiple filter requirements.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testMultiFilterRequirement() throws Exception {
         final Collection<Item<String>> metadataCollection = new ArrayList<>(metadataCollectionPrototype);
@@ -130,4 +142,4 @@ public class ItemMetadataFilterStageTest {
         Assert.assertTrue(metadataCollection.contains(md3));
         Assert.assertFalse(metadataCollection.contains(md4));
     }
-}
\ No newline at end of file
+}
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemMetadataTerminationStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemMetadataTerminationStageTest.java
index 58cb97a..0e48bc1 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemMetadataTerminationStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/ItemMetadataTerminationStageTest.java
@@ -70,7 +70,11 @@ public class ItemMetadataTerminationStageTest {
         metadataCollectionPrototype.add(md4);
     }
 
-    /** Tests a {@link ItemMetadataFilterStage} without any selection requirements. */
+    /**
+     * Tests a {@link ItemMetadataFilterStage} without any selection requirements.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testNoSelectionRequirements() throws Exception {
         final Collection<Item<String>> metadataCollection = new ArrayList<>(metadataCollectionPrototype);
@@ -91,7 +95,11 @@ public class ItemMetadataTerminationStageTest {
 
     }
 
-    /** Tests a {@link ItemMetadataFilterStage} containing one selection requirement. */
+    /**
+     * Tests a {@link ItemMetadataFilterStage} containing one selection requirement.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testSingleSelectionRequirement() throws Exception {
         final Collection<Item<String>> metadataCollection = new ArrayList<>(metadataCollectionPrototype);
@@ -111,7 +119,11 @@ public class ItemMetadataTerminationStageTest {
         }
     }
 
-    /** Tests a {@link ItemMetadataFilterStage} containing multiple selection requirements. */
+    /**
+     * Tests a {@link ItemMetadataFilterStage} containing multiple selection requirements.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test
     public void testMultiSelectionRequirement() throws Exception {
         final Collection<Item<String>> metadataCollection = new ArrayList<>(metadataCollectionPrototype);
@@ -132,4 +144,4 @@ public class ItemMetadataTerminationStageTest {
             // expected this
         }
     }
-}
\ No newline at end of file
+}
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/PipelineMergeStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/PipelineMergeStageTest.java
index 8aa53da..a39117a 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/PipelineMergeStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/PipelineMergeStageTest.java
@@ -189,8 +189,13 @@ public class PipelineMergeStageTest {
         Assert.assertEquals(target.size(), 6);
     }
     
-    /** Tests the case where one of the pipelines throws an exception. */
-    @Test public void testThrow() throws Exception {
+    /**
+     * Tests the case where one of the pipelines throws an exception.
+     * 
+     * @throws Exception if something bad happens
+     */
+    @Test
+    public void testThrow() throws Exception {
         Item<String> md1 = new MockItem("one");
         StaticItemSourceStage<String> source1 = new StaticItemSourceStage<>();
         source1.setId("src1");
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/SplitMergeStageTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/SplitMergeStageTest.java
index 6b0d97a..f62b571 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/SplitMergeStageTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/SplitMergeStageTest.java
@@ -152,7 +152,11 @@ public class SplitMergeStageTest {
         Assert.assertTrue(items.contains(item3));
     }
     
-    /** Tests case where the selected items pipeline throws a {@link TerminationException}. */
+    /**
+     * Tests case where the selected items pipeline throws a {@link TerminationException}.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test public void testThrowSelected() throws Exception {
         final SimplePipeline<String> selectedPipeline = new SimplePipeline<>();
         selectedPipeline.setId("selectedPipeline");
@@ -187,7 +191,11 @@ public class SplitMergeStageTest {
         }
     }
     
-    /** Tests case where the nonselected items pipeline throws a {@link TerminationException}. */
+    /**
+     * Tests case where the nonselected items pipeline throws a {@link TerminationException}.
+     * 
+     * @throws Exception if something bad happens
+     */
     @Test public void testThrowNonselected() throws Exception {
         final SimplePipeline<String> selectedPipeline = new SimplePipeline<>();
         selectedPipeline.setId("selectedPipeline");
@@ -283,4 +291,4 @@ public class SplitMergeStageTest {
         stage.execute(items);
         Assert.assertEquals(3, items.size());
     }
-}
\ No newline at end of file
+}
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/TerminatingStage.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/TerminatingStage.java
index d8d0acd..d08f9d3 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/TerminatingStage.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/pipeline/TerminatingStage.java
@@ -21,7 +21,11 @@ import java.util.Collection;
 
 import net.shibboleth.metadata.Item;
 
-/** Stage that throws a TerminationException when it is called. */
+/**
+ * Stage that throws a {@link TerminationException} when it is called.
+ * 
+ * @param <T> type of {@link Item} being processed
+ */
 class TerminatingStage<T> extends AbstractStage<T> {
 
     /** Constructor. */
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509ROCAValidatorTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509ROCAValidatorTest.java
index 776ad93..4986ab0 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509ROCAValidatorTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509ROCAValidatorTest.java
@@ -12,7 +12,6 @@ import org.testng.annotations.Test;
 
 public class X509ROCAValidatorTest extends BaseX509ValidatorTest {
 
-    /** Constructor sets class under test. */
     public X509ROCAValidatorTest() throws Exception {
         super(X509ROCAValidator.class);
     }
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAExponentValidatorTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAExponentValidatorTest.java
index afd8b9c..ed999d4 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAExponentValidatorTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAExponentValidatorTest.java
@@ -29,7 +29,6 @@ import org.testng.annotations.Test;
 
 public class X509RSAExponentValidatorTest extends BaseX509ValidatorTest {
     
-    /** Constructor sets class under test. */
     public X509RSAExponentValidatorTest() throws Exception {
         super(X509RSAExponentValidator.class);
     }
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAKeyLengthValidatorTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAKeyLengthValidatorTest.java
index 34a58ac..7518866 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAKeyLengthValidatorTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAKeyLengthValidatorTest.java
@@ -29,7 +29,6 @@ import org.testng.annotations.Test;
 
 public class X509RSAKeyLengthValidatorTest extends BaseX509ValidatorTest {
     
-    /** Constructor sets class under test. */
     public X509RSAKeyLengthValidatorTest() throws Exception {
         super(X509RSAKeyLengthValidator.class);
     }
diff --git a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAOpenSSLBlacklistValidatorTest.java b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAOpenSSLBlacklistValidatorTest.java
index 75104b4..2a07ddf 100644
--- a/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAOpenSSLBlacklistValidatorTest.java
+++ b/aggregator-pipeline/src/test/java/net/shibboleth/metadata/validate/x509/X509RSAOpenSSLBlacklistValidatorTest.java
@@ -29,7 +29,6 @@ import net.shibboleth.metadata.validate.Validator;
 
 public class X509RSAOpenSSLBlacklistValidatorTest extends BaseX509ValidatorTest {
     
-    /** Constructor sets class under test. */
     public X509RSAOpenSSLBlacklistValidatorTest() throws Exception {
         super(X509RSAOpenSSLBlacklistValidator.class);
     }

-- 
To stop receiving notification emails like this one, please contact
the administrator of this repository.


More information about the commits mailing list