Added simple Tests for constructors

unused.methods/pdetestsrc/test/SuperClassWithExplicitlyUsedConstructor.java_

@@ -0,0 +1,17 @@
+package test;
+
+public class SuperClassWithExplicitlyUsedConstructor {
+
+	public static void main(String args) {
+		new Subclass();
+	}
+
+	public SuperClassWithExplicitlyUsedConstructor() {
+	}
+
+	class Subclass extends SuperClassWithExplicitlyUsedConstructor {
+		Subclass() {
+			super();
+		}
+	}
+}

unused.methods/pdetestsrc/test/SuperClassWithUnusedConstructor.java_

@@ -0,0 +1,7 @@
+package test;
+
+public class SuperClassWithUnusedConstructor {
+
+	public SuperClassWithUnusedConstructor() {
+	}
+}

unused.methods/test/unused/methods/AllPdeTests.java

@@ -5,7 +5,7 @@
 import org.junit.runners.Suite.SuiteClasses;
 
 @RunWith(Suite.class)
-@SuiteClasses({ FindUnusedMethodsInJavaProjectPdeTest.class })
+@SuiteClasses({ FindUnusedMethodsInJavaProjectPdeTest.class, FindUnusedContructorsPdeTest.class })
 // TODO test Markers
 public class AllPdeTests {
 

unused.methods/test/unused/methods/FindUnusedContructorsPdeTest.java

@@ -0,0 +1,39 @@
+package unused.methods;
+
+import static org.hamcrest.CoreMatchers.is;
+import static org.hamcrest.CoreMatchers.not;
+import static org.junit.Assert.assertThat;
+import static org.junit.matchers.JUnitMatchers.hasItem;
+import static unused.methods.MethodWithName.methodWithName;
+
+import java.io.IOException;
+import java.util.List;
+
+import org.eclipse.core.runtime.CoreException;
+import org.eclipse.core.runtime.NullProgressMonitor;
+import org.eclipse.jdt.core.IMethod;
+import org.eclipse.jdt.core.JavaModelException;
+import org.junit.Test;
+
+public class FindUnusedContructorsPdeTest extends PdeTestCaseWithTestProject {
+
+	@Test
+	public void findUnusedConstructorsInProject() throws CoreException, IOException {
+		List<IMethod> result = calculateUnusedMethods();
+		assertThat(result, not(hasItem(methodWithName("SuperClassWithExplicitlyUsedConstructor"))));
+		assertThat(result, hasItem(methodWithName("SuperClassWithUnusedConstructor")));
+		assertThat(result.size(), is(1));
+	}
+
+	private List<IMethod> calculateUnusedMethods() throws JavaModelException {
+		FindUnusedMethodsInJavaProject finder = new FindUnusedMethodsInJavaProject(project.asJavaProject());
+		finder.run(new NullProgressMonitor());
+		return finder.getUnusedMethods();
+	}
+
+	@Override
+	protected String[] getTestFiles() {
+		return new String[] { "/test/SuperClassWithExplicitlyUsedConstructor.java",
+				"/test/SuperClassWithUnusedConstructor.java" };
+	}
+}

unused.methods/test/unused/methods/FindUnusedMethodsInJavaProjectPdeTest.java

@@ -40,4 +40,9 @@ private List<IMethod> calculateUnusedMethods() throws JavaModelException {
 		finder.run(new NullProgressMonitor());
 		return finder.getUnusedMethods();
 	}
+
+	@Override
+	protected String[] getTestFiles() {
+		return new String[] { "/test/Movie.java", "/test/Actor.java" };
+	}
 }

unused.methods/test/unused/methods/PdeTestCaseWithTestProject.java

@@ -10,7 +10,7 @@
  * Do not use classes from jdt in example code as no jre or classpath is set up
  * in test project!!!
  */
-public class PdeTestCaseWithTestProject {
+public abstract class PdeTestCaseWithTestProject {
 
 	@Rule
 	public ProjectInWorkspace project = new ProjectInWorkspace().withJavaNature();
@@ -19,8 +19,12 @@ public class PdeTestCaseWithTestProject {
 	public void createTestJavaProject() throws CoreException, IOException {
 		project.createFolder("src");
 		project.createFolder("src/test");
-		project.createFile("src/test/Movie.java", new ResourceAsString("/test/Movie.java_").read());
-		project.createFile("src/test/Actor.java", new ResourceAsString("/test/Actor.java_").read());
+		for (String file : getTestFiles()) {
+			project.createFile("src" + file, new ResourceAsString(file + "_").read());
+		}
 		project.asJavaProject();
 	}
+
+	protected abstract String[] getTestFiles();
+
 }