From 1e8bc10b04ce65554fc9b5e8c289dec7b57daf83 Mon Sep 17 00:00:00 2001 From: ZenGround0 <5515260+ZenGround0@users.noreply.github.com> Date: Tue, 25 Jun 2024 12:16:00 -0400 Subject: [PATCH] Rough draft (#12139) * FIP 0085 test * Review response --------- Co-authored-by: zenground0 --- itests/migration_test.go | 69 ++++++++++++++++++++++++++++++++++++++++ 1 file changed, 69 insertions(+) diff --git a/itests/migration_test.go b/itests/migration_test.go index c0c61d010..4d7e01d52 100644 --- a/itests/migration_test.go +++ b/itests/migration_test.go @@ -7,6 +7,7 @@ import ( "testing" "time" + "github.com/stretchr/testify/assert" "github.com/stretchr/testify/require" "github.com/filecoin-project/go-address" @@ -17,6 +18,7 @@ import ( miner11 "github.com/filecoin-project/go-state-types/builtin/v11/miner" power11 "github.com/filecoin-project/go-state-types/builtin/v11/power" adt11 "github.com/filecoin-project/go-state-types/builtin/v11/util/adt" + account "github.com/filecoin-project/go-state-types/builtin/v14/account" markettypes "github.com/filecoin-project/go-state-types/builtin/v9/market" migration "github.com/filecoin-project/go-state-types/builtin/v9/migration/test" miner9 "github.com/filecoin-project/go-state-types/builtin/v9/miner" @@ -828,3 +830,70 @@ func TestMigrationNV21(t *testing.T) { //todo @zen Direct data onboarding tests } + +func TestMigrationNV23(t *testing.T) { + kit.QuietMiningLogs() + f090Addr, err := address.NewIDAddress(90) + require.NoError(t, err) + nv23epoch := abi.ChainEpoch(100) + testClient, _, ens := kit.EnsembleMinimal(t, kit.MockProofs(), + kit.UpgradeSchedule(stmgr.Upgrade{ + Network: network.Version22, + Height: -1, + }, stmgr.Upgrade{ + Network: network.Version23, + Height: nv23epoch, + Migration: filcns.UpgradeActorsV14, + }, + )) + + ens.InterconnectAll().BeginMining(10 * time.Millisecond) + + clientApi := testClient.FullNode.(*impl.FullNodeAPI) + ctx, cancel := context.WithCancel(context.Background()) + defer cancel() + + testClient.WaitTillChain(ctx, kit.HeightAtLeast(nv23epoch+5)) + + bs := blockstore.NewAPIBlockstore(testClient) + ctxStore := gstStore.WrapBlockStore(ctx, bs) + + preMigrationTs, err := clientApi.ChainGetTipSetByHeight(ctx, nv23epoch-1, types.EmptyTSK) + require.NoError(t, err) + + root := preMigrationTs.Blocks()[0].ParentStateRoot + preStateTree, err := state.LoadStateTree(ctxStore, root) + require.NoError(t, err) + require.Equal(t, types.StateTreeVersion5, preStateTree.Version()) + + // Check f090 actor before migration + msigCodeNv22, ok := actors.GetActorCodeID(actorstypes.Version13, manifest.MultisigKey) + assert.True(t, ok) + f090ActorPre, err := preStateTree.GetActor(f090Addr) + require.NoError(t, err) + require.True(t, f090ActorPre.Code.Equals(msigCodeNv22)) + + // Get state after the migration + postMigrationTs, err := clientApi.ChainHead(ctx) + require.NoError(t, err) + postStateTree, err := state.LoadStateTree(ctxStore, postMigrationTs.Blocks()[0].ParentStateRoot) + require.NoError(t, err) + + // Check the new system actor + systemAct, err := postStateTree.GetActor(builtin.SystemActorAddr) + require.NoError(t, err) + systemCode, ok := actors.GetActorCodeID(actorstypes.Version14, manifest.SystemKey) + require.True(t, ok) + require.Equal(t, systemCode, systemAct.Code) + + // Check f090 actor after migration + f090ActorPost, err := postStateTree.GetActor(f090Addr) + require.NoError(t, err) + accountNV23, ok := actors.GetActorCodeID(actorstypes.Version14, manifest.AccountKey) + assert.True(t, ok) + require.True(t, f090ActorPost.Code.Equals(accountNV23)) + f090StatePost, err := clientApi.StateReadState(ctx, f090Addr, types.EmptyTSK) + require.NoError(t, err) + state := f090StatePost.State.(*account.State) + require.Equal(t, state.Address, f090Addr) +}