Remove trailing whitespace in JUnit assertion messages

garydgregory · garydgregory · commit 430519e46be5 · 2024-10-06T09:47:44.000-04:00
diff --git a/src/changes/changes.xml b/src/changes/changes.xml
@@ -48,10 +48,11 @@ The <action> type attribute can be add,update,fix,remove.
   <body>
     <release version="2.17.1" date="YYYY-MM-DD" description="Version 2.17.1: Java 8 is required.">
       <!-- FIX -->
+      <action dev="ggregory" type="fix"                due-to="Gary Gregory">Clean ups in unit tests.</action>
       <!-- ADD -->
       <!-- UPDATE -->
-      <action dev="ggregory" type="update"                due-to="Gary Gregory">Bump org.apache.commons:commons-parent from 74 to 76 #670, #676.</action>
-      <action dev="ggregory" type="update"                due-to="Gary Gregory">Bump commons.bytebuddy.version from 1.15.1 to 1.15.3 #672, #673.</action> 
+      <action dev="ggregory" type="update"             due-to="Gary Gregory">Bump org.apache.commons:commons-parent from 74 to 76 #670, #676.</action>
+      <action dev="ggregory" type="update"             due-to="Gary Gregory">Bump commons.bytebuddy.version from 1.15.1 to 1.15.3 #672, #673.</action> 
     </release>
     <release version="2.17.0" date="2024-09-15" description="Version 2.17.0: Java 8 is required.">
       <!-- ADD --> 
diff --git a/src/test/java/org/apache/commons/io/input/SwappedDataInputStreamTest.java b/src/test/java/org/apache/commons/io/input/SwappedDataInputStreamTest.java
@@ -114,8 +114,7 @@ public void testReadInt() throws IOException {
 
     @Test
     public void testReadLine() {
-        assertThrows(UnsupportedOperationException.class, () ->  this.sdis.readLine(),
-                "readLine should be unsupported. ");
+        assertThrows(UnsupportedOperationException.class, () -> this.sdis.readLine(), "readLine should be unsupported.");
     }
 
     @Test
@@ -140,7 +139,7 @@ public void testReadUnsignedShort() throws IOException {
 
     @Test
     public void testReadUTF() {
-        assertThrows(UnsupportedOperationException.class, () -> this.sdis.readUTF(), "readUTF should be unsupported. ");
+        assertThrows(UnsupportedOperationException.class, () -> this.sdis.readUTF(), "readUTF should be unsupported.");
     }
 
     @Test
diff --git a/src/test/java/org/apache/commons/io/output/LockableFileWriterTest.java b/src/test/java/org/apache/commons/io/output/LockableFileWriterTest.java
@@ -89,10 +89,10 @@ private void testAlternateLockDir(final LockableFileWriter lfw1) {
 
         // try to open a second writer
         try (LockableFileWriter lfw2 = new LockableFileWriter(file, StandardCharsets.UTF_8, true, altLockDir.getAbsolutePath())) {
-            fail("Somehow able to open a locked file. ");
+            fail("Somehow able to open a locked file.");
         } catch (final IOException ioe) {
             final String msg = ioe.getMessage();
-            assertTrue(msg.startsWith("Can't write file, lock "), "Exception message does not start correctly. ");
+            assertTrue(msg.startsWith("Can't write file, lock "), "Exception message does not start correctly.");
             assertTrue(file.exists());
             assertTrue(altLockFile.exists());
         }
@@ -163,20 +163,20 @@ public void testFileLocked() throws IOException {
 
             // try to open a second writer
             try (LockableFileWriter lfw2 = new LockableFileWriter(file)) {
-                fail("Somehow able to open a locked file. ");
+                fail("Somehow able to open a locked file.");
             } catch (final IOException ioe) {
                 final String msg = ioe.getMessage();
-                assertTrue(msg.startsWith("Can't write file, lock "), "Exception message does not start correctly. ");
+                assertTrue(msg.startsWith("Can't write file, lock "), "Exception message does not start correctly.");
                 assertTrue(file.exists());
                 assertTrue(lockFile.exists());
             }
 
             // try to open a third writer
             try (LockableFileWriter lfw3 = new LockableFileWriter(file)) {
-                fail("Somehow able to open a locked file. ");
+                fail("Somehow able to open a locked file.");
             } catch (final IOException ioe) {
                 final String msg = ioe.getMessage();
-                assertTrue(msg.startsWith("Can't write file, lock "), "Exception message does not start correctly. ");
+                assertTrue(msg.startsWith("Can't write file, lock "), "Exception message does not start correctly.");
                 assertTrue(file.exists());
                 assertTrue(lockFile.exists());
             }

Original file line number	Diff line number	Diff line change
`@@ -114,8 +114,7 @@ public void testReadInt() throws IOException {`
`114`	`114`
`115`	`115`	`@Test`
`116`	`116`	`public void testReadLine() {`
`117`		`- assertThrows(UnsupportedOperationException.class, () -> this.sdis.readLine(),`
`118`		`- "readLine should be unsupported. ");`
	`117`	`+ assertThrows(UnsupportedOperationException.class, () -> this.sdis.readLine(), "readLine should be unsupported.");`
`119`	`118`	`}`
`120`	`119`
`121`	`120`	`@Test`
`@@ -140,7 +139,7 @@ public void testReadUnsignedShort() throws IOException {`
`140`	`139`
`141`	`140`	`@Test`
`142`	`141`	`public void testReadUTF() {`
`143`		`- assertThrows(UnsupportedOperationException.class, () -> this.sdis.readUTF(), "readUTF should be unsupported. ");`
	`142`	`+ assertThrows(UnsupportedOperationException.class, () -> this.sdis.readUTF(), "readUTF should be unsupported.");`
`144`	`143`	`}`
`145`	`144`
`146`	`145`	`@Test`