RAT-202: Reverting IntelliJ induced changes in resource handling.
* Solved the local problem to run tests in apache-rat-core in IntelliJ by
changing the folder structure locally.
* Now only ant tasks fail (separate mail to dev@).
git-svn-id: https://svn.apache.org/repos/asf/creadur/rat/trunk@1681876 13f79535-47bb-0310-9956-ffa450edef68
diff --git a/apache-rat-core/src/test/java/org/apache/rat/analysis/AnalyserFactoryTest.java b/apache-rat-core/src/test/java/org/apache/rat/analysis/AnalyserFactoryTest.java
index c8a6592..30d3365 100644
--- a/apache-rat-core/src/test/java/org/apache/rat/analysis/AnalyserFactoryTest.java
+++ b/apache-rat-core/src/test/java/org/apache/rat/analysis/AnalyserFactoryTest.java
@@ -33,9 +33,6 @@
public class AnalyserFactoryTest {
- // Marks where to insert a path prefix to make tests run from within IntelliJ due to different path settings
- private static String INFIX_MARKER = "UNDER_INTELLIJ_THERE_IS_A_SUBDIRECTORY_HERE";
-
private static final IHeaderMatcher MATCHES_NOTHING_MATCHER = new IHeaderMatcher() {
public boolean match(Document subject, String line) throws RatHeaderAnalysisException {
return false;
@@ -52,18 +49,18 @@
@Before
public void setUp() throws Exception {
out = new StringWriter();
- XmlWriter writer = new XmlWriter(out);
+ final XmlWriter writer = new XmlWriter(out);
reporter = new SimpleXmlClaimReporter(writer);
analyser = DefaultAnalyserFactory.createDefaultAnalyser(MATCHES_NOTHING_MATCHER);
}
@Test
public void standardTypeAnalyser() throws Exception {
- MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/Text.txt"));
+ final MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/Text.txt"));
analyser.analyse(document);
reporter.report(document);
- assertEqualsWithPathInfix("Open standard element", //
- "<resource name='" + INFIX_MARKER + "src/test/resources/elements/Text.txt'><header-sample>/*\n" +
+ assertEquals("Open standard element", //
+ "<resource name='src/test/resources/elements/Text.txt'><header-sample>/*\n" +
" * Licensed to the Apache Software Foundation (ASF) under one\n" +
" * or more contributor license agreements. See the NOTICE file\n" +
" * distributed with this work for additional information\n" +
@@ -84,56 +81,37 @@
"\n" +
" \n" +
"</header-sample><header-type name='?????'/><license-family name='?????'/><type name='standard'/>", out.toString());
-
}
@Test
public void noteTypeAnalyser() throws Exception {
- MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/LICENSE"));
+ final MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/LICENSE"));
analyser.analyse(document);
reporter.report(document);
- assertEqualsWithPathInfix("Open note element", "<resource name='" + INFIX_MARKER + "src/test/resources/elements/LICENSE'><type name='notice'/>", out.toString());
+ assertEquals("Open note element", "<resource name='src/test/resources/elements/LICENSE'><type name='notice'/>", out.toString());
}
@Test
public void binaryTypeAnalyser() throws Exception {
- MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/Image.png"));
+ final MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/Image.png"));
analyser.analyse(document);
reporter.report(document);
- assertEqualsWithPathInfix("Open binary element", "<resource name='" + INFIX_MARKER + "src/test/resources/elements/Image.png'><type name='binary'/>", out.toString());
+ assertEquals("Open binary element", "<resource name='src/test/resources/elements/Image.png'><type name='binary'/>", out.toString());
}
@Test
public void archiveTypeAnalyser() throws Exception {
- MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/dummy.jar"));
+ final MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/dummy.jar"));
analyser.analyse(document);
reporter.report(document);
- assertEqualsWithPathInfix("Open archive element", "<resource name='" + INFIX_MARKER + "src/test/resources/elements/dummy.jar'><type name='archive'/>", out.toString());
+ assertEquals("Open archive element", "<resource name='src/test/resources/elements/dummy.jar'><type name='archive'/>", out.toString());
}
@Test
public void archiveTypeAnalyserIntelliJ() throws Exception {
- MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/dummy.jar"));
+ final MonolithicFileDocument document = new MonolithicFileDocument(Resources.getResourceFile("/elements/dummy.jar"));
analyser.analyse(document);
reporter.report(document);
- assertEqualsWithPathInfix("Open archive element", "<resource name='" + INFIX_MARKER + "src/test/resources/elements/dummy.jar'><type name='archive'/>", out.toString());
- }
-
- private static void assertEqualsWithPathInfix(final String messagePrefix, final String expectedWithMarker, final String actual) {
- // if the given string is parameter expectedWithMarker is <code>null</code>,
- // the test code fails with NPE since it's used in a wrong way.
-
- boolean anyMatch = true;
- foundMatch:
- for (String marker : Resources.INTELLIJ_PROJECT_PREFIXES) {
- if (actual.equals(expectedWithMarker)) {
- anyMatch = false;
- break foundMatch;
- }
- }
-
- if (!anyMatch) {
- assertEquals(messagePrefix, expectedWithMarker.replaceAll(INFIX_MARKER, ""), actual);
- }
+ assertEquals("Open archive element", "<resource name='src/test/resources/elements/dummy.jar'><type name='archive'/>", out.toString());
}
}
diff --git a/apache-rat-core/src/test/java/org/apache/rat/test/utils/Resources.java b/apache-rat-core/src/test/java/org/apache/rat/test/utils/Resources.java
index bf5ab23..52968a5 100644
--- a/apache-rat-core/src/test/java/org/apache/rat/test/utils/Resources.java
+++ b/apache-rat-core/src/test/java/org/apache/rat/test/utils/Resources.java
@@ -29,8 +29,6 @@
import java.io.InputStream;
import java.io.InputStreamReader;
import java.io.Reader;
-import java.util.Arrays;
-import java.util.List;
/**
@@ -45,9 +43,6 @@
// Does nothing
}
- // If started in IntelliJ the working directory is different, thus tests are not running through
- public static final List<String> INTELLIJ_PROJECT_PREFIXES = Arrays.asList("", "apache-rat-core/" /*,"apache-rat-plugin/"*/);
-
public static final String SRC_TEST_RESOURCES = "src/test/resources";
public static final String SRC_MAIN_RESOURCES = "src/main/resources";
private static File TEST_RESOURCE_BASE_PATH = new File(SRC_TEST_RESOURCES);
@@ -73,17 +68,7 @@
private static File getResourceFromBase(File baseDir, String pResource) throws IOException {
File f = new File(baseDir, pResource);
if (!f.isFile()) {
- // try IntelliJ workaround before giving up
- for (String prefix : INTELLIJ_PROJECT_PREFIXES) {
- f = new File(new File(prefix + baseDir.getPath()), pResource);
- System.out.println("Trying: " + f.getAbsolutePath());
- if (!f.isFile()) continue;
- }
-
- if (!f.isFile()) {
- throw new FileNotFoundException("Unable to locate resource file: " + pResource);
- }
-
+ throw new FileNotFoundException("Unable to locate resource file: " + pResource);
}
return f;
}
@@ -95,16 +80,7 @@
public static File[] getResourceFiles(String pResource) throws IOException {
File f = new File(TEST_RESOURCE_BASE_PATH, pResource);
if (!f.isDirectory()) {
- // try IntelliJ workaround before giving up
- for (String prefix : INTELLIJ_PROJECT_PREFIXES) {
- f = new File(new File(prefix + TEST_RESOURCE_BASE_PATH.getPath()), pResource);
- System.out.println("Trying: " + f.getAbsolutePath());
- if (!f.isDirectory()) continue;
- }
-
- if (!f.isDirectory()) {
- throw new FileNotFoundException("Unable to locate resource directory: " + pResource);
- }
+ throw new FileNotFoundException("Unable to locate resource directory: " + pResource);
}
return f.listFiles(new FileFilter() {