summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--framework/src/main/java/org/onap/cli/fw/OnapCommand.java15
-rw-r--r--framework/src/main/java/org/onap/cli/fw/OnapCommandRegistrar.java28
-rw-r--r--framework/src/main/java/org/onap/cli/fw/OnapCommandSchema.java9
-rw-r--r--framework/src/main/java/org/onap/cli/fw/cmd/OnapHttpCommand.java3
-rw-r--r--framework/src/main/java/org/onap/cli/fw/conf/Constants.java40
-rw-r--r--framework/src/main/java/org/onap/cli/fw/utils/OnapCommandUtils.java13
-rw-r--r--framework/src/main/java/org/onap/cli/fw/utils/SchemaInfo.java25
-rw-r--r--framework/src/main/resources/META-INF/services/org.onap.cli.fw.OnapCommand1
-rw-r--r--framework/src/main/resources/open-cli.properties5
-rw-r--r--framework/src/main/resources/version.info2
-rw-r--r--framework/src/test/java/org/onap/cli/fw/OnapCommandRegistrarTest.java53
-rw-r--r--framework/src/test/java/org/onap/cli/fw/ad/OnapAuthClientCommandBasedTest.java19
-rw-r--r--framework/src/test/java/org/onap/cli/fw/utils/OnapCommandUtilsTest.java2
-rw-r--r--main/src/main/java/org/onap/cli/main/OnapCli.java2
-rw-r--r--main/src/main/java/org/onap/cli/main/error/OnapCliArgumentValueMissing.java2
-rw-r--r--main/src/main/java/org/onap/cli/main/error/OnapCliInvalidArgument.java2
-rw-r--r--main/src/test/java/org/onap/cli/main/error/OnapCliArgumentTest.java6
17 files changed, 87 insertions, 140 deletions
diff --git a/framework/src/main/java/org/onap/cli/fw/OnapCommand.java b/framework/src/main/java/org/onap/cli/fw/OnapCommand.java
index 89c62207..87c1ba74 100644
--- a/framework/src/main/java/org/onap/cli/fw/OnapCommand.java
+++ b/framework/src/main/java/org/onap/cli/fw/OnapCommand.java
@@ -19,12 +19,8 @@ package org.onap.cli.fw;
import java.util.ArrayList;
import java.util.List;
import java.util.Map;
-import java.util.Optional;
-import org.onap.cli.fw.ad.OnapService;
-import org.onap.cli.fw.cmd.CommandType;
import org.onap.cli.fw.conf.Constants;
-import org.onap.cli.fw.conf.OnapCommandConfg;
import org.onap.cli.fw.error.OnapCommandException;
import org.onap.cli.fw.error.OnapCommandHelpFailed;
import org.onap.cli.fw.error.OnapCommandInvalidParameterType;
@@ -33,7 +29,6 @@ import org.onap.cli.fw.error.OnapCommandInvalidResultAttributeScope;
import org.onap.cli.fw.error.OnapCommandInvalidSchema;
import org.onap.cli.fw.error.OnapCommandInvalidSchemaVersion;
import org.onap.cli.fw.error.OnapCommandNotInitialized;
-import org.onap.cli.fw.error.OnapCommandParameterMissing;
import org.onap.cli.fw.error.OnapCommandParameterNameConflict;
import org.onap.cli.fw.error.OnapCommandParameterOptionConflict;
import org.onap.cli.fw.error.OnapCommandRegistrationFailed;
@@ -198,7 +193,7 @@ public abstract class OnapCommand {
Map<String, OnapCommandParameter> paramMap = this.getParametersMap();
// -h or --help is always higher precedence !, user can set this value to get help message
- if ("true".equals(paramMap.get(Constants.DEFAULT_PARAMETER_HELP).getValue())) {
+ if (Constants.BOOLEAN_TRUE.equals(paramMap.get(Constants.DEFAULT_PARAMETER_HELP).getValue())) {
OnapCommandResult result = new OnapCommandResult();
result.setType(ResultType.TEXT);
result.setOutput(this.printHelp());
@@ -206,7 +201,7 @@ public abstract class OnapCommand {
}
// -v or --version is next higher precedence !, user can set this value to get help message
- if ("true".equals(paramMap.get(Constants.DEFAULT_PARAMETER_VERSION).getValue())) {
+ if (Constants.BOOLEAN_TRUE.equals(paramMap.get(Constants.DEFAULT_PARAMETER_VERSION).getValue())) {
OnapCommandResult result = new OnapCommandResult();
result.setType(ResultType.TEXT);
result.setOutput(this.printVersion());
@@ -219,16 +214,16 @@ public abstract class OnapCommand {
// -f or --format
this.cmdResult.setType(
ResultType.get(paramMap.get(Constants.DEFAULT_PARAMETER_OUTPUT_FORMAT).getValue().toString()));
- if ("true".equals(paramMap.get(Constants.DEFAULT_PARAMETER_OUTPUT_ATTR_LONG).getValue())) {
+ if (Constants.BOOLEAN_TRUE.equals(paramMap.get(Constants.DEFAULT_PARAMETER_OUTPUT_ATTR_LONG).getValue())) {
this.cmdResult.setScope(OnapCommandResultAttributeScope.LONG);
}
// --no-title
- if ("true".equals(paramMap.get(Constants.DEFAULT_PARAMETER_OUTPUT_NO_TITLE).getValue())) {
+ if (Constants.BOOLEAN_TRUE.equals(paramMap.get(Constants.DEFAULT_PARAMETER_OUTPUT_NO_TITLE).getValue())) {
this.cmdResult.setIncludeTitle(false);
}
// --debug
- if ("true".equals(paramMap.get(Constants.DEFAULT_PARAMETER_DEBUG).getValue())) {
+ if (Constants.BOOLEAN_TRUE.equals(paramMap.get(Constants.DEFAULT_PARAMETER_DEBUG).getValue())) {
this.cmdResult.setDebug(true);
}
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 1d2c5166..f42f68f9 100644
--- a/framework/src/main/java/org/onap/cli/fw/OnapCommandRegistrar.java
+++ b/framework/src/main/java/org/onap/cli/fw/OnapCommandRegistrar.java
@@ -19,7 +19,6 @@ package org.onap.cli.fw;
import java.io.IOException;
import java.lang.reflect.Constructor;
import java.lang.reflect.InvocationTargetException;
-import java.net.URISyntaxException;
import java.util.HashMap;
import java.util.HashSet;
import java.util.List;
@@ -27,7 +26,6 @@ import java.util.Map;
import java.util.Set;
import org.apache.commons.io.IOUtils;
-import org.onap.cli.fw.cmd.OnapHttpCommand;
import org.onap.cli.fw.conf.Constants;
import org.onap.cli.fw.conf.OnapCommandConfg;
import org.onap.cli.fw.error.OnapCommandException;
@@ -101,7 +99,7 @@ public class OnapCommandRegistrar {
* Invalid registration exception
* @throws OnapCommandRegistrationProductInfoMissing
*/
- public void register(String name, String version, Class<? extends OnapCommand> cmd) throws OnapCommandInvalidRegistration, OnapCommandRegistrationProductInfoMissing {
+ private void register(String name, String version, Class<? extends OnapCommand> cmd) throws OnapCommandInvalidRegistration, OnapCommandRegistrationProductInfoMissing {
if (version == null || version.isEmpty()) {
throw new OnapCommandRegistrationProductInfoMissing(name);
}
@@ -210,12 +208,7 @@ public class OnapCommandRegistrar {
Constructor<?> constr = cls.getConstructor();
cmd = (OnapCommand) constr.newInstance();
- String schemaName;
- if (cmd.getClass().equals(OnapHttpCommand.class)) { // NOSONAR
- schemaName = OnapCommandUtils.getSchemaInfo(cmdName, version).getSchemaName();
- } else {
- schemaName = this.getSchemaFileName(cls);
- }
+ String schemaName = OnapCommandUtils.getSchemaInfo(cmdName, version).getSchemaName();
cmd.initializeSchema(schemaName);
} catch (OnapCommandException | NoSuchMethodException | SecurityException | InstantiationException
@@ -233,7 +226,13 @@ public class OnapCommandRegistrar {
for (Class<OnapCommand> cmd : cmds) {
if (cmd.isAnnotationPresent(OnapCommandSchema.class)) {
OnapCommandSchema ano = cmd.getAnnotation(OnapCommandSchema.class);
- map.put(ano.schema(), cmd);
+ if (ano.schema() != null && !ano.schema().isEmpty()) {
+ map.put(ano.schema(), cmd);
+ } else if (ano.type() != null && !ano.type().isEmpty()) {
+ map.put(ano.type(), cmd);
+ } else {
+ throw new OnapUnsupportedSchemaProfile(ano.schema());
+ }
}
}
@@ -246,8 +245,8 @@ public class OnapCommandRegistrar {
Map<String, Class<OnapCommand>> plugins = this.autoDiscoverCommandPlugins();
for (SchemaInfo schema : schemas) {
- if (schema.isHttp()) {
- this.register(schema.getCmdName(), schema.getProduct(), OnapHttpCommand.class);
+ if (plugins.containsKey(schema.getSchemaProfile())) {
+ this.register(schema.getCmdName(), schema.getProduct(), plugins.get(schema.getSchemaProfile()));
} else if (plugins.containsKey(schema.getSchemaName())) {
this.register(schema.getCmdName(), schema.getProduct(), plugins.get(schema.getSchemaName()));
} else {
@@ -256,11 +255,6 @@ public class OnapCommandRegistrar {
}
}
- private String getSchemaFileName(Class<? extends OnapCommand> cmd) {
- OnapCommandSchema ano = (OnapCommandSchema) cmd.getAnnotation(OnapCommandSchema.class);
- return ano.schema();
- }
-
/**
* Helps to find the Oclip CLI version, could be used with --version or -v option.
*
diff --git a/framework/src/main/java/org/onap/cli/fw/OnapCommandSchema.java b/framework/src/main/java/org/onap/cli/fw/OnapCommandSchema.java
index 16b53ea6..d5f71044 100644
--- a/framework/src/main/java/org/onap/cli/fw/OnapCommandSchema.java
+++ b/framework/src/main/java/org/onap/cli/fw/OnapCommandSchema.java
@@ -26,7 +26,7 @@ import java.lang.annotation.RetentionPolicy;
* default if the schema declaration is missing for a command abc-create, schema file name could be
* abc-create-schema.yaml, corresponding command would like as below
*
- * @OnapCommandSchema(name="abc-create", schema="onap-abc-create-schema.yaml") public class AbcCreate extends
+ * @OnapCommandSchema(type="http", schema="onap-abc-create-schema.yaml") public class AbcCreate extends
* OnapCommand { ... }
*/
@Retention(RetentionPolicy.RUNTIME)
@@ -37,5 +37,10 @@ public @interface OnapCommandSchema {
*
* @return
*/
- String schema();
+ String schema() default "";
+
+ /**
+ * Schema type
+ */
+ String type() default "";
}
diff --git a/framework/src/main/java/org/onap/cli/fw/cmd/OnapHttpCommand.java b/framework/src/main/java/org/onap/cli/fw/cmd/OnapHttpCommand.java
index 198affbc..7ff38b64 100644
--- a/framework/src/main/java/org/onap/cli/fw/cmd/OnapHttpCommand.java
+++ b/framework/src/main/java/org/onap/cli/fw/cmd/OnapHttpCommand.java
@@ -23,6 +23,7 @@ import java.util.List;
import java.util.Map;
import org.onap.cli.fw.OnapCommand;
+import org.onap.cli.fw.OnapCommandSchema;
import org.onap.cli.fw.ad.OnapAuthClient;
import org.onap.cli.fw.ad.OnapService;
import org.onap.cli.fw.conf.Constants;
@@ -32,7 +33,6 @@ import org.onap.cli.fw.error.OnapCommandExecutionFailed;
import org.onap.cli.fw.error.OnapCommandFailedMocoGenerate;
import org.onap.cli.fw.http.HttpInput;
import org.onap.cli.fw.http.HttpResult;
-import org.onap.cli.fw.input.OnapCommandParameter;
import org.onap.cli.fw.output.OnapCommandResultAttribute;
import org.onap.cli.fw.utils.OnapCommandUtils;
import org.onap.cli.http.mock.MockJsonGenerator;
@@ -43,6 +43,7 @@ import org.onap.cli.http.mock.MockResponse;
* Oclip Command.
*
*/
+@OnapCommandSchema(type = Constants.HTTP_SCHEMA_PROFILE)
public class OnapHttpCommand extends OnapCommand {
private HttpInput input = new HttpInput();
diff --git a/framework/src/main/java/org/onap/cli/fw/conf/Constants.java b/framework/src/main/java/org/onap/cli/fw/conf/Constants.java
index bd416446..248ea068 100644
--- a/framework/src/main/java/org/onap/cli/fw/conf/Constants.java
+++ b/framework/src/main/java/org/onap/cli/fw/conf/Constants.java
@@ -28,26 +28,26 @@ public class Constants {
public static final String OPEN_CLI_VERSION = "cli.version";
public static final String HTTP_API_KEY_USE_COOKIES = "cli.http.api_key_use_cookies";
- public static final String TOP_LEVEL_PARAMS_LIST ="cli.schema.top_level_params_list";
- public static final String TOP_LEVEL_MANDATORY_LIST ="cli.schema.top_level_mandatory_list";
- public static final String SERVICE_PARAMS_LIST ="cli.schema.service_params_list";
- public static final String SERVICE_PARAMS_MANDATORY_LIST ="cli.schema.service_params_mandatory_list";
- public static final String INFO_PARAMS_LIST ="cli.schema.info_params_list";
- public static final String INFO_PARAMS_MANDATORY_LIST ="cli.schema.info_params_mandatory_list";
- public static final String INPUT_PARAMS_LIST ="cli.schema.input_params_list";
- public static final String INPUT_PARAMS_MANDATORY_LIST ="cli.schema.input_params_mandatory_list";
- public static final String RESULT_PARAMS_LIST ="cli.schema.result_params_list";
- public static final String RESULT_PARAMS_MANDATORY_LIST ="cli.schema.result_params_mandatory_list";
- public static final String HTTP_SECTIONS ="cli.schema.http_sections";
- public static final String HTTP_MANDATORY_SECTIONS ="cli.schema.http_mandatory_sections";
- public static final String HTTP_REQUEST_PARAMS ="cli.schema.http_request_params";
- public static final String HTTP_REQUEST_MANDATORY_PARAMS ="cli.schema.http_request_mandatory_params";
- public static final String HTTP_METHODS ="cli.schema.http_methods";
- public static final String BOOLEAN_VALUE ="cli.schema.boolean_values";
- public static final String AUTH_VALUES="cli.schema.auth_values";
- public static final String MODE_VALUES="cli.schema.mode_values";
- public static final String COMMAND_TYPE_VALUES="cli.command.type";
-
+ public static final String TOP_LEVEL_PARAMS_LIST = "cli.schema.top_level_params_list";
+ public static final String TOP_LEVEL_MANDATORY_LIST = "cli.schema.top_level_mandatory_list";
+ public static final String SERVICE_PARAMS_LIST = "cli.schema.service_params_list";
+ public static final String SERVICE_PARAMS_MANDATORY_LIST = "cli.schema.service_params_mandatory_list";
+ public static final String INFO_PARAMS_LIST = "cli.schema.info_params_list";
+ public static final String INFO_PARAMS_MANDATORY_LIST = "cli.schema.info_params_mandatory_list";
+ public static final String INPUT_PARAMS_LIST = "cli.schema.input_params_list";
+ public static final String INPUT_PARAMS_MANDATORY_LIST = "cli.schema.input_params_mandatory_list";
+ public static final String RESULT_PARAMS_LIST = "cli.schema.result_params_list";
+ public static final String RESULT_PARAMS_MANDATORY_LIST = "cli.schema.result_params_mandatory_list";
+ public static final String HTTP_SECTIONS = "cli.schema.http_sections";
+ public static final String HTTP_MANDATORY_SECTIONS = "cli.schema.http_mandatory_sections";
+ public static final String HTTP_REQUEST_PARAMS = "cli.schema.http_request_params";
+ public static final String HTTP_REQUEST_MANDATORY_PARAMS = "cli.schema.http_request_mandatory_params";
+ public static final String HTTP_METHODS = "cli.schema.http_methods";
+ public static final String BOOLEAN_VALUE = "cli.schema.boolean_values";
+ public static final String AUTH_VALUES = "cli.schema.auth_values";
+ public static final String MODE_VALUES = "cli.schema.mode_values";
+ public static final String COMMAND_TYPE_VALUES = "cli.command.type";
+ public static final String SCHEMA_TYPES_SUPPORTED = "cli.schema.type.supported";
//http connection
public static final String SSLCONTEST_TLS = "TLSV1.2";
public static final String APPLICATION_JSON = "application/json";
diff --git a/framework/src/main/java/org/onap/cli/fw/utils/OnapCommandUtils.java b/framework/src/main/java/org/onap/cli/fw/utils/OnapCommandUtils.java
index ca708274..0f64ed3a 100644
--- a/framework/src/main/java/org/onap/cli/fw/utils/OnapCommandUtils.java
+++ b/framework/src/main/java/org/onap/cli/fw/utils/OnapCommandUtils.java
@@ -1658,8 +1658,11 @@ public class OnapCommandUtils {
}
private static String identitySchemaProfileType(Map<String, ?> schemaYamlMap) {
- if (schemaYamlMap.get(Constants.HTTP) != null) {
- return Constants.HTTP_SCHEMA_PROFILE;
+
+ for (String schemeType : OnapCommandConfg.getSchemaAttrInfo(Constants.SCHEMA_TYPES_SUPPORTED)) {
+ if (schemaYamlMap.get(schemeType) != null) {
+ return schemeType;
+ }
}
return Constants.BASIC_SCHEMA_PROFILE;
@@ -1852,16 +1855,16 @@ public class OnapCommandUtils {
*/
public static SchemaInfo getSchemaInfo(String cmd, String version) throws OnapCommandException {
List<SchemaInfo> list = discoverOrLoadSchemas();
- SchemaInfo schemaStr = null;
+ SchemaInfo schemaInfo = null;
if (list != null) {
for (SchemaInfo schema : list) {
if (cmd.equals(schema.getCmdName()) && version.equals(schema.getProduct())) {
- schemaStr = schema;
+ schemaInfo = schema;
break;
}
}
}
- return schemaStr;
+ return schemaInfo;
}
/**
diff --git a/framework/src/main/java/org/onap/cli/fw/utils/SchemaInfo.java b/framework/src/main/java/org/onap/cli/fw/utils/SchemaInfo.java
index 462fcceb..85d6b1f8 100644
--- a/framework/src/main/java/org/onap/cli/fw/utils/SchemaInfo.java
+++ b/framework/src/main/java/org/onap/cli/fw/utils/SchemaInfo.java
@@ -19,35 +19,33 @@ package org.onap.cli.fw.utils;
import org.onap.cli.fw.cmd.CommandType;
import org.onap.cli.fw.conf.Constants;
-import com.fasterxml.jackson.annotation.JsonIgnore;
-
/**
* SchemaInfo is used in discovery caching.
*
*/
public class SchemaInfo {
- /**
- * Name of the schema file name
- */
+ /**
+ * Name of the schema file name
+ */
private String schemaName;
-
+
/**
* Schema location in complete path
*/
private String schemaURI;
-
+
private String cmdName;
-
+
private String product;
-
+
/**
* OCS version
*/
private String version;
-
+
private String type = CommandType.CMD.name();
-
+
private String schemaProfile = Constants.BASIC_SCHEMA_PROFILE;
public String getSchemaName() {
@@ -98,11 +96,6 @@ public class SchemaInfo {
this.schemaProfile = internal;
}
- @JsonIgnore
- public boolean isHttp() {
- return this.getSchemaProfile().equals(Constants.HTTP_SCHEMA_PROFILE);
- }
-
public String getType() {
return type;
}
diff --git a/framework/src/main/resources/META-INF/services/org.onap.cli.fw.OnapCommand b/framework/src/main/resources/META-INF/services/org.onap.cli.fw.OnapCommand
index e29c2c28..1993bcb6 100644
--- a/framework/src/main/resources/META-INF/services/org.onap.cli.fw.OnapCommand
+++ b/framework/src/main/resources/META-INF/services/org.onap.cli.fw.OnapCommand
@@ -3,3 +3,4 @@ org.onap.cli.fw.cmd.OnapSchemaRefreshCommand
org.onap.cli.fw.cmd.BasicAuthLoginCommand
org.onap.cli.fw.cmd.BasicAuthLogoutCommand
org.onap.cli.fw.cmd.CatalogCommand
+org.onap.cli.fw.cmd.OnapHttpCommand \ No newline at end of file
diff --git a/framework/src/main/resources/open-cli.properties b/framework/src/main/resources/open-cli.properties
index 84a4736e..757b0259 100644
--- a/framework/src/main/resources/open-cli.properties
+++ b/framework/src/main/resources/open-cli.properties
@@ -34,7 +34,6 @@ cli.schema.auth_values=none,basic
cli.schema.mode_values=direct,catalog
cli.command.type=cmd,auth,catalog
-<<<<<<< 5301969fc26714a8494882027116282898278f32:framework/src/main/resources/onap.properties
#product version
cli.product.version=cli-1.0
@@ -42,5 +41,5 @@ cli.product.version=cli-1.0
cli.sample.gen.enable=false
cli.sample.gen.target=.
-=======
->>>>>>> Migrate onap-cli-schema into open-cli-schema:framework/src/main/resources/open-cli.properties
+# mrkanag Move this to db, once exteranl command registration is supported in place of discovery
+cli.schema.type.supported=http
diff --git a/framework/src/main/resources/version.info b/framework/src/main/resources/version.info
index 34f4d969..fc959d19 100644
--- a/framework/src/main/resources/version.info
+++ b/framework/src/main/resources/version.info
@@ -2,6 +2,6 @@ CLI version : __VERSION__
Available products: __AVAILABLE_PRODUCT_VERSIONS__
Enabled product : __ENABLED_PRODUCT_VERSIONS__
-To enable a product , use one of following methods:
+To enable a product, use one of following methods:
1. In scripting mode, Set environment variable OPEN_CLI_PRODUCT_IN_USE
2. In interactive mode, set the directive 'use <product>'
diff --git a/framework/src/test/java/org/onap/cli/fw/OnapCommandRegistrarTest.java b/framework/src/test/java/org/onap/cli/fw/OnapCommandRegistrarTest.java
index 8a69ae61..339a0f66 100644
--- a/framework/src/test/java/org/onap/cli/fw/OnapCommandRegistrarTest.java
+++ b/framework/src/test/java/org/onap/cli/fw/OnapCommandRegistrarTest.java
@@ -25,6 +25,7 @@ import java.io.File;
import java.net.URL;
import org.junit.Before;
+import org.junit.Ignore;
import org.junit.Test;
import org.onap.cli.fw.error.OnapCommandException;
import org.onap.cli.fw.error.OnapCommandHelpFailed;
@@ -57,24 +58,6 @@ public class OnapCommandRegistrarTest {
}
@Test
- public void registerTest() throws OnapCommandException {
- OnapCommand test = new OnapCommandTest();
- Class<OnapCommand> cmd = (Class<OnapCommand>) test.getClass();
- registerar.register("Test", "open-cli", cmd);
- OnapCommand cc = registerar.get("Test");
- assertTrue(cmd == cc.getClass());
-
- }
-
- @Test
- public void cmdTestSchema() throws OnapCommandException {
- OnapCommand test = new OnapCommandTest();
- Class<OnapCommand> cmd = (Class<OnapCommand>) test.getClass();
- registerar.register("Test", "open-cli", cmd);
- OnapCommand cc = registerar.get("Test");
- }
-
- @Test
public void oclipCommandNotFoundTest() throws OnapCommandException {
try {
registerar = OnapCommandRegistrar.getRegistrar();
@@ -88,40 +71,6 @@ public class OnapCommandRegistrarTest {
}
@Test
- public void oclipCommandRegistrationFailedTest() throws OnapCommandException {
-
- @OnapCommandSchema(schema = "sample-test-schema.yaml")
- class Test extends OnapCommand {
-
- @Override
- protected void run() throws OnapCommandException {
-
- }
-
- }
-
- OnapCommand com = new Test();
- Class<OnapCommand> cmd = (Class<OnapCommand>) com.getClass();
- try {
- registerar.register("Test2", "open-cli", cmd);
- registerar.get("Test2");
- fail("This should have thrown an exception");
- } catch (OnapCommandRegistrationFailed e) {
- assertEquals("0x2002", e.getErrorCode());
- }
- }
-
- @Test(expected = OnapCommandHelpFailed.class)
- public void helpTestException() throws OnapCommandException {
- OnapCommand test = new OnapCommandTest1();
- Class<OnapCommand> cmd = (Class<OnapCommand>) test.getClass();
- registerar = new OnapCommandRegistrar();
- registerar.register("test1", "open-cli", cmd);
- String help = registerar.getHelp();
- assertNotNull(help);
- }
-
- @Test
public void helpTest() throws OnapCommandException {
String help = registerar.getHelp();
assertNotNull(help);
diff --git a/framework/src/test/java/org/onap/cli/fw/ad/OnapAuthClientCommandBasedTest.java b/framework/src/test/java/org/onap/cli/fw/ad/OnapAuthClientCommandBasedTest.java
index a2a25bc2..9c733c94 100644
--- a/framework/src/test/java/org/onap/cli/fw/ad/OnapAuthClientCommandBasedTest.java
+++ b/framework/src/test/java/org/onap/cli/fw/ad/OnapAuthClientCommandBasedTest.java
@@ -18,6 +18,7 @@ package org.onap.cli.fw.ad;
import static org.junit.Assert.fail;
+import org.junit.Before;
import org.junit.Test;
import org.onap.cli.fw.OnapCommand;
import org.onap.cli.fw.OnapCommandRegistrar;
@@ -25,9 +26,15 @@ import org.onap.cli.fw.cmd.OnapHttpCommand;
import org.onap.cli.fw.conf.Constants;
import org.onap.cli.fw.conf.OnapCommandConfg;
import org.onap.cli.fw.error.OnapCommandException;
+import org.onap.cli.fw.error.OnapCommandProductVersionInvalid;
public class OnapAuthClientCommandBasedTest {
+ @Before
+ public void setup() throws OnapCommandProductVersionInvalid, OnapCommandException {
+ OnapCommandRegistrar.getRegistrar().setEnabledProductVersion(OnapCommandConfg.getProductName());
+ }
+
@Test
public void internalCommandTest() {
try {
@@ -37,7 +44,7 @@ public class OnapAuthClientCommandBasedTest {
cmd.execute();
} catch (OnapCommandException e) {
fail("Internal command failed to run");
- e.printStackTrace();
+ e.printStackTrace(System.out);
}
}
@@ -52,7 +59,7 @@ public class OnapAuthClientCommandBasedTest {
cmd.execute();
} catch (OnapCommandException e) {
fail("External command Yes Auth Yes Catalog failed to run");
- e.printStackTrace();
+ e.printStackTrace(System.out);
}
}
@@ -64,8 +71,8 @@ public class OnapAuthClientCommandBasedTest {
cmd.execute();
} catch (OnapCommandException e) {
- fail("External command Yes Auth No Catalog failed to run");
- e.printStackTrace();
+ fail("External command Yes Auth No Catalog failed to run " + e.getMessage());
+ e.printStackTrace(System.out);
}
}
@@ -80,7 +87,7 @@ public class OnapAuthClientCommandBasedTest {
cmd.execute();
} catch (OnapCommandException e) {
fail("External command Yes Auth No Catalog failed to run");
- e.printStackTrace();
+ e.printStackTrace(System.out);
}
}
@@ -93,7 +100,7 @@ public class OnapAuthClientCommandBasedTest {
cmd.execute();
} catch (OnapCommandException e) {
fail("External command No Auth No Catalog failed to run");
- e.printStackTrace();
+ e.printStackTrace(System.out);
}
}
diff --git a/framework/src/test/java/org/onap/cli/fw/utils/OnapCommandUtilsTest.java b/framework/src/test/java/org/onap/cli/fw/utils/OnapCommandUtilsTest.java
index e874dc96..419c4edc 100644
--- a/framework/src/test/java/org/onap/cli/fw/utils/OnapCommandUtilsTest.java
+++ b/framework/src/test/java/org/onap/cli/fw/utils/OnapCommandUtilsTest.java
@@ -255,7 +255,7 @@ public class OnapCommandUtilsTest {
@Test
public void findOnapCommandsTest() {
List<Class<OnapCommand>> cmds = OnapCommandUtils.discoverCommandPlugins();
- assertTrue(cmds.size() == 6);
+ assertTrue(cmds.size() == 7);
}
@Test
diff --git a/main/src/main/java/org/onap/cli/main/OnapCli.java b/main/src/main/java/org/onap/cli/main/OnapCli.java
index 357ee5e2..77de4408 100644
--- a/main/src/main/java/org/onap/cli/main/OnapCli.java
+++ b/main/src/main/java/org/onap/cli/main/OnapCli.java
@@ -98,7 +98,7 @@ public class OnapCli {
try {
if ((args.size() == 1) && (this.getLongOption(OnapCliConstants.PARAM_HELP_LOGN).equals(args.get(0))
|| this.getShortOption(OnapCliConstants.PARAM_HELP_SHORT).equals(args.get(0)))) {
- this.print(IOUtils.toString(this.getClass().getClassLoader().getResourceAsStream("onap-readme.txt")));
+ this.print(IOUtils.toString(this.getClass().getClassLoader().getResourceAsStream("oclip-readme.txt")));
String help = OnapCommandRegistrar.getRegistrar().getHelp();
this.print(help);
this.exitSuccessfully();
diff --git a/main/src/main/java/org/onap/cli/main/error/OnapCliArgumentValueMissing.java b/main/src/main/java/org/onap/cli/main/error/OnapCliArgumentValueMissing.java
index f1dd1ef7..15a93953 100644
--- a/main/src/main/java/org/onap/cli/main/error/OnapCliArgumentValueMissing.java
+++ b/main/src/main/java/org/onap/cli/main/error/OnapCliArgumentValueMissing.java
@@ -27,6 +27,6 @@ public class OnapCliArgumentValueMissing extends OnapCommandException {
private static final long serialVersionUID = -6362824415803063442L;
public OnapCliArgumentValueMissing(String arg) {
- super("0x7001", "Value for argument " + arg + " is missing");
+ super("0x7101", "Value for argument " + arg + " is missing");
}
}
diff --git a/main/src/main/java/org/onap/cli/main/error/OnapCliInvalidArgument.java b/main/src/main/java/org/onap/cli/main/error/OnapCliInvalidArgument.java
index e9f3adc5..10fbb75f 100644
--- a/main/src/main/java/org/onap/cli/main/error/OnapCliInvalidArgument.java
+++ b/main/src/main/java/org/onap/cli/main/error/OnapCliInvalidArgument.java
@@ -25,7 +25,7 @@ import org.onap.cli.fw.error.OnapCommandException;
public class OnapCliInvalidArgument extends OnapCommandException {
private static final long serialVersionUID = -1438492553530993246L;
- private static final String ERROR_CODE = "0x7000";
+ private static final String ERROR_CODE = "0x7102";
private static final String ERROR_MESSAGE1 = "Invalid argument ";
public OnapCliInvalidArgument(String arg) {
diff --git a/main/src/test/java/org/onap/cli/main/error/OnapCliArgumentTest.java b/main/src/test/java/org/onap/cli/main/error/OnapCliArgumentTest.java
index fcb919b9..aebdbcf2 100644
--- a/main/src/test/java/org/onap/cli/main/error/OnapCliArgumentTest.java
+++ b/main/src/test/java/org/onap/cli/main/error/OnapCliArgumentTest.java
@@ -25,15 +25,15 @@ public class OnapCliArgumentTest {
@Test
public void onapCliArgumentValueMissingTest() {
OnapCliArgumentValueMissing failed = new OnapCliArgumentValueMissing("Argument value missing");
- assertEquals("0x7001::Value for argument Argument value missing is missing", failed.getMessage());
+ assertEquals("0x7101::Value for argument Argument value missing is missing", failed.getMessage());
}
@Test
public void onapCliInvalidArgumentTest() {
OnapCliInvalidArgument failed = new OnapCliInvalidArgument("Invalid Argument");
- assertEquals("0x7000::Invalid argument Invalid Argument", failed.getMessage());
+ assertEquals("0x7102::Invalid argument Invalid Argument", failed.getMessage());
failed = new OnapCliInvalidArgument("Invalid Argument", new Exception(""));
- assertEquals("0x7000::Invalid argument Invalid Argument , ", failed.getMessage());
+ assertEquals("0x7102::Invalid argument Invalid Argument , ", failed.getMessage());
}
}