[GR-53444] Typo in error message "java.lang.AssertionError: Expected number of params 2, actual 2"

[illarionov]

Describe the issue

When executing a WASM function passing an incorrect number of arguments, the assertion is triggered with the following error message:

java.lang.AssertionError: Expected number of params 2, actual 2
org.graalvm.polyglot.PolyglotException: java.lang.AssertionError: Expected number of params 2, actual 2
	at org.graalvm.wasm.nodes.WasmRootNode.moveArgumentsToLocals(WasmRootNode.java:243)
	at org.graalvm.wasm.nodes.WasmRootNode.executeWithContext(WasmRootNode.java:160)
	at org.graalvm.wasm.nodes.WasmRootNode.execute(WasmRootNode.java:144)
	at <wasm> sqlite3__wasm_config_i(Unknown)
	at org.graalvm.polyglot.Value.execute(Value.java:930)

The "actual" number of arguments in this message should be 1 and not 2.

This is a typo on the line

graal/wasm/src/org.graalvm.wasm/src/org/graalvm/wasm/nodes/WasmRootNode.java

Line 251 in 62b2745

assert WasmArguments.getArgumentCount(args) == paramCount : "Expected number of params " + paramCount + ", actual " + args.length;

(args.length should be replaced with WasmArguments.getArgumentCount(args)).

Describe GraalVM and your environment:

• GraalVM version 24.0.0
• JDK major version: 21
• OS: Linux
• Architecture: AMD64

[selhagani]

Hi @illarionov, thank you for sharing this with us.

I will share this with our dev team and we'll keep you updated.

[illarionov]

I see that the merge was done in b134d48 and this issue can be closed now.