fix: argument validation of parameters with defaults, Map and Set #7435
Conversation
|
@zwave-js-bot automerge |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
| class Test { | ||
| @validateArgs() | ||
| foo(arg1: number = 5): void { | ||
| arg1; |
Check warning
Code scanning / CodeQL
Expression has no effect Warning test
|
|
||
| @validateArgs() | ||
| bar(arg1: number | string = 5): void { | ||
| arg1; |
Check warning
Code scanning / CodeQL
Expression has no effect Warning test
| class Test { | ||
| @validateArgs() | ||
| map(arg1: Map<any, any>): void { | ||
| arg1; |
Check warning
Code scanning / CodeQL
Expression has no effect Warning test
|
|
||
| @validateArgs() | ||
| readonlyMap(arg1: ReadonlyMap<any, any>): void { | ||
| arg1; |
Check warning
Code scanning / CodeQL
Expression has no effect Warning test
|
|
||
| @validateArgs() | ||
| set(arg1: Set<any>): void { | ||
| arg1; |
Check warning
Code scanning / CodeQL
Expression has no effect Warning test
|
|
||
| @validateArgs() | ||
| readonlySet(arg1: ReadonlySet<any>): void { | ||
| arg1; |
Check warning
Code scanning / CodeQL
Expression has no effect Warning test
### Bugfixes * Fixed another issue where some CC API methods would incorrectly fail validation of their arguments, causing the node interview to fail (#7435)
fixes the issue reported in #7430 (reply in thread)