neo.mjs 2.3.16 → 3.0.0

package/src/util/VDom.mjs

@@ -14,7 +14,6 @@ class VDom extends Base {
     }}
 
     /**
-     *
      * @param {Object} vdom
      * @param {Boolean} [removeIds=true]
      * @returns {Object} cloned vdom
@@ -160,7 +159,6 @@ class VDom extends Base {
     }
 
     /**
-     *
      * @param {Object} vdom
      * @param {Number} index
      * @returns {Array}
@@ -180,7 +178,6 @@ class VDom extends Base {
     }
 
     /**
-     *
      * @param {Object} vdom
      * @param {Number} index
      * @returns {Array}
@@ -190,7 +187,6 @@ class VDom extends Base {
     }
 
     /**
-     *
      * @param {Object} vdom
      * @param {String} flag
      * @param {Array} [matchArray]
@@ -217,7 +213,6 @@ class VDom extends Base {
     }
 
     /**
-     *
      * @param {Object} vdom
      * @param {String} id
      * @param {Boolean} topLevel=true Internal flag, do not use it

package/src/util/_export.mjs

@@ -5,10 +5,11 @@ import Date             from './Date.mjs';
 import Function         from './Function.mjs';
 import HashHistory      from './HashHistory.mjs';
 import KeyNavigation    from './KeyNavigation.mjs';
+import Logger           from './Logger.mjs';
 import Matrix           from './Matrix.mjs';
 import Rectangle        from './Rectangle.mjs';
 import Style            from './Style.mjs';
 import VDom             from './VDom.mjs';
 import VNode            from './VNode.mjs';
 
-export {NeoArray, ClassSystem, Css, Date, Function, HashHistory, KeyNavigation, Matrix, Rectangle, Style, VDom, VNode};
+export {NeoArray, ClassSystem, Css, Date, Function, HashHistory, KeyNavigation, Logger, Matrix, Rectangle, Style, VDom, VNode};

package/src/vdom/Helper.mjs

@@ -579,7 +579,6 @@ class Helper extends Base {
     }
 
     /**
-     *
      * @param {Object} vnode
      */
     createStringFromVnode(vnode) {
@@ -629,7 +628,6 @@ class Helper extends Base {
     }
 
     /**
-     *
      * @param {Neo.vdom.VNode} vnode
      * @param {String} id
      * @param {Neo.vdom.VNode} parentNode
@@ -672,7 +670,6 @@ class Helper extends Base {
     }
 
     /**
-     *
      * @param {Object} opts
      * @returns {Object|Neo.vdom.VNode|null}
      */

package/src/vdom/VNode.mjs

@@ -4,7 +4,6 @@
  */
 class VNode {
     /**
-     *
      * @param config
      */
     constructor(config) {

package/src/worker/App.mjs

@@ -51,16 +51,14 @@ class App extends Base {
     }}
 
     /**
-     *
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
         Neo.applyDeltas = this.applyDeltas.bind(this); // convenience shortcut
     }
 
     /**
-     *
      * @param {String} appName
      * @param {Array|Object} deltas
      * @returns {Promise<*>}
@@ -74,7 +72,6 @@ class App extends Base {
     }
 
     /**
-     *
      * @param {JSON} data
      */
     createThemeMap(data) {
@@ -95,7 +92,6 @@ class App extends Base {
     }
 
     /**
-     *
      * @param {String} path
      * @returns {Promise}
      */
@@ -216,7 +212,6 @@ class App extends Base {
     }
 
     /**
-     *
      * @param {Object} msg
      */
     onRegisterNeoConfig(msg) {
@@ -247,7 +242,6 @@ class App extends Base {
     }
 
     /**
-     *
      * @param {Object} msg
      */
     onRegisterPort(msg) {
@@ -260,7 +254,6 @@ class App extends Base {
     }
 
     /**
-     *
      * @param {Object} data
      */
     onWindowPositionChange(data) {

package/src/worker/Base.mjs

@@ -50,11 +50,10 @@ class Base extends CoreBase {
     }}
 
     /**
-     *
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         let me = this;
 
@@ -81,7 +80,6 @@ class Base extends CoreBase {
     afterConnect() {}
 
     /**
-     *
      * @param {Object} opts
      * @returns {Object|null}
      */
@@ -170,7 +168,6 @@ class Base extends CoreBase {
     }
 
     /**
-     *
      * @param {Object} e
      */
     onMessage(e) {
@@ -198,7 +195,6 @@ class Base extends CoreBase {
     }
 
     /**
-     *
      * @param {Object} msg
      */
     onPing(msg) {
@@ -208,7 +204,6 @@ class Base extends CoreBase {
     }
 
     /**
-     *
      * @param {Object} msg
      */
     onRegisterNeoConfig(msg) {
@@ -217,7 +212,6 @@ class Base extends CoreBase {
     }
 
     /**
-     *
      * @param {String} dest app, data, main or vdom (excluding the current worker)
      * @param {Object} opts configs for Neo.worker.Message
      * @param {Array} [transfer] An optional array of Transferable objects to transfer ownership of.

package/src/worker/Canvas.mjs

@@ -49,7 +49,6 @@ class Canvas extends Base {
     }
 
     /**
-     *
      * @param {Object} data
      */
     onRegisterCanvas(data) {
@@ -63,7 +62,6 @@ class Canvas extends Base {
     }
 
     /**
-     *
      * @param {Object} msg
      */
     onRegisterNeoConfig(msg) {

package/src/worker/Manager.mjs

@@ -95,16 +95,14 @@ class Manager extends Base {
     /**
      * @param {Object} config
      */
-    constructor(config) {
-        super(config);
+    construct(config) {
+        super.construct(config);
 
         let me = this;
 
         me.detectFeatures();
 
-        if (!Neo.insideWorker) {
-            me.createWorkers();
-        }
+        !Neo.insideWorker && me.createWorkers();
 
         Neo.workerId = 'main';
 

package/src/worker/Message.mjs

@@ -7,7 +7,6 @@ import IdGenerator from '../core/IdGenerator.mjs';
  */
 class Message {
     /**
-     *
      * @param {Object} config
      */
     constructor(config) {

package/src/worker/mixin/RemoteMethodAccess.mjs

@@ -19,7 +19,6 @@ class RemoteMethodAccess extends Base {
     }}
 
     /**
-     *
      * @param {Object} remote
      * @param method
      * @returns {function(*=, *=): Promise<any>}
@@ -47,7 +46,6 @@ class RemoteMethodAccess extends Base {
     }
 
     /**
-     *
      * @param {Object} remote
      */
     onRegisterRemote(remote) {
@@ -78,7 +76,6 @@ class RemoteMethodAccess extends Base {
     }
 
     /**
-     *
      * @param {Object} msg
      */
     onRemoteMethod(msg) {

package/test/siesta/siesta.js

@@ -17,10 +17,11 @@ project.configure({
 });
 
 project.plan(
+    'tests/ClassConfigsAndFields.mjs',
     'tests/ClassSystem.mjs',
     'tests/CollectionBase.mjs',
     'tests/VdomHelper.mjs',
     'tests/VdomCalendar.mjs'
 );
 
-project.start();
+project.start();

package/test/siesta/tests/ClassConfigsAndFields.mjs

@@ -0,0 +1,307 @@
+import Neo       from '../../../src/Neo.mjs';
+import * as core from '../../../src/core/_export.mjs';
+
+class TestClass extends core.Base {
+    fieldA = 1;
+    fieldB = 2;
+
+    static getConfig() {return {
+        className: 'Neo.TestClass',
+        configA_ : 3,
+        configB_ : 4
+    }}
+
+    beforeSetConfigA(value) {
+        return this.fieldA + value;
+    }
+
+    beforeSetConfigB(value) {
+        return this.fieldB + value;
+    }
+}
+
+Neo.applyClassConfig(TestClass);
+
+StartTest(t => {
+    t.it('Default class fields inside constructors', t => {
+        t.diag("Default class");
+
+        class CtorTest {
+            fieldA = 1;
+            fieldB = 2;
+
+            constructor() {
+                t.isStrict(this.fieldA, 1, 'fieldA equals ' + 1);
+                t.isStrict(this.fieldB, 2, 'fieldB equals ' + 2);
+            }
+        }
+
+        new CtorTest();
+
+        t.diag("Default class extension");
+
+        class CtorTestExtension extends CtorTest {
+            fieldA = 3;
+            fieldB = 4;
+
+            constructor() {
+                super();
+
+                t.isStrict(this.fieldA, 3, 'fieldA equals ' + 3);
+                t.isStrict(this.fieldB, 4, 'fieldB equals ' + 4);
+            }
+        }
+
+        new CtorTestExtension();
+    });
+
+    t.it('Neo default class fields inside constructors', t => {
+        t.diag("Neo default class");
+
+        class NeoCtorTest extends core.Base {
+            fieldA = 1;
+            fieldB = 2;
+
+            construct(config) {
+                super.construct(config);
+
+                let me        = this,
+                    extension = me.extension;
+
+                t.isStrict(me.fieldA, extension ? 3 : 1, 'fieldA equals ' + (extension ? 3 : 1));
+                t.isStrict(me.fieldB, extension ? 4 : 2, 'fieldB equals ' + (extension ? 4 : 2));
+            }
+        }
+
+        Neo.applyClassConfig(NeoCtorTest);
+
+        Neo.create(NeoCtorTest);
+
+        t.diag("Neo default class extension");
+
+        class NeoCtorTestExtension extends NeoCtorTest {
+            extension = true; // flag for the base ctor tests
+            fieldA    = 3;
+            fieldB    = 4;
+
+            construct(config) {
+                super.construct(config);
+
+                t.isStrict(this.fieldA, 3, 'fieldA equals ' + 3);
+                t.isStrict(this.fieldB, 4, 'fieldB equals ' + 4);
+            }
+        }
+
+        Neo.create(NeoCtorTestExtension);
+    });
+
+    t.it('Default class fields', t => {
+        t.diag("Testing class fields");
+
+        class DefaultTestClass {
+            fieldA   = 1;
+            fieldB   = 2;
+            _configA = 3;
+            _configB = 4;
+
+            get configA() {
+                return this._configA;
+            }
+
+            set configA(value) {
+                this._configA = this.fieldA + value;
+            }
+
+            get configB() {
+                return this._configB;
+            }
+
+            set configB(value) {
+                this._configB = this.fieldB + value;
+            }
+
+            constructor(config) {
+                Object.assign(this, config);
+            }
+        }
+
+        const instance = new DefaultTestClass({
+            fieldA : 5,
+            configA: 6,
+            configB: 7,
+            fieldB : 8
+        });
+
+        // not consistent
+        t.isStrict(instance.configA, 11, 'configA equals 11'); // 5 + 6 => new value of fieldA + new value of configA
+        t.isStrict(instance.configB,  9, 'configB equals  9'); // 7 + 2 => old value of fieldB + new value of configB
+        t.isStrict(instance.fieldA,   5, 'fieldA equals   5');
+        t.isStrict(instance.fieldB,   8, 'fieldB equals   8');
+
+        t.diag('Reversed order');
+
+        const instance2 = new DefaultTestClass({
+            fieldB : 8, // reversed order
+            configB: 7,
+            configA: 6,
+            fieldA : 5
+        });
+
+        // not consistent
+        t.isStrict(instance2.configA,  7, 'configA equals  7'); // 5 + 6 => old value of fieldA + new value of configA
+        t.isStrict(instance2.configB, 15, 'configB equals 15'); // 8 + 7 => new value of fieldB + new value of configB
+        t.isStrict(instance2.fieldA,   5, 'fieldA equals   5');
+        t.isStrict(instance2.fieldB,   8, 'fieldB equals   8');
+    });
+
+    t.it('Class based class configs and fields', t => {
+        t.diag("Testing class based values");
+
+        const instance = Neo.create({
+            className: 'Neo.TestClass'
+        });
+
+        t.isStrict(instance.configA, 4, 'configA equals 4'); // 1 + 3
+        t.isStrict(instance.configB, 6, 'configB equals 6'); // 2 + 4
+        t.isStrict(instance.fieldA,  1, 'fieldA equals  1');
+        t.isStrict(instance.fieldB,  2, 'fieldB equals  2');
+    });
+
+    t.it('Instance based class configs and fields', t => {
+        t.diag("Testing values which got set on instance level");
+
+        const instance = Neo.create({
+            className: 'Neo.TestClass',
+            fieldA   : 5,
+            configA  : 6,
+            configB  : 7,
+            fieldB   : 8
+        });
+
+        t.isStrict(instance.configA, 11, 'configA equals 11'); // 5 + 6
+        t.isStrict(instance.configB, 15, 'configB equals 15'); // 8 + 7
+        t.isStrict(instance.fieldA,   5, 'fieldA equals   5');
+        t.isStrict(instance.fieldB,   8, 'fieldB equals   8');
+
+        t.diag('Reversed order');
+
+        const instance2 = Neo.create({
+            className: 'Neo.TestClass',
+            fieldB   : 8, // reversed order
+            configB  : 7,
+            configA  : 6,
+            fieldA   : 5
+        });
+
+        t.isStrict(instance2.configA, 11, 'configA equals 11'); // 5 + 6
+        t.isStrict(instance2.configB, 15, 'configB equals 15'); // 8 + 7
+        t.isStrict(instance2.fieldA,   5, 'fieldA equals   5');
+        t.isStrict(instance2.fieldB,   8, 'fieldB equals   8');
+    });
+
+    t.it('Dynamically changed class configs and fields', t => {
+        t.diag("Testing instance.set()");
+
+        const instance = Neo.create({
+            className: 'Neo.TestClass'
+        });
+
+        instance.set({
+            fieldA : 5,
+            configA: 6,
+            configB: 7,
+            fieldB : 8
+        });
+
+        t.isStrict(instance.configA, 11, 'configA equals 11'); // 5 + 6
+        t.isStrict(instance.configB, 15, 'configB equals 15'); // 8 + 7
+        t.isStrict(instance.fieldA,   5, 'fieldA equals   5');
+        t.isStrict(instance.fieldB,   8, 'fieldB equals   8');
+
+        t.diag('Reversed order');
+
+        const instance2 = Neo.create({
+            className: 'Neo.TestClass'
+        });
+
+        instance2.set({
+            fieldB : 8, // reversed order
+            configB: 7,
+            configA: 6,
+            fieldA : 5
+        });
+
+        t.isStrict(instance2.configA, 11, 'configA equals 11'); // 5 + 6
+        t.isStrict(instance2.configB, 15, 'configB equals 15'); // 8 + 7
+        t.isStrict(instance2.fieldA,   5, 'fieldA equals   5');
+        t.isStrict(instance2.fieldB,   8, 'fieldB equals   8');
+    });
+
+    t.it('Advanced configs and fields example', t => {
+        class AdvancedClass extends core.Base {
+            fieldA = 2;
+            fieldB = 1;
+
+            static getConfig() {return {
+                configA_: 0,
+                configB_: 0,
+                configC_: 0
+            }}
+
+            afterSetConfigA(value, oldValue) {
+                let me  = this,
+                    sum = me.fieldA === 1 ? 21 : 6;
+
+                t.isStrict(me.configA + me.configB + me.configC + me.fieldA + me.fieldB, sum, `sum equals ${sum}`);
+            }
+
+            afterSetConfigC(value, oldValue) {
+                let me  = this,
+                    sum = me.fieldA === 1 ? 21 : 6;
+
+                t.isStrict(me.configA + me.configB + me.configC + me.fieldA + me.fieldB, sum, `sum equals ${sum}`);
+            }
+
+            beforeSetConfigA(value) {
+                return this.fieldA + value;
+            }
+
+            beforeSetConfigB(value) {
+                let me  = this,
+                    sum = me.fieldA === 1 ? 21 : 6;
+
+                t.isStrict(me.configA + me.configB + me.configC + me.fieldA + me.fieldB, sum, `sum equals ${sum}`);
+
+                return value;
+            }
+
+            beforeSetConfigC(value) {
+                return this.fieldB + value;
+            }
+        }
+
+        Neo.applyClassConfig(AdvancedClass);
+
+        let instance = Neo.create(AdvancedClass);
+
+        instance.set({
+            fieldA : 1,
+            configA: 2,
+            configB: 3,
+            configC: 4,
+            fieldB : 5
+        });
+
+        t.diag('Reversed order');
+
+        let instance2 = Neo.create(AdvancedClass);
+
+        instance2.set({
+            fieldB : 5,
+            configC: 4,
+            configB: 3,
+            configA: 2,
+            fieldA : 1
+        });
+    });
+});

package/test/siesta/tests/ClassSystem.mjs

@@ -67,4 +67,4 @@ StartTest(t => {
             b: valueB
         });
     });
-});
+});