added some unit tests, + mockito

buildergen/pom.xml

@@ -12,7 +12,7 @@
 
 	<properties>
 		<project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
-		<!--project.release.type should be set to "releases" or "snapshots"  -->
+		<!--project.release.type should be set to "releases" or "snapshots" -->
 		<project.release.type>releases</project.release.type>
 		<altDeploymentRepository>${project.release.type}-repo::default::file:../../${project.release.type}</altDeploymentRepository>
 	</properties>
@@ -21,7 +21,13 @@
 		<dependency>
 			<groupId>junit</groupId>
 			<artifactId>junit</artifactId>
-			<version>3.8.1</version>
+			<version>4.8.1</version>
+			<scope>test</scope>
+		</dependency>
+		<dependency>
+			<groupId>org.mockito</groupId>
+			<artifactId>mockito-all</artifactId>
+			<version>1.8.1</version>
 			<scope>test</scope>
 		</dependency>
 		<dependency>

buildergen/src/main/java/info/piwai/buildergen/generation/SourceCodewriter.java → buildergen/src/main/java/info/piwai/buildergen/generation/SourceCodeWriter.java

@@ -30,11 +30,11 @@
  * 
  * @author Pierre-Yves Ricau (py.ricau at gmail.com)
  */
-public class SourceCodewriter extends CodeWriter {
+public class SourceCodeWriter extends CodeWriter {
 
 	private final Filer filer;
 
-	public SourceCodewriter(Filer filer) {
+	public SourceCodeWriter(Filer filer) {
 		this.filer = filer;
 	}
 

buildergen/src/main/java/info/piwai/buildergen/generation/SourceGenerator.java

@@ -49,7 +49,7 @@ public SourceGenerator(Filer filer) {
 	 */
 	public void generate(JCodeModel codeModel) throws IOException {
 
-		SourceCodewriter sourceCodeWriter = new SourceCodewriter(filer);
+		SourceCodeWriter sourceCodeWriter = new SourceCodeWriter(filer);
 
 		PrologCodeWriter prologCodeWriter = new PrologCodeWriter(sourceCodeWriter, HEADER_WARNING);
 

buildergen/src/test/java/info/piwai/buildergen/generation/ResourceCodeWriterTest.java

@@ -0,0 +1,82 @@
+package info.piwai.buildergen.generation;
+
+import static org.junit.Assert.assertSame;
+import static org.mockito.Matchers.anyString;
+import static org.mockito.Mockito.doThrow;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
+import java.io.IOException;
+import java.io.OutputStream;
+
+import javax.annotation.processing.Filer;
+import javax.tools.FileObject;
+import javax.tools.JavaFileManager.Location;
+
+import org.junit.Test;
+import org.mockito.Mockito;
+
+import com.sun.codemodel.JCodeModel;
+import com.sun.codemodel.JPackage;
+
+public class ResourceCodeWriterTest {
+
+	/**
+	 * This test is just for the fun of using Mockito. It does not really test
+	 * the external behavior of the method.
+	 */
+	@Test
+	public void usesFilerToOpenStream() throws IOException {
+
+		Filer filer = mock(Filer.class);
+		FileObject fileObject = mock(FileObject.class);
+		when(filer.createResource(Mockito.<Location> any(), anyString(), anyString())).thenReturn(fileObject);
+
+		OutputStream expectedOS = mock(OutputStream.class);
+		when(fileObject.openOutputStream()).thenReturn(expectedOS);
+
+		ResourceCodeWriter resourceCodeWriter = new ResourceCodeWriter(filer);
+		JPackage jPackage = new JCodeModel()._package("some.package");
+		OutputStream resultingOS = resourceCodeWriter.openBinary(jPackage, null);
+
+		assertSame(expectedOS, resultingOS);
+	}
+
+	@Test
+	public void doesNotCallOutputStreamCloseMethod() throws IOException {
+
+		Filer filer = mock(Filer.class);
+		FileObject fileObject = mock(FileObject.class);
+		when(filer.createResource(Mockito.<Location> any(), anyString(), anyString())).thenReturn(fileObject);
+
+		OutputStream expectedOS = mock(OutputStream.class);
+		when(fileObject.openOutputStream()).thenReturn(expectedOS);
+
+		ResourceCodeWriter resourceCodeWriter = new ResourceCodeWriter(filer);
+		JPackage jPackage = new JCodeModel()._package("some.package");
+		resourceCodeWriter.openBinary(jPackage, null);
+
+		doThrow(new RuntimeException("Close method should not be called")).when(expectedOS).close();
+
+		resourceCodeWriter.close();
+	}
+
+	@Test
+	public void multipleCallsAreDelegatedToFiler() throws IOException{
+		Filer filer = mock(Filer.class);
+
+		FileObject fileObject = mock(FileObject.class);
+		when(filer.createResource(Mockito.<Location> any(), anyString(), anyString())).thenReturn(fileObject);
+
+		ResourceCodeWriter resourceCodeWriter = new ResourceCodeWriter(filer);
+		JPackage jPackage = new JCodeModel()._package("some.package");
+		resourceCodeWriter.openBinary(jPackage, null);
+		resourceCodeWriter.openBinary(jPackage, null);
+
+		 verify(filer, times(2)).createResource(Mockito.<Location> any(), anyString(), anyString());
+
+	}
+
+}

buildergen/src/test/java/info/piwai/buildergen/generation/SourceCodeWriterTest.java

@@ -0,0 +1,76 @@
+package info.piwai.buildergen.generation;
+
+import static org.junit.Assert.assertSame;
+import static org.mockito.Matchers.anyString;
+import static org.mockito.Mockito.doThrow;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.times;
+import static org.mockito.Mockito.verify;
+import static org.mockito.Mockito.when;
+
+import java.io.IOException;
+import java.io.OutputStream;
+
+import javax.annotation.processing.Filer;
+import javax.tools.JavaFileObject;
+
+import org.junit.Test;
+
+import com.sun.codemodel.JCodeModel;
+import com.sun.codemodel.JPackage;
+
+public class SourceCodeWriterTest {
+
+	@Test
+	public void usesFilerToOpenStream() throws IOException {
+
+		Filer filer = mock(Filer.class);
+		JavaFileObject fileObject = mock(JavaFileObject.class);
+		when(filer.createSourceFile(anyString())).thenReturn(fileObject);
+
+		OutputStream expectedOS = mock(OutputStream.class);
+		when(fileObject.openOutputStream()).thenReturn(expectedOS);
+
+		SourceCodeWriter sourceCodeWriter = new SourceCodeWriter(filer);
+		JPackage jPackage = new JCodeModel()._package("some.package");
+		OutputStream resultingOS = sourceCodeWriter.openBinary(jPackage, "SomeClass.java");
+
+		assertSame(expectedOS, resultingOS);
+	}
+
+	@Test
+	public void doesNotCallOutputStreamCloseMethod() throws IOException {
+
+		Filer filer = mock(Filer.class);
+		JavaFileObject fileObject = mock(JavaFileObject.class);
+		when(filer.createSourceFile(anyString())).thenReturn(fileObject);
+
+		OutputStream expectedOS = mock(OutputStream.class);
+		when(fileObject.openOutputStream()).thenReturn(expectedOS);
+
+		SourceCodeWriter sourceCodeWriter = new SourceCodeWriter(filer);
+		JPackage jPackage = new JCodeModel()._package("some.package");
+		sourceCodeWriter.openBinary(jPackage, "SomeClass.java");
+
+		doThrow(new RuntimeException("Close method should not be called")).when(expectedOS).close();
+
+		sourceCodeWriter.close();
+	}
+
+	@Test
+	public void multipleCallsAreDelegatedToFiler() throws IOException {
+		Filer filer = mock(Filer.class);
+
+		JavaFileObject fileObject = mock(JavaFileObject.class);
+		when(filer.createSourceFile(anyString())).thenReturn(fileObject);
+
+		SourceCodeWriter sourceCodeWriter = new SourceCodeWriter(filer);
+		JPackage jPackage = new JCodeModel()._package("some.package");
+		sourceCodeWriter.openBinary(jPackage, "SomeClass.java");
+		sourceCodeWriter.openBinary(jPackage, "SomeClass.java");
+
+		verify(filer, times(2)).createSourceFile(anyString());
+
+	}
+
+}

buildergen/src/test/java/info/piwai/buildergen/validation/IsValidTest.java

@@ -0,0 +1,31 @@
+package info.piwai.buildergen.validation;
+
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertTrue;
+
+import org.junit.Test;
+
+public class IsValidTest {
+
+	@Test
+	public void initialStateIsValid() {
+		IsValid valid = new IsValid();
+		assertTrue(valid.isValid());
+	}
+
+	@Test
+	public void invalidateInvalidates() {
+		IsValid valid = new IsValid();
+		valid.invalidate();
+		assertFalse(valid.isValid());
+	}
+
+	@Test
+	public void doubleInvalidateInvalidates() {
+		IsValid valid = new IsValid();
+		valid.invalidate();
+		valid.invalidate();
+		assertFalse(valid.isValid());
+	}
+
+}