Some others errors in cdk-test

johnmay · egonw · commit 2a1b1032ee0f · 2022-01-06T22:08:33.000+01:00
diff --git a/base/test/src/main/java/org/openscience/cdk/CDKTestCase.java b/base/test/src/main/java/org/openscience/cdk/CDKTestCase.java
@@ -251,7 +251,7 @@ protected void assertHydrogenCounts(int[] hydrogenCounts, IAtomContainer contain
     /**
      * Asserts that the given String has zero length.
      *
-     * @param String String to test the length of.
+     * @param testString String to test the length of.
      */
     public void assertZeroLength(String testString) {
         Assert.assertNotNull("Expected a non-null String.", testString);
@@ -262,7 +262,7 @@ public void assertZeroLength(String testString) {
      * Asserts that the given String consists of a single line, and thus
      * does not contain any '\r' and/or '\n' characters.
      *
-     * @param String String to test.
+     * @param testString String to test.
      */
     public void assertOneLiner(String testString) {
         Assert.assertNotNull("Expected a non-null String.", testString);
@@ -274,8 +274,8 @@ public void assertOneLiner(String testString) {
     }
 
     /**
-     * This test allows people to use the {@link TestMethod} annotation for
-     * methods that are testing in other classes than identified with {@link TestClass}.
+     * This test allows people to use the {@code TestMethod} annotation for
+     * methods that are testing in other classes than identified with {@code TestClass}.
      * Bit of a workaround for the current set up, but useful in situations where
      * a methods is rather untestable, such as SAXHandler's endElement() methods.
      *
diff --git a/base/test/src/main/java/org/openscience/cdk/interfaces/AbstractPseudoAtomTest.java b/base/test/src/main/java/org/openscience/cdk/interfaces/AbstractPseudoAtomTest.java
@@ -142,7 +142,7 @@ public void testToString() {
      * Test for bug #1778479 "MDLWriter writes empty PseudoAtom label string".
      * We decided to let the pseudo atoms have a default label of '*'.
      *
-     * Author: Andreas Schueller <a.schueller@chemie.uni-frankfurt.de>
+     * Author: Andreas Schueller &lt;a.schueller@chemie.uni-frankfurt.de&gt;
      *
      * @cdk.bug 1778479
      */
diff --git a/base/test/src/main/java/org/openscience/cdk/interfaces/AbstractRingSetTest.java b/base/test/src/main/java/org/openscience/cdk/interfaces/AbstractRingSetTest.java
@@ -312,7 +312,7 @@ public void testGetConnectedRings_IRing() {
      * Test for RingSetTest bug #1772613.
      * When using method getConnectedRings(...) of RingSet.java fused or bridged rings
      * returned a list of connected rings that contained duplicates.
-     * Bug fix by Andreas Schueller <a.schueller@chemie.uni-frankfurt.de>
+     * Bug fix by Andreas Schueller &lt;a.schueller@chemie.uni-frankfurt.de&gt;
      * @cdk.bug 1772613
      */
     @Test
diff --git a/base/test/src/main/java/org/openscience/cdk/qsar/descriptors/DescriptorTest.java b/base/test/src/main/java/org/openscience/cdk/qsar/descriptors/DescriptorTest.java
@@ -57,16 +57,15 @@ public void setDescriptor(Class<? extends T> descriptorClass,
     /**
      * Makes sure that the extending class has set the super.descriptor.
      * Each extending class should have this bit of code (JUnit3 formalism):
-     * <pre>
+     * <pre>{@code
      * @Test public void setUp() {
      *   // Pass a Class, not an Object!
      *   setDescriptor(SomeDescriptor.class);
-     * }
+     * }}
+     * </pre>
      *
      * <p>The unit tests in the extending class may use this instance, but
      * are not required.
-     *
-     * </pre>
      */
     @Test
     public void testHasSetSuperDotDescriptor() {

Original file line number	Diff line number	Diff line change
`@@ -142,7 +142,7 @@ public void testToString() {`
`142`	`142`	`* Test for bug #1778479 "MDLWriter writes empty PseudoAtom label string".`
`143`	`143`	`* We decided to let the pseudo atoms have a default label of '*'.`
`144`	`144`	`*`
`145`		`- * Author: Andreas Schueller <a.schueller@chemie.uni-frankfurt.de>`
	`145`	`+ * Author: Andreas Schueller <a.schueller@chemie.uni-frankfurt.de>`
`146`	`146`	`*`
`147`	`147`	`* @cdk.bug 1778479`
`148`	`148`	`*/`