only log that undefiend is invalid, because devs had no prev warning …

…because it was bugged

- fallback to old yargs, because it requires node12

lib/adapter.js

@@ -6090,7 +6090,9 @@ function Adapter(options) {
 
             if (state.val === undefined && !Object.keys(state).length) {
                 // undefined is not allowed as state.val -> return
-                return tools.maybeCallbackWithError(callback, 'undefined is not a valid state value');
+                this.log.info('undefined is not a valid state value')
+                // TODO: reactivate line below + test in in next controller version (02.05.2021)
+                // return tools.maybeCallbackWithError(callback, 'undefined is not a valid state value');
             }
 
             if (ack !== undefined) {
@@ -6272,7 +6274,9 @@ function Adapter(options) {
 
             if (state.val === undefined && !Object.keys(state).length) {
                 // undefined is not allowed as state.val -> return
-                return tools.maybeCallbackWithError(callback, 'undefined is not a valid state value');
+                this.log.info('undefined is not a valid state value')
+                // TODO: reactivate line below + test in in next controller version (02.05.2021)
+                // return tools.maybeCallbackWithError(callback, 'undefined is not a valid state value');
             }
 
             if (ack !== undefined) {
@@ -6372,7 +6376,9 @@ function Adapter(options) {
 
             if (state.val === undefined && !Object.keys(state).length) {
                 // undefined is not allowed as state.val -> return
-                return tools.maybeCallbackWithError(callback, 'undefined is not a valid state value');
+                this.log.info('undefined is not a valid state value')
+                // TODO: reactivate line below + test in in next controller version (02.05.2021)
+                // return tools.maybeCallbackWithError(callback, 'undefined is not a valid state value');
             }
 
             if (ack !== undefined) {
@@ -6594,7 +6600,9 @@ function Adapter(options) {
 
             if (state.val === undefined && !Object.keys(state).length) {
                 // undefined is not allowed as state.val -> return
-                return tools.maybeCallbackWithError(callback, 'undefined is not a valid state value');
+                this.log.info('undefined is not a valid state value')
+                // TODO: reactivate line below + test in in next controller version (02.05.2021)
+                // return tools.maybeCallbackWithError(callback, 'undefined is not a valid state value');
             }
 
             if (ack !== undefined) {

package.json

@@ -52,7 +52,7 @@
     "tar": "^6.1.0",
     "winston": "^3.3.3",
     "winston-daily-rotate-file": "^4.5.2",
-    "yargs": "^17.0.0"
+    "yargs": "^16.2.0"
   },
   "devDependencies": {
     "@alcalzone/release-script": "^1.10.0",

test/lib/testStates.js

@@ -853,6 +853,8 @@ function register(it, expect, context) {
         });
     });
 
+    /**
+     TODO: Reactivate with next controller (02.05.2021)
     it(testName + 'Should decline undefined state value', async () => {
         // set state to 1
         await context.adapter.setStateAsync(`${gid}undefinedState`, 1);
@@ -870,6 +872,7 @@ function register(it, expect, context) {
             }
         }
     });
+     */
 
     it(testName + 'Should also set object id', async () => {
         // set state with device, channel, state