fix(shape): Fix issue with setting option by deep path

Author: ghaiklor

src/Basic.js

@@ -17,12 +17,9 @@ export default class Shape {
   _options = {};
 
   /**
-   * Constructor is responsible for initializing base properties.
-   * Don't forgot to call `super(...args)` when extending from this class.
-   *
    * @constructor
    * @param {Object} [options]
-   * @param {String} [options.text] Text that will be rendered in shape
+   * @param {String} [options.text] Text that will be rendered in the shape
    * @param {Number} [options.width] Shape width
    * @param {Number} [options.height] Shape height
    * @param {Number} [options.x] Absolute coordinate X
@@ -56,12 +53,22 @@ export default class Shape {
   /**
    * Set new option value.
    *
-   * @param {String} path
+   * @param {String} path Path can be set with dot-notation
    * @param {*} value
    * @returns {Shape}
    */
   set(path, value) {
-    this._options[path] = value;
+    let obj = this._options;
+    let tags = path.split('.');
+    let len = tags.length - 1;
+
+    for (let i = 0; i < len; i++) {
+      if (typeof obj[tags[i]] === 'undefined') obj[tags[i]] = {};
+      obj = obj[tags[i]];
+    }
+
+    obj[tags[len]] = value;
+
     return this;
   }
 

test/unit/Basic.test.js

@@ -14,6 +14,15 @@ describe('Shape', () => {
     assert.equal(shape.getText(), 'test');
   });
 
+  it('Should properly get/set from options object', () => {
+    let shape = new Shape();
+    assert.equal(shape.get('text'), '');
+    assert.instanceOf(shape.set('animation.name', 'print'), Shape);
+    assert.equal(shape.get('animation.name'), 'print');
+    assert.instanceOf(shape.set('animation.name', 'test'), Shape);
+    assert.equal(shape.get('animation.name'), 'test');
+  });
+
   it('Should properly get/set text', () => {
     let shape = new Shape();
     assert.equal(shape.getText(), '');