Fix Checkstyle errors + New integration with the tooling core

This solves two issues thanks to the last tooling core upgrades.
#38 Add JUnit tests for auto-completion
#33 Add auto-completion for imports

Author: vincent-zurczak

plugins/net.roboconf.eclipse.modeler/src/net/roboconf/eclipse/modeler/RoboconfModelerPlugin.java

@@ -43,96 +43,96 @@
  */
 public class RoboconfModelerPlugin extends AbstractUIPlugin {
 
-    // The plug-in ID
-    public static final String PLUGIN_ID = "net.roboconf.eclipse.modeler";
+	// The plug-in ID
+	public static final String PLUGIN_ID = "net.roboconf.eclipse.modeler";
 
-    // The shared instance
-    private static RoboconfModelerPlugin plugin;
-    private Set<Viewpoint> viewpoints;
+	// The shared instance
+	private static RoboconfModelerPlugin plugin;
+	private Set<Viewpoint> viewpoints;
 
 
-    /**
-     * Constructor.
-     */
-    public RoboconfModelerPlugin() {
-    	// nothing
-    }
+	/**
+	 * Constructor.
+	 */
+	public RoboconfModelerPlugin() {
+		// nothing
+	}
 
 
-    @Override
+	@Override
 	public void start(BundleContext context) throws Exception {
-      super.start(context);
-	  plugin = this;
+		super.start(context);
+		plugin = this;
 
-	  this.viewpoints = new HashSet<> ();
-	  this.viewpoints.addAll( ViewpointRegistry.getInstance().registerFromPlugin(
-			  PLUGIN_ID + "/description/graph.odesign" ));
-    }
+		this.viewpoints = new HashSet<> ();
+		this.viewpoints.addAll( ViewpointRegistry.getInstance().registerFromPlugin(
+				PLUGIN_ID + "/description/graph.odesign" ));
+	}
 
 
-    @Override
+	@Override
 	public void stop(BundleContext context) throws Exception {
 
-    	plugin = null;
-    	if (this.viewpoints != null) {
-    		for (final Viewpoint viewpoint: this.viewpoints) {
-    			ViewpointRegistry.getInstance().disposeFromPlugin(viewpoint);
-    		}
+		plugin = null;
+		if (this.viewpoints != null) {
+			for (final Viewpoint viewpoint: this.viewpoints) {
+				ViewpointRegistry.getInstance().disposeFromPlugin(viewpoint);
+			}
 
-    		this.viewpoints.clear();
-    		this.viewpoints = null;
-    	}
+			this.viewpoints.clear();
+			this.viewpoints = null;
+		}
 
-    	super.stop(context);
-    }
+		super.stop(context);
+	}
 
 
-    /**
-     * @return the shared instance
-     */
-    public static RoboconfModelerPlugin getDefault() {
-    	return plugin;
-    }
+	/**
+	 * @return the shared instance
+	 */
+	public static RoboconfModelerPlugin getDefault() {
+		return plugin;
+	}
 
 
-    /**
-     * Logs an exception.
-     * @param e the exception to log
-     * @param severity the severity, given as one of the {@link IStatus} constants
-     */
-    public static void log( Exception e, int severity ) {
+	/**
+	 * Logs an exception.
+	 * @param e the exception to log
+	 * @param severity the severity, given as one of the {@link IStatus} constants
+	 */
+	public static void log( Exception e, int severity ) {
 
-    	String msg = e.getMessage();
-    	if( msg == null || msg.trim().length() == 0 )
-    		msg = "An error occurred.";
+		String msg = e.getMessage();
+		if( msg == null || msg.trim().length() == 0 )
+			msg = "An error occurred.";
 
-    	IStatus status = new Status( severity, PLUGIN_ID, msg, e );
-    	getDefault().getLog().log( status );
-    }
+		IStatus status = new Status( severity, PLUGIN_ID, msg, e );
+		getDefault().getLog().log( status );
+	}
 
 
-    /**
-     * Logs a message.
-     * @param message the message to log
-     * @param severity the severity, given as one of the {@link IStatus} constants
-     */
-    public static void log( String message, int severity ) {
-    	IStatus status = new Status( severity, PLUGIN_ID, message );
-    	getDefault().getLog().log( status );
-    }
+	/**
+	 * Logs a message.
+	 * @param message the message to log
+	 * @param severity the severity, given as one of the {@link IStatus} constants
+	 */
+	public static void log( String message, int severity ) {
+		IStatus status = new Status( severity, PLUGIN_ID, message );
+		getDefault().getLog().log( status );
+	}
 
 
-    public static Image findImage( String filePath ) {
+	public static Image findImage( String filePath ) {
 
-    	ImageDescriptor desc = imageDescriptorFromPlugin( PLUGIN_ID, filePath );
-    	Image img = null;
+		ImageDescriptor desc = imageDescriptorFromPlugin( PLUGIN_ID, filePath );
+		Image img = null;
 		try {
 			img = desc.createImage();
 
 		} catch( Exception e ) {
 			log( "Image " + filePath + " could not be found.", IStatus.ERROR );
 		}
 
-    	return img;
-    }
+		return img;
+	}
 }

plugins/net.roboconf.eclipse.plugin/src/net/roboconf/eclipse/plugin/RoboconfEclipseConstants.java

@@ -34,5 +34,4 @@ public interface RoboconfEclipseConstants {
 	String BUILDER_ID = "net.roboconf.eclipse.project.RoboconfBuilder"; //$NON-NLS-1$
 	String MARKER_ID = "net.roboconf.eclipse.project.RoboconfMarker"; 	//$NON-NLS-1$
 	String MARKER_ERROR_CODE = "net.roboconf.eclipse.plugin.marker.errorCode";	//$NON-NLS-1$
-	String SRC_MAIN_MODEL = "src/main/model/";	//$NON-NLS-1$
 }

plugins/net.roboconf.eclipse.plugin/src/net/roboconf/eclipse/plugin/builder/RoboconfIncrementalBuilder.java

@@ -116,7 +116,7 @@ protected final void clean( IProgressMonitor monitor ) throws CoreException {
 	 */
 	private IFolder findApplicationRootLocation() {
 		// FIXME: it can be under "target" too.
-		return (IFolder) getProject().findMember( RoboconfEclipseConstants.SRC_MAIN_MODEL );
+		return (IFolder) getProject().findMember( "src/main/model/" );
 	}