search

filecoin-project / builtin-actors

The Filecoin built-in actors
Other
81 stars 76 forks source link

test: miner: add niporep params cbor forms vector tests #1543

Closed rvagg closed 3 months ago

rvagg commented 4 months ago

This works agains #1537 but since that's coming from a fork and I only want to have a single commit here I'm including it as if it compiles and tests. It can be reconciled in some form later but for now I just want to get matching go-state-types implementations and tests.

### Depends on
- [ ] https://github.com/filecoin-project/ref-fvm/pull/2010
- [ ] https://github.com/filecoin-project/ref-fvm/issues/2013
- [ ] https://github.com/helix-onchain/filecoin/pull/232
rvagg commented 4 months ago

Matching go-state-types params implementation and tests @ https://github.com/filecoin-project/go-state-types/pull/270

rvagg commented 3 months ago

Updated to match FIP-0092 changes, but this depends on https://github.com/filecoin-project/ref-fvm/pull/2010 and released ref-fvm and it being updated in the helix-onchain crates too, so for now they are path dependencies.

rvagg commented 3 months ago

Cargo.toml, Cargo.lock and actors/miner/src/types.rs are all now identical to https://github.com/filecoin-project/builtin-actors/pull/1537, so this PR will just add the tests on top of that when it lands.

rvagg commented 3 months ago

landed in master in https://github.com/filecoin-project/builtin-actors/pull/1559