Fix Help flag issue

src/__tests__/unit/util/args.test.ts

@@ -146,7 +146,7 @@ describe('util/args: ', () => {
       const result = parseArgs();
 
       expect(info).toHaveBeenCalledWith(HELP);
-      expect(result).toEqual(undefined);
+      expect(result).toBeUndefined();
     });
 
     it('returns manifest and output path.', () => {

src/index.ts

@@ -7,37 +7,34 @@ import {load} from './lib/load';
 import {parameterize} from './lib/parameterize';
 
 import {parseArgs} from './util/args';
-import {ERRORS} from './util/errors';
 import {andHandle} from './util/helpers';
 import {logger} from './util/logger';
 
 import {STRINGS} from './config';
 
 const packageJson = require('../package.json');
 
-const {CliInputError} = ERRORS;
-
-const {DISCLAIMER_MESSAGE, SOMETHING_WRONG} = STRINGS;
+const {DISCLAIMER_MESSAGE} = STRINGS;
 
 const impactEngine = async () => {
-  logger.info(DISCLAIMER_MESSAGE);
   const options = parseArgs();
 
-  if (options) {
-    const {inputPath, outputPath, paramPath} = options;
-
-    const {tree, context, parameters} = await load(inputPath, paramPath);
-    parameterize.combine(context.params, parameters);
-    const pluginStorage = await initalize(context.initialize.plugins);
-    const computedTree = await compute(tree, {context, pluginStorage});
-    const aggregatedTree = aggregate(computedTree, context.aggregation);
-    context['if-version'] = packageJson.version;
-    exhaust(aggregatedTree, context, outputPath);
-
+  if (!options) {
     return;
   }
 
-  return Promise.reject(new CliInputError(SOMETHING_WRONG));
+  logger.info(DISCLAIMER_MESSAGE);
+  const {inputPath, outputPath, paramPath} = options;
+
+  const {tree, context, parameters} = await load(inputPath!, paramPath);
+  parameterize.combine(context.params, parameters);
+  const pluginStorage = await initalize(context.initialize.plugins);
+  const computedTree = await compute(tree, {context, pluginStorage});
+  const aggregatedTree = aggregate(computedTree, context.aggregation);
+  context['if-version'] = packageJson.version;
+  exhaust(aggregatedTree, context, outputPath);
+
+  return;
 };
 
 impactEngine().catch(andHandle);

src/types/process-args.ts

@@ -4,3 +4,9 @@ export interface ManifestProcessArgs {
   'override-params'?: string;
   help?: boolean;
 }
+
+export interface ProcessArgsOutputs {
+  inputPath?: string;
+  outputPath?: string;
+  paramPath?: string;
+}

src/util/args.ts

@@ -6,7 +6,7 @@ import {ERRORS} from './errors';
 
 import {CONFIG, STRINGS} from '../config';
 
-import {ManifestProcessArgs} from '../types/process-args';
+import {ManifestProcessArgs, ProcessArgsOutputs} from '../types/process-args';
 
 const {CliInputError} = ERRORS;
 
@@ -50,7 +50,7 @@ const prependFullFilePath = (filePath: string) => {
  *    If it is, then returns object containing full path.
  * 4. If params are missing or invalid, then rejects with `CliInputError`.
  */
-export const parseArgs = () => {
+export const parseArgs = (): ProcessArgsOutputs | undefined => {
   const {
     manifest,
     output,