search

rchain-community / rchain-api

An API for rchain dapps to communicate with the blockchain
Other
22 stars 12 forks source link

Update to node 0.9.12 #70

Closed nzpr closed 3 years ago

dckc commented 5 years ago

Looks pretty good so far. Are you working on the RHOCore unit test failures?

nzpr commented 5 years ago

@dckc yes, looking into it, my current guess that this is caused by changing GPrivate to GUnforgeable in RhoTypes.proto.

dckc commented 5 years ago

@nutzipper you're right that the code builds DeployService two different ways: one at "compile" time and one at "run" time. I have work-in-progress to switch to all compile time (statically generated code)

dckc commented 5 years ago

I tried to carry this PR forward but I'm losing: https://github.com/dckc/RChain-API/tree/v0.9.12-wip 50eeb61

npm run integrationTest gives:

# smart contract deploy
not ok 5 13 INTERNAL: Received RST_STREAM with error code 2
dckc commented 5 years ago

I just pushed some earlier work: https://github.com/dckc/RChain-API/tree/rnode-0.9 088ac9c Apr 20.

The diagnostic from this version at least makes sense:

# smart contract deploy
not ok 5 12 UNIMPLEMENTED: Method not found: coop.rchain.casper.protocol.DeployService/createBlock
dckc commented 5 years ago

still not winning, but maybe getting closer...

https://github.com/dckc/RChain-API/tree/rnode-0.9 2903258

current symptoms:

# smart contract deploy
...
not ok 5 $root.coop.rchain.casper.protocol.DeployServiceResponse is not a constructor

looks similar to https://github.com/protobufjs/protobuf.js/issues/860 and perhaps https://github.com/protobufjs/protobuf.js/issues/1128

dckc commented 3 years ago

no longer supporting gRPC