RAT-204: Unify resource handling.
* Module apache-rat-tasks
** Replace indexOf() with contains.
** Use IOUtils.closeQuietly in finally blocks.
git-svn-id: https://svn.apache.org/repos/asf/creadur/rat/trunk@1682561 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/apache-rat-tasks/src/main/java/org/apache/rat/anttasks/Report.java b/apache-rat-tasks/src/main/java/org/apache/rat/anttasks/Report.java
index b074389..dc9f5b0 100644
--- a/apache-rat-tasks/src/main/java/org/apache/rat/anttasks/Report.java
+++ b/apache-rat-tasks/src/main/java/org/apache/rat/anttasks/Report.java
@@ -18,6 +18,7 @@
*/
package org.apache.rat.anttasks;
+ import org.apache.commons.io.IOUtils;
import org.apache.rat.Defaults;
import org.apache.rat.ReportConfiguration;
import org.apache.rat.analysis.IHeaderMatcher;
@@ -205,9 +206,7 @@
} catch (RatException e) {
throw new BuildException(e);
} finally {
- if (reportFile != null) {
- FileUtils.close(out);
- }
+ IOUtils.closeQuietly(out);
}
}
diff --git a/apache-rat-tasks/src/test/java/org/apache/rat/anttasks/AbstractRatAntTaskTest.java b/apache-rat-tasks/src/test/java/org/apache/rat/anttasks/AbstractRatAntTaskTest.java
index 4ed9aa9..d40b271 100644
--- a/apache-rat-tasks/src/test/java/org/apache/rat/anttasks/AbstractRatAntTaskTest.java
+++ b/apache-rat-tasks/src/test/java/org/apache/rat/anttasks/AbstractRatAntTaskTest.java
@@ -16,9 +16,9 @@
*/
package org.apache.rat.anttasks;
-import org.junit.Assert;
+import org.apache.commons.io.IOUtils;
import org.apache.tools.ant.BuildFileTest;
-import org.apache.tools.ant.util.FileUtils;
+import org.junit.Assert;
import java.io.File;
import java.io.FileReader;
@@ -39,7 +39,7 @@
configureProject(getAntFile().getPath());
}
- protected void assertLogDoesntMatch(String pPattern) {
+ protected void assertLogDoesNotMatch(String pPattern) {
final String log = super.getLog();
Assert.assertFalse("Log matches the pattern: " + pPattern + ", got " + log,
isMatching(pPattern, log));
@@ -72,7 +72,7 @@
}
}
} finally {
- FileUtils.close(fr);
+ IOUtils.closeQuietly(fr);
}
}
diff --git a/apache-rat-tasks/src/test/java/org/apache/rat/anttasks/ReportTest.java b/apache-rat-tasks/src/test/java/org/apache/rat/anttasks/ReportTest.java
index 7ca437d..fb9518b 100644
--- a/apache-rat-tasks/src/test/java/org/apache/rat/anttasks/ReportTest.java
+++ b/apache-rat-tasks/src/test/java/org/apache/rat/anttasks/ReportTest.java
@@ -16,16 +16,16 @@
*/
package org.apache.rat.anttasks;
+import org.apache.commons.io.IOUtils;
+import org.apache.tools.ant.BuildException;
+import org.junit.Assert;
+
import java.io.BufferedReader;
import java.io.File;
import java.io.FileInputStream;
import java.io.IOException;
import java.io.InputStreamReader;
-import org.apache.tools.ant.BuildException;
-
-import org.junit.Assert;
-
public class ReportTest extends AbstractRatAntTaskTest {
private static final File antFile = new File("src/test/resources/antunit/report-junit.xml").getAbsoluteFile();
@@ -41,28 +41,28 @@
public void testWithReportSentToFile() throws Exception {
final File reportFile = new File(getTempDir(), "selftest.report");
- if(!getTempDir().mkdirs() && !getTempDir().isDirectory()) {
+ if (!getTempDir().mkdirs() && !getTempDir().isDirectory()) {
throw new IOException("Could not create temporary directory " + getTempDir());
}
final String alLine = "AL +\\Q" + getAntFileName() + "\\E";
- if (reportFile.isFile() && !reportFile.delete()) {
+ if (reportFile.isFile() && !reportFile.delete()) {
throw new IOException("Unable to remove report file " + reportFile);
}
executeTarget("testWithReportSentToFile");
- assertLogDoesntMatch(alLine);
+ assertLogDoesNotMatch(alLine);
Assert.assertTrue("Expected report file " + reportFile, reportFile.isFile());
assertFileMatches(reportFile, alLine);
}
public void testWithALUnknown() throws Exception {
executeTarget("testWithALUnknown");
- assertLogDoesntMatch("AL +\\Q" + getAntFileName() + "\\E");
+ assertLogDoesNotMatch("AL +\\Q" + getAntFileName() + "\\E");
assertLogMatches("\\!\\?\\?\\?\\?\\? +\\Q" + getAntFileName() + "\\E");
}
public void testCustomMatcher() throws Exception {
executeTarget("testCustomMatcher");
- assertLogDoesntMatch("AL +\\Q" + getAntFileName() + "\\E");
+ assertLogDoesNotMatch("AL +\\Q" + getAntFileName() + "\\E");
assertLogMatches("EXMPL +\\Q" + getAntFileName() + "\\E");
}
@@ -73,7 +73,7 @@
} catch (BuildException e) {
final String expect = "You must specify at least one file";
assertTrue("Expected " + expect + ", got " + e.getMessage(),
- e.getMessage().indexOf(expect) != -1);
+ e.getMessage().contains(expect));
}
}
@@ -84,7 +84,7 @@
} catch (BuildException e) {
final String expect = "at least one license";
assertTrue("Expected " + expect + ", got " + e.getMessage(),
- e.getMessage().indexOf(expect) != -1);
+ e.getMessage().contains(expect));
}
}
@@ -93,12 +93,21 @@
}
private String getFirstLine(File pFile) throws IOException {
- final FileInputStream fis = new FileInputStream(pFile);
- final InputStreamReader reader = new InputStreamReader(fis, "UTF8");
- final BufferedReader breader = new BufferedReader(reader);
- final String result = breader.readLine();
- breader.close();
- return result;
+ FileInputStream fis = null;
+ InputStreamReader reader = null;
+ BufferedReader breader = null;
+ try {
+ fis = new FileInputStream(pFile);
+ reader = new InputStreamReader(fis, "UTF8");
+ breader = new BufferedReader(reader);
+ final String result = breader.readLine();
+ breader.close();
+ return result;
+ } finally {
+ IOUtils.closeQuietly(fis);
+ IOUtils.closeQuietly(reader);
+ IOUtils.closeQuietly(breader);
+ }
}
public void testAddLicenseHeaders() throws Exception {
@@ -106,11 +115,11 @@
final File origFile = new File("target/anttasks/it-sources/index.apt");
final String origFirstLine = getFirstLine(origFile);
- assertTrue(origFirstLine.indexOf("--") != -1);
- assertTrue(origFirstLine.indexOf("~~") == -1);
+ assertTrue(origFirstLine.contains("--"));
+ assertFalse(origFirstLine.contains("~~"));
final File modifiedFile = new File("target/anttasks/it-sources/index.apt.new");
final String modifiedFirstLine = getFirstLine(modifiedFile);
- assertTrue(modifiedFirstLine.indexOf("--") == -1);
- assertTrue(modifiedFirstLine.indexOf("~~") != -1);
+ assertFalse(modifiedFirstLine.contains("--"));
+ assertTrue(modifiedFirstLine.contains("~~"));
}
}