Merge pull request #160 from schemepunk/delimit-fix

fix(delimit): delimit values fails if value is empty
schemepunk · Jun 20, 2019 · 9e7cfc5 · 9e7cfc5
2 parents 75a9df7 + dbf656d
commit 9e7cfc5
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/__tests__/plugins/transform/delimitValues.js b/__tests__/plugins/transform/delimitValues.js
@@ -39,4 +39,11 @@ describe('Delimit Values tests', () => {
     };
     expect(await delimitValues.transform(value)).toBe('test1-test2-test3');
   });
+  test('no value is ok.', async () => {
+    // Emulate a custom option from a scheme.
+    delimitValues.options = {
+      sourceDelimiter: '-'
+    };
+    expect(await delimitValues.transform()).toBe('');
+  });
 });
diff --git a/lib/plugins/transform/delimitValues.js b/lib/plugins/transform/delimitValues.js
@@ -18,7 +18,7 @@ module.exports = superclass => class extends superclass {
      * @param value
      *  A value to perform a transformation upon.
      */
-  async transform(value) {
+  async transform(value = []) {
     const delimiter = _.get(this, 'options.sourceDelimiter', ',');
     const tempValue = value.join(delimiter);
     // Call super.transform and pass along the new value to honor composition.