search

rchain / rchain-api

Obsoleted in favor of https://github.com/rchain-community/rchain-api. Use at your own risk.
Other
8 stars 4 forks source link

node 0.7.1 `toByteArray()` compatibility - more integration tests #28

Closed dckc closed 6 years ago

dckc commented 6 years ago

When we updated the protos for chan and quote (#6) , we didn't update fromJSData() and such.

It took me a while, but toByteArray() computed locally agrees with having the node do it.

This builds on your PR, @crypto-coder . I'm not sure I got your latest commits; I'll look at that now...

fixes #6 fixes #14

dckc commented 6 years ago

I see 10f5904 uses phloLimit: { value: 10000000 } and such. That's no longer in 0.7.1, which includes https://github.com/rchain/rchain/commit/bc0c4521069f6c521cb21345cfe01970e2c7a3de

dckc commented 6 years ago

I can't, for the life of me, get the protos from 0.7.1 1977b93 to work.

But npm run prepack runs all of our checks / tests now: https://gist.github.com/dckc/f3d7f4850085006b7e3c4ae9eeac3470