Skip to content

Commit 957ec89

Browse files
elharoslachiewicz
authored andcommitted
[MPMD-380] use Apache Commons FileUtils and JDK
1 parent 4f10a00 commit 957ec89

File tree

1 file changed

+16
-16
lines changed

1 file changed

+16
-16
lines changed

src/test/java/org/apache/maven/plugins/pmd/CpdReportTest.java

Lines changed: 16 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@
2727
import java.io.IOException;
2828
import java.util.Locale;
2929

30+
import org.apache.commons.io.FileUtils;
3031
import org.apache.commons.lang3.StringUtils;
31-
import org.codehaus.plexus.util.FileUtils;
3232
import org.w3c.dom.Document;
3333

3434
/**
@@ -54,11 +54,11 @@ protected void setUp() throws Exception {
5454
public void testDefaultConfiguration() throws Exception {
5555
File generatedReport =
5656
generateReport("cpd", "default-configuration/cpd-default-configuration-plugin-config.xml");
57-
assertTrue(FileUtils.fileExists(generatedReport.getAbsolutePath()));
57+
assertTrue(new File(generatedReport.getAbsolutePath()).exists());
5858

5959
// check if the CPD files were generated
6060
File generatedFile = new File(getBasedir(), "target/test/unit/default-configuration/target/cpd.xml");
61-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
61+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
6262

6363
// check the contents of cpd.html
6464
String str = readFile(generatedReport);
@@ -82,9 +82,9 @@ public void testTxtFormat() throws Exception {
8282

8383
// check if the CPD files were generated
8484
File generatedFile = new File(getBasedir(), "target/test/unit/custom-configuration/target/cpd.xml");
85-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
85+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
8686
generatedFile = new File(getBasedir(), "target/test/unit/custom-configuration/target/cpd.txt");
87-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
87+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
8888

8989
// check the contents of cpd.txt
9090
String str = readFile(generatedFile);
@@ -101,11 +101,11 @@ public void testTxtFormat() throws Exception {
101101
*/
102102
public void testCustomConfiguration() throws Exception {
103103
File generatedReport = generateReport("cpd", "custom-configuration/cpd-custom-configuration-plugin-config.xml");
104-
assertTrue(FileUtils.fileExists(generatedReport.getAbsolutePath()));
104+
assertTrue(new File(generatedReport.getAbsolutePath()).exists());
105105

106106
// check if the CPD files were generated
107107
File generatedFile = new File(getBasedir(), "target/test/unit/custom-configuration/target/cpd.csv");
108-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
108+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
109109

110110
String str = readFile(generatedReport);
111111
// Contents that should NOT be in the report
@@ -162,7 +162,7 @@ public void testWriteNonHtml() throws Exception {
162162

163163
// check if the CPD files were generated
164164
File generatedFile = new File(getBasedir(), "target/test/unit/default-configuration/target/cpd.xml");
165-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
165+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
166166

167167
DocumentBuilder builder = DocumentBuilderFactory.newInstance().newDocumentBuilder();
168168
Document pmdCpdDocument = builder.parse(generatedFile);
@@ -183,7 +183,7 @@ public void testIncludeXmlInSite() throws Exception {
183183
generateReport("cpd", "default-configuration/cpd-report-include-xml-in-site-plugin-config.xml");
184184

185185
File generatedFile = new File(getBasedir(), "target/test/unit/default-configuration/target/cpd.xml");
186-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
186+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
187187

188188
DocumentBuilder builder = DocumentBuilderFactory.newInstance().newDocumentBuilder();
189189
Document pmdCpdDocument = builder.parse(generatedFile);
@@ -193,7 +193,7 @@ public void testIncludeXmlInSite() throws Exception {
193193
assertTrue(str.contains("</pmd-cpd>"));
194194

195195
File siteReport = new File(getBasedir(), "target/test/unit/default-configuration/target/site/cpd.xml");
196-
assertTrue(FileUtils.fileExists(siteReport.getAbsolutePath()));
196+
assertTrue(new File(siteReport.getAbsolutePath()).exists());
197197
String siteReportContent = readFile(siteReport);
198198
assertTrue(siteReportContent.contains("</pmd-cpd>"));
199199
assertEquals(str, siteReportContent);
@@ -202,13 +202,13 @@ public void testIncludeXmlInSite() throws Exception {
202202
public void testSkipEmptyReportConfiguration() throws Exception {
203203
// verify the generated files do not exist because PMD was skipped
204204
File generatedReport = generateReport("cpd", "empty-report/cpd-skip-empty-report-plugin-config.xml");
205-
assertFalse(FileUtils.fileExists(generatedReport.getAbsolutePath()));
205+
assertFalse(new File(generatedReport.getAbsolutePath()).exists());
206206
}
207207

208208
public void testEmptyReportConfiguration() throws Exception {
209209
// verify the generated files do exist, even if there are no violations
210210
File generatedReport = generateReport("cpd", "empty-report/cpd-empty-report-plugin-config.xml");
211-
assertTrue(FileUtils.fileExists(generatedReport.getAbsolutePath()));
211+
assertTrue(new File(generatedReport.getAbsolutePath()).exists());
212212

213213
String str = readFile(generatedReport);
214214
assertFalse(lowerCaseContains(str, "Hello.java"));
@@ -224,7 +224,7 @@ public void testCpdEncodingConfiguration() throws Exception {
224224

225225
// check if the CPD files were generated
226226
File generatedFile = new File(getBasedir(), "target/test/unit/default-configuration/target/cpd.xml");
227-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
227+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
228228
String str = readFile(generatedFile);
229229
assertTrue(lowerCaseContains(str, "AppSample.java"));
230230
} finally {
@@ -237,7 +237,7 @@ public void testCpdJavascriptConfiguration() throws Exception {
237237

238238
// verify the generated file to exist and violations are reported
239239
File generatedFile = new File(getBasedir(), "target/test/unit/default-configuration/target/cpd.xml");
240-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
240+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
241241
String str = readFile(generatedFile);
242242
assertTrue(lowerCaseContains(str, "Sample.js"));
243243
assertTrue(lowerCaseContains(str, "SampleDup.js"));
@@ -248,7 +248,7 @@ public void testCpdJspConfiguration() throws Exception {
248248

249249
// verify the generated file to exist and violations are reported
250250
File generatedFile = new File(getBasedir(), "target/test/unit/default-configuration/target/cpd.xml");
251-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
251+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
252252
String str = readFile(generatedFile);
253253
assertTrue(lowerCaseContains(str, "sample.jsp"));
254254
assertTrue(lowerCaseContains(str, "sampleDup.jsp"));
@@ -259,7 +259,7 @@ public void testExclusionsConfiguration() throws Exception {
259259

260260
// verify the generated file to exist and no duplications are reported
261261
File generatedFile = new File(getBasedir(), "target/test/unit/default-configuration/target/cpd.xml");
262-
assertTrue(FileUtils.fileExists(generatedFile.getAbsolutePath()));
262+
assertTrue(new File(generatedFile.getAbsolutePath()).exists());
263263
String str = readFile(generatedFile);
264264
assertEquals(0, StringUtils.countMatches(str, "<duplication"));
265265
}

0 commit comments

Comments
 (0)