review: refactor: Launcher#buildModel return a CtModel

src/main/java/spoon/Launcher.java

@@ -702,10 +702,11 @@ public boolean accept(File file, String s) {
 	};
 
 	@Override
-	public void buildModel() {
+	public CtModel buildModel() {
 		long tstart = System.currentTimeMillis();
 		modelBuilder.build();
 		getEnvironment().debugMessage("model built in " + (System.currentTimeMillis() - tstart));
+		return modelBuilder.getFactory().getModel();
 	}
 
 	@Override

src/main/java/spoon/SpoonAPI.java

@@ -103,7 +103,7 @@ public interface SpoonAPI {
 	/**
 	 * Builds the model
 	 */
-	void buildModel();
+	CtModel buildModel();
 
 	/**
 	 * Processes the model with the processors given previously with {@link #addProcessor(String)}

src/test/java/spoon/LauncherTest.java

@@ -5,6 +5,7 @@
 import org.junit.Test;
 
 import spoon.compiler.Environment;
+import spoon.reflect.CtModel;
 import spoon.reflect.visitor.DefaultJavaPrettyPrinter;
 import spoon.support.JavaOutputProcessor;
 
@@ -15,6 +16,7 @@
 import java.util.List;
 
 import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 
 public class LauncherTest {
@@ -92,4 +94,17 @@ public void testLauncherInEmptyWorkingDir() throws Exception {
 		}
 	}
 
+
+	@Test
+	public void testLLauncherBuildModelReturnAModel() throws Exception {
+		// contract: Launcher#buildModel should return a consistent CtModel
+		final Launcher launcher = new Launcher();
+		launcher.addInputResource("./src/test/resources/spoon/test/api/Foo.java");
+		launcher.getEnvironment().setNoClasspath(true);
+		CtModel model = launcher.buildModel();
+		assertNotNull(model);
+
+		assertEquals(2, model.getAllTypes().size());
+	}
+
 }