search

zeroc-ice / ice

All-in-one solution for creating networked applications with RPC, pub/sub, server deployment, and more.
https://zeroc.com
GNU General Public License v2.0
2k stars 592 forks source link

Port Ice/optional JavaScript test to TypeScript #2353

Closed pepone closed 4 days ago

pepone commented 4 days ago

This PR ports the JavaScript Ice/optional tests to TypeScript. This was the only test that still uses JavaScript.

Updated the TypeScript definitions to use ?: for optional parameters, this is only possible when optional parameters are defined after all required parameters.

It also includes a fix for importing definitions for a generated file corresponding to a Slice include. This only affected code not using "js:module:" metadata, not the Ice code which uses js:module:ice.