fix: update proper error messages (#87)

Co-authored-by: Maciej Urbańczyk <urbanczyk.maciej.95@gmail.com>

Author: mihirterna

src/components/Context/Context.tsx

@@ -4,7 +4,7 @@ import { Box, Text } from 'ink';
 import ContextError from './contexterror';
 import { useContextFile, MissingArgumentstError } from '../../hooks/context';
 import { SpecificationFile } from '../../hooks/validation';
-
+import { ARGUMENTS } from '../../messages';
 export const ListContexts: React.FunctionComponent = () => {
   const { response, error } = useContextFile().list();
 
@@ -40,8 +40,10 @@ export const ShowCurrentContext: React.FunctionComponent = () => {
 export const AddContext: React.FunctionComponent<{ options: any, args: string[] }> = ({ args }) => {
   const [key, path] = args;
 
-  if (!key || !path) {
-    return <ContextError error={new MissingArgumentstError()} />;
+  if (!key) {
+    return <ContextError error={new MissingArgumentstError(ARGUMENTS.CONTEXT_NAME, ARGUMENTS.CONTEXT)} />;
+  } else if (!path) {
+    return <ContextError error={new MissingArgumentstError(ARGUMENTS.SPEC_PATH, ARGUMENTS.CONTEXT)} />;
   }
 
   const { response, error } = useContextFile().addContext(key, new SpecificationFile(path));
@@ -57,7 +59,7 @@ export const SetCurrent: React.FunctionComponent<{ options: any, args: string[]
   const [key,] = args;
 
   if (!key) {
-    return <ContextError error={new MissingArgumentstError()} />;
+    return <ContextError error={new MissingArgumentstError(ARGUMENTS.CONTEXT_NAME, ARGUMENTS.CONTEXT)} />;
   }
 
   const { response, error } = useContextFile().setCurrent(key);
@@ -77,7 +79,7 @@ export const RemoveContext: React.FunctionComponent<{ options: any, args: string
   const [key] = args;
 
   if (!key) {
-    return <ContextError error={new MissingArgumentstError()} />;
+    return <ContextError error={new MissingArgumentstError(ARGUMENTS.CONTEXT_NAME, ARGUMENTS.CONTEXT)} />;
   }
 
   const { response, error } = useContextFile().deleteContext(key);

src/hooks/context/models.ts

@@ -35,9 +35,9 @@ export class MissingCurrentContextError extends Error {
 }
 
 export class MissingArgumentstError extends Error {
-  constructor() {
+  constructor(argument: string, command: string) {
     super();
-    this.message = messages.MISSING_ARGUMENTS;
+    this.message = messages.MISSING_ARGUMENTS(argument, command);
   }
 }
 

src/messages.tsx

@@ -5,7 +5,7 @@ export const CONTEXT_NOT_FOUND = (contextName: string) => `Context ${contextName
 
 export const MISSING_CURRENT_CONTEXT = 'No context is set as current, please set a current context.';
 
-export const MISSING_ARGUMENTS = 'Missing arguments.';
+export const MISSING_ARGUMENTS = (argument: string, command: string) => `Missing arguments: ${argument}\nrun asyncapi ${command} --help to know more.`;
 
 export const NEW_CONTEXT_ADDED = (contextName: string) => `New context added.\n\nYou can set it as your current context: asyncapi context use ${contextName}\nYou can use this context when needed by passing ${contextName} as a parameter: asyncapi valiate ${contextName}`;
 
@@ -21,3 +21,9 @@ export const NO_SPEC_FOUND = (command: string) => `${FALLBACK_MESSAGES[command]}
 export const FALLBACK_MESSAGES: {[name: string]: string} = {
   validate: 'Unable to perform validation. Specify what AsyncAPI file should be validated.'
 };
+
+export const ARGUMENTS = {
+  CONTEXT_NAME: 'context-name',
+  SPEC_PATH: 'spec-path',
+  CONTEXT: 'context'
+};