Merge pull request #103 from salesforcecli/ew/vararg-generic

fix: parse varargs response type

Author: iowillhoit

src/util.ts

@@ -62,8 +62,8 @@ export function toHelpSection(
   return { header, body };
 }
 
-export function parseVarArgs(args: Record<string, unknown>, argv: string[]): Record<string, unknown> {
-  const final: Record<string, unknown> = {};
+export function parseVarArgs(args: Record<string, unknown>, argv: string[]): Record<string, string | undefined> {
+  const final: Record<string, string | undefined> = {};
   const argVals = Object.values(args);
 
   // Remove arguments from varargs

test/unit/util.test.ts

@@ -93,6 +93,16 @@ describe('parseVarArgs', () => {
     expect(varargs).to.deep.equal({ key1: 'value1' });
   });
 
+  it('should parse multiple varargs', () => {
+    const varargs = parseVarArgs({}, ['key1=value1', 'key2=value2']);
+    expect(varargs).to.deep.equal({ key1: 'value1', key2: 'value2' });
+  });
+
+  it('should allow an empty value', () => {
+    const varargs = parseVarArgs({}, ['key1=']);
+    expect(varargs).to.deep.equal({ key1: undefined });
+  });
+
   it('should parse varargs and not arguments', () => {
     const varargs = parseVarArgs({ arg1: 'foobar' }, ['foobar', 'key1=value1']);
     expect(varargs).to.deep.equal({ key1: 'value1' });