Fix warnings in yang-maven-plugin
[yangtools.git] / yang / yang-maven-plugin / src / main / java / org / opendaylight / yangtools / yang2sources / plugin / YangToSourcesMojo.java
index 1f678a02230634155eca3f4e26795fe92e74ff27..8ecc6f230a75b337456239cd356753a0137627b4 100644 (file)
@@ -7,12 +7,17 @@
  */
 package org.opendaylight.yangtools.yang2sources.plugin;
 
+import com.google.common.annotations.VisibleForTesting;
+import com.google.common.collect.Collections2;
+import com.google.common.collect.ImmutableList;
+import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;
 import java.io.File;
 import java.util.Arrays;
+import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
 import java.util.Set;
-
+import java.util.function.Function;
 import org.apache.maven.artifact.repository.ArtifactRepository;
 import org.apache.maven.plugin.AbstractMojo;
 import org.apache.maven.plugin.MojoExecutionException;
@@ -25,36 +30,32 @@ import org.apache.maven.plugins.annotations.ResolutionScope;
 import org.apache.maven.project.MavenProject;
 import org.apache.maven.repository.RepositorySystem;
 import org.opendaylight.yangtools.yang.model.api.SchemaContext;
-import org.opendaylight.yangtools.yang.parser.impl.YangParserImpl;
 import org.opendaylight.yangtools.yang2sources.plugin.ConfigArg.CodeGeneratorArg;
-import org.opendaylight.yangtools.yang2sources.spi.CodeGenerator;
-import org.slf4j.impl.StaticLoggerBinder;
+import org.opendaylight.yangtools.yang2sources.spi.BasicCodeGenerator;
 import org.sonatype.plexus.build.incremental.BuildContext;
 
-import com.google.common.annotations.VisibleForTesting;
-
 /**
  * Generate sources from yang files using user provided set of
- * {@link CodeGenerator}s. Steps of this process:
+ * {@link BasicCodeGenerator}s. Steps of this process:
  * <ol>
  * <li>List yang files from {@link #yangFilesRootDir}</li>
- * <li>Process yang files using {@link YangParserImpl}</li>
- * <li>For each {@link CodeGenerator} from {@link #codeGenerators}:</li>
+ * <li>Process yang files using Yang Parser</li>
+ * <li>For each {@link BasicCodeGenerator} from {@link #codeGenerators}:
  * <ol>
  * <li>Instantiate using default constructor</li>
- * <li>Call {@link CodeGenerator#generateSources(SchemaContext, File, Set)}</li>
- * </ol>
+ * <li>Call {@link BasicCodeGenerator#generateSources(SchemaContext, File, Set, Function)}</li>
+ * </ol></li>
  * </ol>
  */
-@Mojo(name = "generate-sources", defaultPhase = LifecyclePhase.GENERATE_SOURCES, requiresDependencyResolution = ResolutionScope.COMPILE, requiresProject = true)
+@Mojo(name = "generate-sources", defaultPhase = LifecyclePhase.GENERATE_SOURCES,
+    requiresDependencyResolution = ResolutionScope.COMPILE, requiresProject = true)
 public final class YangToSourcesMojo extends AbstractMojo {
     public static final String PLUGIN_NAME = "org.opendaylight.yangtools:yang-maven-plugin";
 
     /**
-     * Classes implementing {@link CodeGenerator} interface. An instance will be
+     * Classes implementing {@link BasicCodeGenerator} interface. An instance will be
      * created out of every class using default constructor. Method {@link
-     * CodeGenerator#generateSources(SchemaContext, File, Set<String>
-     * yangModulesNames)} will be called on every instance.
+     * BasicCodeGenerator#generateSources(SchemaContext, File, Set)} will be called on every instance.
      */
     @Parameter(required = false)
     private CodeGeneratorArg[] codeGenerators;
@@ -73,7 +74,7 @@ public final class YangToSourcesMojo extends AbstractMojo {
     @Parameter(property = "project", required = true, readonly = true)
     private MavenProject project;
 
-    @Parameter(property = "inspectDependencies", required = true, readonly = true)
+    @Parameter(property = "inspectDependencies")
     private boolean inspectDependencies;
 
     @Component
@@ -84,45 +85,48 @@ public final class YangToSourcesMojo extends AbstractMojo {
     @Component
     private RepositorySystem repoSystem;
 
-    @Parameter( readonly = true, defaultValue = "${localRepository}" )
+    @Parameter(readonly = true, defaultValue = "${localRepository}")
     private ArtifactRepository localRepository;
 
-    @Parameter( readonly = true, defaultValue = "${project.remoteArtifactRepositories}" )
+    @Parameter(readonly = true, defaultValue = "${project.remoteArtifactRepositories}")
     private List<ArtifactRepository> remoteRepos;
 
+    // When set to "true", then the execution of the plugin is disabled
+    @Parameter(property = "yang.skip")
+    private String yangSkip;
 
     public YangToSourcesMojo() {
-    }
 
-    public void setProject(MavenProject project) {
-        this.project = project;
     }
 
     @VisibleForTesting
-    YangToSourcesMojo(YangToSourcesProcessor processor) {
+    YangToSourcesMojo(final YangToSourcesProcessor processor) {
         this.yangToSourcesProcessor = processor;
     }
 
+    public void setProject(final MavenProject project) {
+        this.project = project;
+    }
+
     @Override
+    @SuppressFBWarnings(value = "UWF_FIELD_NOT_INITIALIZED_IN_CONSTRUCTOR", justification = "yangFilesRootDir")
     public void execute() throws MojoExecutionException, MojoFailureException {
-        StaticLoggerBinder.getSingleton().setMavenLog(this.getLog());
-
-        Util.checkClasspath(project, repoSystem, localRepository, remoteRepos, getLog());
+        Util.checkClasspath(project, repoSystem, localRepository, remoteRepos);
 
         if (yangToSourcesProcessor == null) {
             List<CodeGeneratorArg> codeGeneratorArgs = processCodeGenerators(codeGenerators);
 
             // defaults to ${basedir}/src/main/yang
             File yangFilesRootFile = processYangFilesRootDir(yangFilesRootDir, project.getBasedir());
-            File[] excludedFiles = processExcludeFiles(excludeFiles, yangFilesRootFile);
+            Collection<File> excludedFiles = processExcludeFiles(excludeFiles, yangFilesRootFile);
 
-            yangToSourcesProcessor = new YangToSourcesProcessor(buildContext, getLog(), yangFilesRootFile,
+            yangToSourcesProcessor = new YangToSourcesProcessor(buildContext, yangFilesRootFile,
                     excludedFiles, codeGeneratorArgs, project, inspectDependencies);
         }
-        yangToSourcesProcessor.execute();
+        yangToSourcesProcessor.conditionalExecute("true".equals(yangSkip));
     }
 
-    private static List<CodeGeneratorArg> processCodeGenerators(CodeGeneratorArg[] codeGenerators) {
+    private static List<CodeGeneratorArg> processCodeGenerators(final CodeGeneratorArg[] codeGenerators) {
         List<CodeGeneratorArg> codeGeneratorArgs;
         if (codeGenerators == null) {
             codeGeneratorArgs = Collections.emptyList();
@@ -132,7 +136,7 @@ public final class YangToSourcesMojo extends AbstractMojo {
         return codeGeneratorArgs;
     }
 
-    private static File processYangFilesRootDir(String yangFilesRootDir, File baseDir) {
+    private static File processYangFilesRootDir(final String yangFilesRootDir, final File baseDir) {
         File yangFilesRootFile;
         if (yangFilesRootDir == null) {
             yangFilesRootFile = new File(baseDir, "src" + File.separator + "main" + File.separator + "yang");
@@ -147,16 +151,12 @@ public final class YangToSourcesMojo extends AbstractMojo {
         return yangFilesRootFile;
     }
 
-    private static File[] processExcludeFiles(String[] excludeFiles, File baseDir) {
+    private static Collection<File> processExcludeFiles(final String[] excludeFiles, final File baseDir) {
         if (excludeFiles == null) {
-            return new File[] {};
-        }
-        File[] result = new File[excludeFiles.length];
-        for (int i = 0; i < excludeFiles.length; i++) {
-            result[i] = new File(baseDir, excludeFiles[i]);
+            return ImmutableList.of();
         }
 
-        return result;
+        return Collections2.transform(Arrays.asList(excludeFiles), f -> new File(baseDir, f));
     }
 
 }