Merge pull request #25 from SergeAstapov/changes-getter

Author: snewcomer

src/utils/get-key-values.ts

@@ -2,32 +2,28 @@ import isObject from './is-object';
 import Err from '../-private/err';
 import { PublicErrors } from '../types';
 
-let keysUpToValue: string[] = [];
-
 /**
  * traverse through target and return leaf nodes with `value` property and key as 'person.name'
  *
  * @method getKeyValues
  * @return {Array} [{ 'person.name': value }]
  */
-export function getKeyValues<T extends Record<string, any>>(obj: T): Record<string, any>[] {
+export function getKeyValues<T extends Record<string, any>>(
+  obj: T,
+  keysUpToValue: Array<string> = []
+): Record<string, any>[] {
   const map = [];
 
   for (let key in obj) {
-    keysUpToValue.push(key);
-
     if (obj[key] && isObject(obj[key])) {
       if (Object.prototype.hasOwnProperty.call(obj[key], 'value')) {
-        map.push({ key: keysUpToValue.join('.'), value: obj[key].value });
-        // stop collecting keys
-        keysUpToValue = [];
+        map.push({ key: [...keysUpToValue, key].join('.'), value: obj[key].value });
       } else if (key !== 'value') {
-        map.push(...getKeyValues(obj[key]));
+        map.push(...getKeyValues(obj[key], [...keysUpToValue, key]));
       }
     }
   }
 
-  keysUpToValue = [];
   return map;
 }
 

test/utils/get-key-values.test.ts

@@ -0,0 +1,29 @@
+import { getKeyValues } from '../../src/utils/get-key-values';
+
+describe('Unit | Utility | getKeyValues', function() {
+  it('it works with single level values', () => {
+    const result = getKeyValues({ test: { value: 1 } });
+
+    expect(result).toEqual([{ key: 'test', value: 1 }]);
+  });
+
+  it('it works with nested keys', () => {
+    const result = getKeyValues({
+      user: {
+        firstName: { value: 'Michael' },
+        lastName: { value: 'Bolton' },
+        address: {
+          city: { value: 'NYC' },
+          state: { value: 'New York' }
+        }
+      }
+    });
+
+    expect(result).toEqual([
+      { key: 'user.firstName', value: 'Michael' },
+      { key: 'user.lastName', value: 'Bolton' },
+      { key: 'user.address.city', value: 'NYC' },
+      { key: 'user.address.state', value: 'New York' }
+    ]);
+  });
+});