diff --git a/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/BuildBundleTestCatalog.java b/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/BuildBundleTestCatalog.java index a68c756..783ff14 100644 --- a/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/BuildBundleTestCatalog.java +++ b/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/BuildBundleTestCatalog.java @@ -63,18 +63,16 @@ public class BuildBundleTestCatalog extends AbstractMojo { @Parameter(defaultValue = "${project.compileClasspathElements}", readonly = true, required = true) private List classpathElements; - private boolean skip; - @Parameter(defaultValue = "${galasa.skip.bundletestcatatlog}", readonly = true, required = false) private boolean typoSkip; - + @Parameter(defaultValue = "${galasa.skip.bundletestcatalog}", readonly = true, required = false) private boolean correctSkip; + + private boolean skip = DeployTestCatalog.setSkip(correctSkip, typoSkip); public void execute() throws MojoExecutionException, MojoFailureException { - skip = DeployTestCatalog.setSkip(correctSkip, typoSkip); - if (skip) { getLog().info("Skipping Bundle Test Catalog build"); return; diff --git a/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/BuildGherkinTestCatalog.java b/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/BuildGherkinTestCatalog.java index 62e855f..336a342 100644 --- a/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/BuildGherkinTestCatalog.java +++ b/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/BuildGherkinTestCatalog.java @@ -39,9 +39,14 @@ public class BuildGherkinTestCatalog extends AbstractMojo { @Parameter(defaultValue = "${project.build.directory}", property = "outputDir", required = true) private File outputDirectory; - + @Parameter(defaultValue = "${galasa.skip.gherkintestcatatlog}", readonly = true, required = false) - private boolean skip; + private boolean typoSkip; + + @Parameter(defaultValue = "${galasa.skip.gherkintestcatalog}", readonly = true, required = false) + private boolean correctSkip; + + private boolean skip = DeployTestCatalog.setSkip(correctSkip, typoSkip); @Override public void execute() throws MojoExecutionException, MojoFailureException { diff --git a/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/DeployTestCatalog.java b/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/DeployTestCatalog.java index 367af1c..0d13b9b 100644 --- a/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/DeployTestCatalog.java +++ b/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/DeployTestCatalog.java @@ -39,21 +39,21 @@ public class DeployTestCatalog extends AbstractMojo { @Parameter(defaultValue = "${galasa.bootstrap}", readonly = true, required = false) private URL bootstrapUrl; - private boolean skip; - private boolean skipDeploy; - - @Parameter(defaultValue = "${galasa.skip.bundletestcatalog}", readonly = true, required = false) + + @Parameter(defaultValue = "${galasa.skip.bundletestcatatlog}", readonly = true, required = false) private boolean typoSkip; - + @Parameter(defaultValue = "${galasa.skip.bundletestcatalog}", readonly = true, required = false) private boolean correctSkip; - + @Parameter(defaultValue = "${galasa.skip.deploytestcatatlog}", readonly = true, required = false) private boolean typoSkipDeploy; - + @Parameter(defaultValue = "${galasa.skip.deploytestcatalog}", readonly = true, required = false) private boolean correctSkipDeploy; - + + private boolean skip = setSkip(correctSkip, typoSkip); + private boolean skipDeploy = setSkip(correctSkip, typoSkip); protected static boolean setSkip(boolean correctSkip, boolean typoSkip) { boolean skip = false; @@ -67,9 +67,6 @@ protected static boolean setSkip(boolean correctSkip, boolean typoSkip) { public void execute() throws MojoExecutionException, MojoFailureException { - skip = setSkip(correctSkip, typoSkip); - skipDeploy = setSkip(correctSkip, typoSkip); - if (skip || skipDeploy) { getLog().info("Skipping Deploy Test Catalog"); return; diff --git a/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/MergeTestCatalogs.java b/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/MergeTestCatalogs.java index d3b8e17..193b44d 100644 --- a/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/MergeTestCatalogs.java +++ b/galasa-maven-plugin/src/main/java/dev/galasa/maven/plugin/MergeTestCatalogs.java @@ -62,9 +62,14 @@ public class MergeTestCatalogs extends AbstractMojo { @Parameter(defaultValue = "${project.build.directory}", property = "outputDir", required = true) private File outputDirectory; - + @Parameter(defaultValue = "${galasa.skip.bundletestcatatlog}", readonly = true, required = false) - private boolean skip; + private boolean typoSkip; + + @Parameter(defaultValue = "${galasa.skip.bundletestcatalog}", readonly = true, required = false) + private boolean correctSkip; + + private boolean skip = DeployTestCatalog.setSkip(correctSkip, typoSkip); @Parameter(defaultValue = "${galasa.build.job}", readonly = true, required = false) private String buildJob;