Merge pull request #2 from explore-node-js/master

sync. with main repository

Author: eugene-matvejev

README.md

@@ -1,8 +1,23 @@
 # node.js-object-field-resolver
-allows overwrite object fields with value
+allows overwrite object field value using abstract path
 
+## software requirements
+ * node.js 6.9+ [with v8 enabled]
+ * npm 3+
 
-how to use:
+## used technologies
+ * jest
+ 
+## how to execute tests
+```
+npm test
+```
+to execute tests with coverage
+```
+npm test -- --coverage
+```
+
+## how to use
 
 ```
 import overwriteObjectField from "node-object-field-resolver"

src/overwrite_object_field_value.js

@@ -10,7 +10,7 @@ module.exports = function overwriteObjectFieldValue(abstractPath, value, object,
     for (let i = 0; i <= lastPathIndex; i++) {
         const index = indexes[i];
 
-        if (undefined === obj[index]) {
+        if (undefined === obj[index] || !(obj[index] instanceof Object)) {
             obj[index] = {};
         }
 

tests/overwrite_object_field_value.test.js

@@ -14,6 +14,12 @@ describe(`resolver: object field`, () => {
             object: { node1: ['test'] },
             expected: { node1: {} }
         },
+        {
+            absolutePath: 'node1.node2',
+            value: 'string',
+            object: { node1: 'value' },
+            expected: { node1: { node2: 'string' } }
+        },
         {
             absolutePath: 'node1',
             value: {},