Write simpler subpackage dependence unit tests

Author: gselzer

src/test/java/org/scijava/maven/plugin/enforcer/NoSubpackageDependenceRuleIntegrationTest.java

@@ -6,13 +6,13 @@
  * %%
  * Redistribution and use in source and binary forms, with or without
  * modification, are permitted provided that the following conditions are met:
- * 
+ *
  * 1. Redistributions of source code must retain the above copyright notice,
  *    this list of conditions and the following disclaimer.
  * 2. Redistributions in binary form must reproduce the above copyright notice,
  *    this list of conditions and the following disclaimer in the documentation
  *    and/or other materials provided with the distribution.
- * 
+ *
  * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
  * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
  * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
@@ -56,54 +56,66 @@
 import java.io.IOException;
 import java.net.URISyntaxException;
 import java.net.URL;
+import java.nio.charset.Charset;
 
 import org.apache.commons.io.IOUtils;
 import org.apache.maven.enforcer.rule.api.EnforcerRuleException;
 import org.junit.Before;
 import org.junit.Test;
 
 public class NoSubpackageDependenceRuleIntegrationTest {
-	private static final URL FITNESSE_TARGET_FOLDER = getResource("fitnesse-target");
-	private static final URL FITNESSE_EXPECTED_OUTPUT = getResource("fitnesse-expected-output-subpackage-dependence.txt");
-	private static final URL JUNIT_TARGET_FOLDER = getResource("junit-target");
-	private static final URL JUNIT_EXPECTED_OUTPUT = getResource("junit-expected-output-subpackage-dependence.txt");
-
-	private NoSubpackageDependenceRule rule;
-	private EnforcerRuleHelperMock helper;
+    // Subpackage dependence test files
+    private static final URL SUBPACKAGE_DEPENDENT_TARGET_FOLDER = //
+            getResource("subpackage-dependent-target");
+    private static final URL SUBPACKAGE_DEPENDENT_EXPECTED_OUTPUT = //
+            getResource("subpackage-dependent-expected-output.txt");
+    // Control test files
+    private static final URL CONTROL_TARGET_FOLDER = getResource("control-target");
 
-	@Before
-	public void setUp() throws Exception {
-		rule = new NoSubpackageDependenceRule();
-		helper = new EnforcerRuleHelperMock();
-	}
+    private NoSubpackageDependenceRule rule;
+    private EnforcerRuleHelperMock helper;
 
-	@Test
-	public void fitnesseIntegrationTest() throws Exception {
-		assertPackageCycles(FITNESSE_TARGET_FOLDER, FITNESSE_EXPECTED_OUTPUT);
-	}
+    @Before
+    public void setUp() throws Exception {
+        rule = new NoSubpackageDependenceRule();
+        helper = new EnforcerRuleHelperMock();
+    }
 
-	@Test
-	public void junitIntegrationTest() throws Exception {
-		assertPackageCycles(JUNIT_TARGET_FOLDER, JUNIT_EXPECTED_OUTPUT);
-	}
+    /**
+     * Test that a subpackage dependence throws an error
+     */
+    @Test
+    public void subpackageDependentIntegrationTest() throws URISyntaxException, IOException {
+        helper.setTestClassesDir(new File("non-existent"));
+        helper.setClassesDir(new File(SUBPACKAGE_DEPENDENT_TARGET_FOLDER.toURI()));
+        try {
+            rule.execute(helper);
+            fail("expected EnforcerRuleException");
+        } catch (EnforcerRuleException e) {
+            String expected = IOUtils.toString(SUBPACKAGE_DEPENDENT_EXPECTED_OUTPUT.openStream(), (Charset) null) //
+                    .replaceAll("\r", "") //
+                    .trim();
+            String actual = e.getMessage().trim();
+            assertEquals(expected, actual);
+        }
+    }
 
-	private void assertPackageCycles(URL targetFolder, URL expectedOutput) throws URISyntaxException, IOException {
-		helper.setTestClassesDir(new File("non-existent"));
-		helper.setClassesDir(new File(targetFolder.toURI()));
-		try {
-			rule.execute(helper);
-			fail("expected EnforcerRuleException");
-		} catch (EnforcerRuleException e) {
-			// using assertEquals to get a nice comparison editor in eclipse
-			assertEquals(getExpectedOutput(expectedOutput), e.getMessage());
-		}
-	}
+    /**
+     * Test no error thrown for no subpackage dependence
+     */
+    @Test
+    public void controlIntegrationTest() throws URISyntaxException {
+        helper.setTestClassesDir(new File("non-existent"));
+        helper.setClassesDir(new File(CONTROL_TARGET_FOLDER.toURI()));
+        try {
+            rule.execute(helper);
+        } catch (EnforcerRuleException e) {
+            fail("expected EnforcerRuleException");
+        }
+    }
 
-	private String getExpectedOutput(URL expectedOutput) throws IOException {
-		return IOUtils.toString(expectedOutput.openStream()).replaceAll("\r", "");
-	}
-
-	private static URL getResource(String path) {
-		return Thread.currentThread().getContextClassLoader().getResource(path);
-	}
+    private static URL getResource(String path) {
+        return Thread.currentThread().getContextClassLoader().getResource(path);
+    }
 }
+

src/test/resources/control-target/com/example/foo/Foo.class

@@ -0,0 +1,5 @@
+package com.example.foo;
+
+public class Foo {
+    public Foo() {}
+}

src/test/resources/control-target/com/example/foo/Foo.java

@@ -0,0 +1,9 @@
+package com.example.foo.bar;
+
+import com.example.foo.Foo;
+
+public class Bar {
+    public Bar() {
+        final Foo foo = new Foo();
+    }
+}

src/test/resources/control-target/com/example/foo/bar/Bar.class

@@ -0,0 +1,4 @@
+Some packages depend on subpackages:
+
+Package com.example.foo depends on subpackages: 
+        com.example.foo.bar (Foo)

src/test/resources/control-target/com/example/foo/bar/Bar.java

@@ -0,0 +1,9 @@
+package com.example.foo;
+
+import com.example.foo.bar.Bar;
+
+public class Foo {
+    public Foo() {
+        final Bar bar = new Bar();
+    }
+}

src/test/resources/subpackage-dependent-expected-output.txt

@@ -0,0 +1,7 @@
+package com.example.foo.bar;
+
+public class Bar {
+    public Bar() {
+
+    }
+}