[JavaScript] add xlang tests to ensure protocol consistency between JavaScript and Java

[chaokunyang]

Feature Request

Currently we have Xlang tests for python/rust/c++/go:

• src/test/java/org/apache/fory/xlang/GoXlangTest.java
• src/test/java/org/apache/fory/xlang/CPPXlangTest.java
• src/test/java/org/apache/fory/xlang/RustXlangTest.java
• src/test/java/org/apache/fory/xlang/PythonXlangTest.java

We need to add JavaScriptXlangTest.java and align javascript implementation with current xlang_serialization_spec.md

Is your feature request related to a problem? Please describe

No response

Describe the solution you'd like

No response

Describe alternatives you've considered

No response

Additional context

No response

[theweipeng]

I’m starting work on this issue.

[dheeraj12347]

Hi @theweipeng, @chaokunyang 👋
I noticed this issue is still open and there have not been any updates since your comment.
I’m very interested in contributing to Apache Fory’s JavaScript xlang support and would be happy to help with this.

My plan would be:
– Mirror the existing Xlang tests for Python/Rust/C++/Go under src/test/java/org/apache/fory/xlang/JavaScriptXlangTest.java.
– Compare the JavaScript implementation with the xlang serialization spec and Java reference behavior.
– Add/adjust JS code so xlang serialization is fully consistent with the spec, then update tests accordingly.

Would it be okay if I start working on this issue? If you already have work in progress, I’d also be happy to help review or contribute to that instead.

[theweipeng]

Hi @theweipeng, @chaokunyang 👋 I noticed this issue is still open and there have not been any updates since your comment. I’m very interested in contributing to Apache Fory’s JavaScript xlang support and would be happy to help with this.

My plan would be: – Mirror the existing Xlang tests for Python/Rust/C++/Go under src/test/java/org/apache/fory/xlang/JavaScriptXlangTest.java. – Compare the JavaScript implementation with the xlang serialization spec and Java reference behavior. – Add/adjust JS code so xlang serialization is fully consistent with the spec, then update tests accordingly.

Would it be okay if I start working on this issue? If you already have work in progress, I’d also be happy to help review or contribute to that instead.

Yes, It's okay for you to start working on this issue.
Can you please compare JavaScript implementation with the xlang serialization spec base types in (https://fory.apache.org/docs/next/specification/xlang_serialization_spec#data-types) first, and implement this part.

[theweipeng]

Hi @theweipeng, @chaokunyang 👋 I noticed this issue is still open and there have not been any updates since your comment. I’m very interested in contributing to Apache Fory’s JavaScript xlang support and would be happy to help with this.
My plan would be: – Mirror the existing Xlang tests for Python/Rust/C++/Go under src/test/java/org/apache/fory/xlang/JavaScriptXlangTest.java. – Compare the JavaScript implementation with the xlang serialization spec and Java reference behavior. – Add/adjust JS code so xlang serialization is fully consistent with the spec, then update tests accordingly.
Would it be okay if I start working on this issue? If you already have work in progress, I’d also be happy to help review or contribute to that instead.

Yes, It's okay for you to start working on this issue. Can you please compare JavaScript implementation with the xlang serialization spec base types in (https://fory.apache.org/docs/next/specification/xlang_serialization_spec#data-types) first, and implement this part.

@dheeraj12347 I am now working in adding JavaScriptXlangTest.java for JavaScript.

[dheeraj12347]

Thanks for the clarification and for sharing the spec link.

Since you are already working on JavaScriptXlangTest.java and opened PR #3161, I’d like to focus on the JavaScript implementation side. My plan is:
– Compare the current JavaScript xlang implementation with the “Data Types” section of the xlang serialization spec and the Java reference behavior.
– Fix or extend the JavaScript encoder/decoder so that all base types follow the spec exactly.
– Add or update JavaScript tests to cover these base types, without duplicating the Java JavaScriptXlangTest.java work you’re doing in #3161.

Please let me know if this division of work sounds good, or if you’d prefer me to help in a different way.