commit 722a5af4f1786599f22e91a34af459883c7a6a8b
author pascalschumacher <pascalschumacher@gmx.net> Sun Apr 23 17:12:14 2017 +0200
committer pascalschumacher <pascalschumacher@gmx.net> Sun Apr 23 17:12:14 2017 +0200
tree de5acf23d76c60e8c7cfa71cd82dc4ec46ee2c9b
parent f5c46b9fc80754c601205e6ef13c5b7e9a5003e2

TrailerTest: replace usages of IOUtils#closeQuietly with try-with-resources statements

(side effects: close #7, close #21, close #20)

src/test/java/org/apache/commons/io/input/TailerTest.java

diff --git a/src/test/java/org/apache/commons/io/input/TailerTest.java b/src/test/java/org/apache/commons/io/input/TailerTest.java
index 5818f26..df16f45 100644
--- a/src/test/java/org/apache/commons/io/input/TailerTest.java
+++ b/src/test/java/org/apache/commons/io/input/TailerTest.java
@@ -125,11 +125,9 @@
         final Thread thread = new Thread(tailer);
         thread.start();
 
-        Writer out = new OutputStreamWriter(new FileOutputStream(file), charsetUTF8);
-        BufferedReader reader = null;
-        try{
+        try (Writer out = new OutputStreamWriter(new FileOutputStream(file), charsetUTF8);
+             BufferedReader reader = new BufferedReader(new InputStreamReader(new FileInputStream(origin), charsetUTF8))) {
             List<String> lines = new ArrayList<>();
-            reader = new BufferedReader(new InputStreamReader(new FileInputStream(origin), charsetUTF8));
             String line;
             while((line = reader.readLine()) != null){
                 out.write(line);
@@ -151,10 +149,8 @@
                            + "\nAct: (" + actual.length() + ") "+ actual);
                }
            }
-        }finally{
+        } finally{
             tailer.stop();
-            IOUtils.closeQuietly(reader);
-            IOUtils.closeQuietly(out);
         }
     }
 
@@ -170,7 +166,6 @@
         thread.start();
 
         // Write some lines to the file
-        final FileWriter writer = null;
         try {
             writeString(file, "Line");
 
@@ -189,7 +184,6 @@
         } finally {
             tailer.stop();
             TestUtils.sleep(delay * 2);
-            IOUtils.closeQuietly(writer);
         }
     }
 
@@ -298,12 +292,9 @@
             throw new IOException("Cannot create file " + file
                     + " as the parent directory does not exist");
         }
-        final BufferedOutputStream output =
-                new BufferedOutputStream(new FileOutputStream(file));
-        try {
+        try (final BufferedOutputStream output =
+                new BufferedOutputStream(new FileOutputStream(file))) {
             TestUtils.generateTestData(output, size);
-        } finally {
-            IOUtils.closeQuietly(output);
         }
 
         // try to make sure file is found
@@ -328,31 +319,22 @@
 
     /** Append some lines to a file */
     private void write(final File file, final String... lines) throws Exception {
-        FileWriter writer = null;
-        try {
-            writer = new FileWriter(file, true);
+        try (FileWriter writer = new FileWriter(file, true)) {
             for (final String line : lines) {
                 writer.write(line + "\n");
             }
-        } finally {
-            IOUtils.closeQuietly(writer);
         }
     }
 
     /** Append a string to a file */
     private void writeString(final File file, final String ... strings) throws Exception {
-        FileWriter writer = null;
-        try {
-            writer = new FileWriter(file, true);
+        try (FileWriter writer = new FileWriter(file, true)) {
             for (final String string : strings) {
                 writer.write(string);
             }
-        } finally {
-            IOUtils.closeQuietly(writer);
         }
     }
 
-
     @Test
     public void testStopWithNoFile() throws Exception {
         final File file = new File(getTestDirectory(),"nosuchfile");