search

btc-ag / service-idl

Xtext-based Service IDL (Interface Definition Language) and Code Generators for Protobuf, C++, Java and .NET
Eclipse Public License 2.0
8 stars 8 forks source link

Fixed #113: PRINS-3998: Add support for ServiceComm.NET 0.7 #209

Closed pekueble closed 5 years ago

pekueble commented 5 years ago

ServerRunner and ClientConsole work together (Not implemented exception), however there are heartbeat-exceptions logged.

pekueble commented 5 years ago

Did requested changes

sigiesec commented 5 years ago

@pekueble Most of the integration tests work, but the "event" test case is failing (also the BTC.GA.* test cases, but these probably fail for the same reason). The IDL file for the test case is at https://github.com/btc-ag/service-idl/blob/master/com.btc.serviceidl.tests/src/com/btc/serviceidl/tests/testdata/good/event.idl

Could you please give this test case a look and fix the generated code for this as well?

Details on the error can be found in https://bpaste.net/show/6128a9067054, the first syntax error is mentioned in line 725.

pekueble commented 5 years ago

@sigiesec: using event.idl I could reproduce and fix this error.

sigiesec commented 5 years ago

Thanks! The integration tests are all fine now :)