From 6dd7fd8b3a96e15c121d6e4a5e7b5743afb1b99b Mon Sep 17 00:00:00 2001 From: Kanagaraj Manickam k00365106 Date: Wed, 30 Aug 2017 12:28:29 +0530 Subject: Add validation for all commands Add new project to validate the commands and optionally add required test cases CLI-35 Change-Id: I8bd437c77421a590b0e60e4aec12cc99997451a1 Signed-off-by: Kanagaraj Manickam k00365106 --- .../java/org/onap/cli/fw/OnapCommandRegistrar.java | 39 ++++++----- .../onap/cli/fw/cmd/OnapSchemaRefreshCommand.java | 2 +- .../onap/cli/fw/cmd/OnapSchemaValidateCommand.java | 2 +- .../java/org/onap/cli/main/OnapCliMainTest.java | 21 ------ pom.xml | 1 + validation/pom.xml | 38 +++++++++++ .../org/onap/cli/validation/OnapCliMainTest.java | 76 ++++++++++++++++++++++ 7 files changed, 136 insertions(+), 43 deletions(-) create mode 100644 validation/src/test/java/org/onap/cli/validation/OnapCliMainTest.java diff --git a/framework/src/main/java/org/onap/cli/fw/OnapCommandRegistrar.java b/framework/src/main/java/org/onap/cli/fw/OnapCommandRegistrar.java index ca875b35..56f04e16 100644 --- a/framework/src/main/java/org/onap/cli/fw/OnapCommandRegistrar.java +++ b/framework/src/main/java/org/onap/cli/fw/OnapCommandRegistrar.java @@ -52,6 +52,8 @@ public class OnapCommandRegistrar { private Set availableProductVersions = new HashSet<>(); + private String enabledProductVersion = OnapCommandConfg.getEnabledProductVersion(); + private static OnapCommandRegistrar registrar = null; /** @@ -105,7 +107,7 @@ public class OnapCommandRegistrar { * @return set */ public Set listCommandsForEnabledProductVersion() { - String version = OnapCommandConfg.getEnabledProductVersion(); + String version = this.getEnabledProductVersion(); Set cmds = new HashSet<>(); if (!this.availableProductVersions.contains(version)) { @@ -120,30 +122,27 @@ public class OnapCommandRegistrar { return cmds; } + public Set getAvailableProductVersions() { + return this.availableProductVersions; + } + + public void setEnabledProductVersion(String version) { + this.enabledProductVersion = version; + } + + public String getEnabledProductVersion() { + return this.enabledProductVersion; + } + /** - * Returns map of command to schema. + * Returns command details. * * @return map * @throws OnapCommandException * exception */ - public Map getAllCommandToSchemaMap() throws OnapCommandException { - Map map = new HashMap<>(); - List schemas = OnapCommandUtils.findAllExternalSchemas(); - if (schemas != null) { - for (ExternalSchema schema : schemas) { - map.put(schema.getCmdName() + ":" + schema.getCmdVersion(), schema.getSchemaName()); - } - } - if (this.registry != null) { - for (String cmd : this.registry.keySet()) { - if (!map.containsKey(cmd) && registry.get(cmd) != null) { - map.put(cmd, this.getSchemaFileName(registry.get(cmd))); - } - } - } - - return map; + public List listCommandInfo() throws OnapCommandException { + return OnapCommandUtils.findAllExternalSchemas(); } /** @@ -156,7 +155,7 @@ public class OnapCommandRegistrar { * Exception */ public OnapCommand get(String cmdName) throws OnapCommandException { - return this.get(cmdName, OnapCommandConfg.getEnabledProductVersion()); + return this.get(cmdName, this.getEnabledProductVersion()); } private OnapCommand get(String cmdName, String version) throws OnapCommandException { diff --git a/framework/src/main/java/org/onap/cli/fw/cmd/OnapSchemaRefreshCommand.java b/framework/src/main/java/org/onap/cli/fw/cmd/OnapSchemaRefreshCommand.java index 7a156cac..823f0d77 100644 --- a/framework/src/main/java/org/onap/cli/fw/cmd/OnapSchemaRefreshCommand.java +++ b/framework/src/main/java/org/onap/cli/fw/cmd/OnapSchemaRefreshCommand.java @@ -30,7 +30,7 @@ import java.util.List; * Refresh external schema. * */ -@OnapCommandSchema(name = "schema-refresh", version="cli-1.0", schema = "schema-refresh.yaml") +@OnapCommandSchema(name = "schema-refresh", version = "cli-1.0", schema = "schema-refresh.yaml") public class OnapSchemaRefreshCommand extends OnapCommand { @Override diff --git a/framework/src/main/java/org/onap/cli/fw/cmd/OnapSchemaValidateCommand.java b/framework/src/main/java/org/onap/cli/fw/cmd/OnapSchemaValidateCommand.java index 3119f9c7..4028cc94 100644 --- a/framework/src/main/java/org/onap/cli/fw/cmd/OnapSchemaValidateCommand.java +++ b/framework/src/main/java/org/onap/cli/fw/cmd/OnapSchemaValidateCommand.java @@ -29,7 +29,7 @@ import java.util.Map; /** * Validate schema command. */ -@OnapCommandSchema(name = "schema-validate", version="cli-1.0", schema = "schema-validate.yaml") +@OnapCommandSchema(name = "schema-validate", version = "cli-1.0", schema = "schema-validate.yaml") public class OnapSchemaValidateCommand extends OnapCommand { @Override diff --git a/main/src/test/java/org/onap/cli/main/OnapCliMainTest.java b/main/src/test/java/org/onap/cli/main/OnapCliMainTest.java index 78278852..ff4b4362 100644 --- a/main/src/test/java/org/onap/cli/main/OnapCliMainTest.java +++ b/main/src/test/java/org/onap/cli/main/OnapCliMainTest.java @@ -124,27 +124,6 @@ public class OnapCliMainTest { this.handle(new String[] { "sample-test", "-h" }); } - @Test - public void validateCommands() throws IOException, OnapCommandException { - Map cmdSchemaMap = OnapCommandRegistrar.getRegistrar().getAllCommandToSchemaMap(); - for (String cmdName : cmdSchemaMap.keySet()) { - System.out.println( - "************************* '" + cmdSchemaMap.get(cmdName) + "' *******************************"); - this.handle(new String[] { "schema-validate", "-l", cmdSchemaMap.get(cmdName), "-i"}); - } - } - - @Test - public void commandHelpTest() throws OnapCommandException { - Set cmds = OnapCommandRegistrar.getRegistrar().listCommands(); - - for (String cmdName : cmds) { - System.out.println("************************* '" + cmdName + "' *******************************"); - this.handle(new String[] { cmdName, "-h" }); - } - - } - @Test public void interactiveTest() { cli = new OnapCli(new String[] { "-i" }); diff --git a/pom.xml b/pom.xml index 46a4bf0c..63e48d75 100644 --- a/pom.xml +++ b/pom.xml @@ -40,6 +40,7 @@ plugins main deployment + validation diff --git a/validation/pom.xml b/validation/pom.xml index adeb49b8..dc42e2f7 100644 --- a/validation/pom.xml +++ b/validation/pom.xml @@ -37,5 +37,43 @@ 4.11 test + + org.onap.cli + cli-main + ${project.version} + + + org.onap.cli + cli-plugins-msb + ${project.version} + + + org.onap.cli + cli-plugins-sdc + ${project.version} + + + org.onap.cli + cli-plugins-aai + ${project.version} + + diff --git a/validation/src/test/java/org/onap/cli/validation/OnapCliMainTest.java b/validation/src/test/java/org/onap/cli/validation/OnapCliMainTest.java new file mode 100644 index 00000000..f89e265d --- /dev/null +++ b/validation/src/test/java/org/onap/cli/validation/OnapCliMainTest.java @@ -0,0 +1,76 @@ +/* + * Copyright 2017 Huawei Technologies Co., Ltd. + * + * Licensed under the Apache License, Version 2.0 (the "License"); + * you may not use this file except in compliance with the License. + * You may obtain a copy of the License at + * + * http://www.apache.org/licenses/LICENSE-2.0 + * + * Unless required by applicable law or agreed to in writing, software + * distributed under the License is distributed on an "AS IS" BASIS, + * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. + * See the License for the specific language governing permissions and + * limitations under the License. + */ + +package org.onap.cli.validation; + +import java.io.IOException; +import java.util.Map; +import java.util.Set; + +import org.aspectj.lang.annotation.After; +import org.junit.Test; +import org.onap.cli.fw.OnapCommandRegistrar; +import org.onap.cli.fw.error.OnapCommandException; +import org.onap.cli.fw.utils.ExternalSchema; +import org.onap.cli.fw.utils.OnapCommandUtils; +import org.onap.cli.main.OnapCli; + +public class OnapCliMainTest { + + OnapCli cli = null; + + /** + * Clean up. + */ + @After(value = "") + public void cleanup() { + if (this.cli != null) { + if (cli.getExitCode() != 0) { + // Fail test case + } + } + } + + private void handle(String[] args) { + cli = new OnapCli(args); + cli.handle(); + } + + @Test + public void validateCommands() throws IOException, OnapCommandException { + OnapCommandRegistrar.getRegistrar().setEnabledProductVersion("cli-1.0"); + for (ExternalSchema sch : OnapCommandRegistrar.getRegistrar().listCommandInfo()) { + System.out.println( + "************************* validate '" + sch.getCmdName() + "' *******************************"); + this.handle(new String[] { "schema-validate", "-l", sch.getSchemaName(), "-i"}); + } + } + + @Test + public void commandHelpTest() throws OnapCommandException { + for (String version: OnapCommandRegistrar.getRegistrar().getAvailableProductVersions()) { + OnapCommandRegistrar.getRegistrar().setEnabledProductVersion(version); + for (ExternalSchema sch : OnapCommandRegistrar.getRegistrar().listCommandInfo()) { + if (sch.getCmdVersion().equals(version)) { + System.out.println( + "************************* help '" + sch.getCmdName() + "' *******************************"); + this.handle(new String[] { sch.getCmdName(), "-h"}); + } + } + } + } + + } -- cgit 1.2.3-korg