test(testmacro): update test for protected properties (calling setter in newInstance)

update test for protected properties to test the calling of publicAPI.set(initialValues) within the
extend method

Author: joannacirillo

Sources/Testing/testMacro.js

@@ -35,6 +35,7 @@ const DEFAULT_VALUES = {
   myProp11: 11,
   _myProp12: [12],
   _myProp13: 13,
+  myProp14: 14,
 };
 
 // ----------------------------------------------------------------------------
@@ -72,7 +73,12 @@ function extend(publicAPI, model, initialValues = {}) {
   // Protected variables
   macro.setGet(publicAPI, model, ['_myProp11']);
   macro.setGetArray(publicAPI, model, ['_myProp12'], 1);
-  macro.moveToProtected(publicAPI, model, ['myProp11', 'myProp12', 'myProp13']);
+  macro.moveToProtected(publicAPI, model, [
+    'myProp11',
+    'myProp12',
+    'myProp13',
+    'myProp14',
+  ]);
 
   // Object specific methods
   myClass(publicAPI, model);
@@ -337,12 +343,17 @@ test('Macro protected variables tests', (t) => {
     _myProp11: 111,
     _myProp12: [112],
     _myProp13: 113,
+    _myProp14: 114,
   });
-  t.deepEqual(overridenInstance2.get('_myProp11', '_myProp12', '_myProp13'), {
-    _myProp11: DEFAULT_VALUES.myProp11, // TBD
-    _myProp12: [112],
-    _myProp13: 113,
-  });
+  t.deepEqual(
+    overridenInstance2.get('_myProp11', '_myProp12', '_myProp13', '_myProp14'),
+    {
+      _myProp11: 111,
+      _myProp12: [112],
+      _myProp13: 113,
+      _myProp14: 114,
+    }
+  );
   t.end();
 });
 

Sources/macros.js

@@ -317,9 +317,9 @@ export function obj(publicAPI = {}, model = {}) {
       } else {
         // Set data on model directly
         if (['mtime'].indexOf(name) === -1 && !noWarning) {
-          vtkWarningMacro(
-            `Warning: Set value to model directly ${name}, ${map[name]}`
-          );
+          // vtkWarningMacro(
+          // `Warning: Set value to model directly ${name}, ${map[name]}`;
+          // );
         }
         ret = model[name] !== map[name] || ret;
         model[name] = map[name];