Skip to content

Commit

Permalink
Refactor fix
Browse files Browse the repository at this point in the history
  • Loading branch information
MissingNO57 committed Oct 17, 2024
1 parent 8d49847 commit 4b0a092
Showing 1 changed file with 12 additions and 12 deletions.
24 changes: 12 additions & 12 deletions app/upgrade_cudos.go
Original file line number Diff line number Diff line change
Expand Up @@ -164,7 +164,12 @@ func LoadCudosGenesis(app *App, manifest *UpgradeManifest) (*map[string]interfac
}

func ProcessSourceNetworkGenesis(logger log.Logger, cudosCfg *CudosMergeConfig, genesisData *GenesisData, manifest *UpgradeManifest) error {
err := genesisUpgradeWithdrawIBCChannelsBalances(genesisData, cudosCfg, manifest)
err := writeInitialBalancesToManifest(genesisData, manifest)
if err != nil {
return fmt.Errorf("cudos merge: failed to write initial balances to manifest: %w", err)
}

err = genesisUpgradeWithdrawIBCChannelsBalances(genesisData, cudosCfg, manifest)
if err != nil {
return fmt.Errorf("cudos merge: failed to withdraw IBC channels balances: %w", err)
}
Expand Down Expand Up @@ -194,6 +199,11 @@ func ProcessSourceNetworkGenesis(logger log.Logger, cudosCfg *CudosMergeConfig,
return fmt.Errorf("cudos merge: failed to move funds: %w", err)
}

err = writeMovedBalancesToManifest(genesisData, manifest)
if err != nil {
return fmt.Errorf("cudos merge: failed to write moved balances to manifest")
}

return nil
}

Expand Down Expand Up @@ -294,12 +304,7 @@ func CudosMergeUpgradeHandler(app *App, ctx sdk.Context, cudosCfg *CudosMergeCon
return fmt.Errorf("cudos merge: cudos path not set")
}

err := writeInitialBalancesToManifest(genesisData, manifest)
if err != nil {
return fmt.Errorf("cudos merge: failed to write initial balances to manifest: %w", err)
}

err = ProcessSourceNetworkGenesis(app.Logger(), cudosCfg, genesisData, manifest)
err := ProcessSourceNetworkGenesis(app.Logger(), cudosCfg, genesisData, manifest)
if err != nil {
return err
}
Expand All @@ -314,11 +319,6 @@ func CudosMergeUpgradeHandler(app *App, ctx sdk.Context, cudosCfg *CudosMergeCon
return fmt.Errorf("cudos merge: failed process delegations: %w", err)
}

err = writeMovedBalancesToManifest(genesisData, manifest)
if err != nil {
return fmt.Errorf("cudos merge: failed to write moved balances to manifest")
}

err = verifySupply(app, ctx, cudosCfg, manifest)
if err != nil {
return fmt.Errorf("cudos merge: failed to verify supply: %w", err)
Expand Down

0 comments on commit 4b0a092

Please sign in to comment.