Colin/supply genesis

client/lcd/swagger-ui/swagger.yaml

@@ -1862,7 +1862,7 @@ paths:
         200:
           description: OK
           schema:
-             type: integer
+             type: string
         400:
           description: Invalid coin denomination
         500:

docs/spec/supply/01_concepts.md

@@ -30,9 +30,10 @@ The `ModuleAccount` interface is defined as follows:
 
 ```go
 type ModuleAccount interface {
-  auth.Account            // same methods as the Account interface
-  GetName() string        // name of the module; used to obtain the address
-  GetPermission() string  // permission of module account (minter/burner/holder)
+  auth.Account               // same methods as the Account interface
+  GetName() string           // name of the module; used to obtain the address
+  GetPermissions() []string  // permissions of module account 
+  HasPermission(string) bool 
 }
 ```
 
@@ -51,10 +52,11 @@ Each `ModuleAccount` has a different set of permissions that provide different
 object capabilities to perform certain actions. Permissions need to be
 registered upon the creation of the supply `Keeper` so that every time a
 `ModuleAccount` calls the allowed functions, the `Keeper` can lookup the
-permission to that specific account and perform or not the action.
+permissions to that specific account and perform or not the action.
 
 The available permissions are:
 
 - `Basic`: is allowed to only transfer its coins to other accounts.
 - `Minter`: allows for a module to mint a specific amount of coins as well as perform the `Basic` permissioned actions.
 - `Burner`: allows for a module to burn a specific amount of coins as well as perform the `Basic` permissioned actions.
+- `Staking`: allows for a module to delegate and undelegate a specific amount of coins.

simapp/app.go

@@ -144,15 +144,19 @@ func NewSimApp(logger log.Logger, db dbm.DB, traceStore io.Writer, loadLatest bo
 	crisisSubspace := app.paramsKeeper.Subspace(crisis.DefaultParamspace)
 
 	// account permissions
-	basicModuleAccs := []string{auth.FeeCollectorName, distr.ModuleName}
-	minterModuleAccs := []string{mint.ModuleName}
-	burnerModuleAccs := []string{staking.BondedPoolName, staking.NotBondedPoolName, gov.ModuleName}
+	maccPerms := map[string][]string{
+		auth.FeeCollectorName:     []string{supply.Basic},
+		distr.ModuleName:          []string{supply.Basic},
+		mint.ModuleName:           []string{supply.Minter},
+		staking.BondedPoolName:    []string{supply.Burner, supply.Staking},
+		staking.NotBondedPoolName: []string{supply.Burner, supply.Staking},
+		gov.ModuleName:            []string{supply.Burner},
+	}
 
 	// add keepers
 	app.accountKeeper = auth.NewAccountKeeper(app.cdc, app.keyAccount, authSubspace, auth.ProtoBaseAccount)
 	app.bankKeeper = bank.NewBaseKeeper(app.accountKeeper, bankSubspace, bank.DefaultCodespace)
-	app.supplyKeeper = supply.NewKeeper(app.cdc, app.keySupply, app.accountKeeper,
-		app.bankKeeper, supply.DefaultCodespace, basicModuleAccs, minterModuleAccs, burnerModuleAccs)
+	app.supplyKeeper = supply.NewKeeper(app.cdc, app.keySupply, app.accountKeeper, app.bankKeeper, supply.DefaultCodespace, maccPerms)
 	stakingKeeper := staking.NewKeeper(app.cdc, app.keyStaking, app.tkeyStaking,
 		app.supplyKeeper, stakingSubspace, staking.DefaultCodespace)
 	app.mintKeeper = mint.NewKeeper(app.cdc, app.keyMint, mintSubspace, &stakingKeeper, app.supplyKeeper, auth.FeeCollectorName)

x/auth/test_common.go

@@ -3,9 +3,9 @@ package auth
 
 import (
 	abci "github.com/tendermint/tendermint/abci/types"
+	"github.com/tendermint/tendermint/crypto"
 	dbm "github.com/tendermint/tendermint/libs/db"
 	"github.com/tendermint/tendermint/libs/log"
-	"github.com/tendermint/tendermint/crypto"
 
 	"github.com/cosmos/cosmos-sdk/codec"
 	"github.com/cosmos/cosmos-sdk/store"
@@ -25,18 +25,28 @@ type testInput struct {
 // moduleAccount defines an account for modules that holds coins on a pool
 type moduleAccount struct {
 	*types.BaseAccount
-	Name       string `json:"name" yaml:"name"`            // name of the module
-	Permission string `json:"permission" yaml"permission"` // permission of module account (minter/burner/holder)
+	name        string   `json:"name" yaml:"name"`              // name of the module
+	permissions []string `json:"permissions" yaml"permissions"` // permissions of module account
+}
+
+// HasPermission returns whether or not the module account has permission.
+func (ma moduleAccount) HasPermission(permission string) bool {
+	for _, perm := range ma.permissions {
+		if perm == permission {
+			return true
+		}
+	}
+	return false
 }
 
 // GetName returns the the name of the holder's module
 func (ma moduleAccount) GetName() string {
-	return ma.Name
+	return ma.name
 }
 
-// GetPermission returns permission granted to the module account (holder/minter/burner)
-func (ma moduleAccount) GetPermission() string {
-	return ma.Permission
+// GetPermissions returns permissions granted to the module account
+func (ma moduleAccount) GetPermissions() []string {
+	return ma.permissions
 }
 
 func setupTestInput() testInput {
@@ -125,8 +135,8 @@ func (sk DummySupplyKeeper) GetModuleAccount(ctx sdk.Context, moduleName string)
 	// create a new module account
 	macc := &moduleAccount{
 		BaseAccount: &baseAcc,
-		Name:        moduleName,
-		Permission:  "basic",
+		name:        moduleName,
+		permissions: []string{"basic"},
 	}
 
 	maccI := (sk.ak.NewAccount(ctx, macc)).(exported.ModuleAccountI)

x/distribution/keeper/test_common.go

@@ -120,8 +120,13 @@ func CreateTestInputAdvanced(t *testing.T, isCheckTx bool, initPower int64,
 	ctx := sdk.NewContext(ms, abci.Header{ChainID: "foochainid"}, isCheckTx, log.NewNopLogger())
 	accountKeeper := auth.NewAccountKeeper(cdc, keyAcc, pk.Subspace(auth.DefaultParamspace), auth.ProtoBaseAccount)
 	bankKeeper := bank.NewBaseKeeper(accountKeeper, pk.Subspace(bank.DefaultParamspace), bank.DefaultCodespace)
-	supplyKeeper := supply.NewKeeper(cdc, keySupply, accountKeeper, bankKeeper, supply.DefaultCodespace,
-		[]string{auth.FeeCollectorName, types.ModuleName}, []string{}, []string{staking.NotBondedPoolName, staking.BondedPoolName})
+	maccPerms := map[string][]string{
+		auth.FeeCollectorName:     []string{supply.Basic},
+		types.ModuleName:          []string{supply.Basic},
+		staking.NotBondedPoolName: []string{supply.Burner, supply.Staking},
+		staking.BondedPoolName:    []string{supply.Burner, supply.Staking},
+	}
+	supplyKeeper := supply.NewKeeper(cdc, keySupply, accountKeeper, bankKeeper, supply.DefaultCodespace, maccPerms)
 
 	sk := staking.NewKeeper(cdc, keyStaking, tkeyStaking, supplyKeeper, pk.Subspace(staking.DefaultParamspace), staking.DefaultCodespace)
 	sk.SetParams(ctx, staking.DefaultParams())
@@ -140,8 +145,8 @@ func CreateTestInputAdvanced(t *testing.T, isCheckTx bool, initPower int64,
 
 	// create module accounts
 	feeCollectorAcc := supply.NewEmptyModuleAccount(auth.FeeCollectorName, supply.Basic)
-	notBondedPool := supply.NewEmptyModuleAccount(staking.NotBondedPoolName, supply.Burner)
-	bondPool := supply.NewEmptyModuleAccount(staking.BondedPoolName, supply.Burner)
+	notBondedPool := supply.NewEmptyModuleAccount(staking.NotBondedPoolName, supply.Burner, supply.Staking)
+	bondPool := supply.NewEmptyModuleAccount(staking.BondedPoolName, supply.Burner, supply.Staking)
 	distrAcc := supply.NewEmptyModuleAccount(types.ModuleName, supply.Basic)
 
 	keeper.supplyKeeper.SetModuleAccount(ctx, feeCollectorAcc)

x/genaccounts/genesis_account.go

@@ -27,8 +27,8 @@ type GenesisAccount struct {
 	EndTime          int64     `json:"end_time" yaml:"end_time"`                   // vesting end time (UNIX Epoch time)
 
 	// module account fields
-	ModuleName       string `json:"module_name" yaml:"module_name"`             // name of the module account
-	ModulePermission string `json:"module_permission" yaml:"module_permission"` // permission of module account
+	ModuleName        string   `json:"module_name" yaml:"module_name"`               // name of the module account
+	ModulePermissions []string `json:"module_permissions" yaml:"module_permissions"` // permissions of module account
 }
 
 // Validate checks for errors on the vesting and module account parameters
@@ -53,20 +53,20 @@ func (ga GenesisAccount) Validate() error {
 // NewGenesisAccountRaw creates a new GenesisAccount object
 func NewGenesisAccountRaw(address sdk.AccAddress, coins,
 	vestingAmount sdk.Coins, vestingStartTime, vestingEndTime int64,
-	module, permission string) GenesisAccount {
+	module string, permissions ...string) GenesisAccount {
 
 	return GenesisAccount{
-		Address:          address,
-		Coins:            coins,
-		Sequence:         0,
-		AccountNumber:    0, // ignored set by the account keeper during InitGenesis
-		OriginalVesting:  vestingAmount,
-		DelegatedFree:    sdk.Coins{}, // ignored
-		DelegatedVesting: sdk.Coins{}, // ignored
-		StartTime:        vestingStartTime,
-		EndTime:          vestingEndTime,
-		ModuleName:       module,
-		ModulePermission: permission,
+		Address:           address,
+		Coins:             coins,
+		Sequence:          0,
+		AccountNumber:     0, // ignored set by the account keeper during InitGenesis
+		OriginalVesting:   vestingAmount,
+		DelegatedFree:     sdk.Coins{}, // ignored
+		DelegatedVesting:  sdk.Coins{}, // ignored
+		StartTime:         vestingStartTime,
+		EndTime:           vestingEndTime,
+		ModuleName:        module,
+		ModulePermissions: permissions,
 	}
 }
 
@@ -102,7 +102,7 @@ func NewGenesisAccountI(acc authexported.Account) (GenesisAccount, error) {
 		gacc.EndTime = acc.GetEndTime()
 	case supplyexported.ModuleAccountI:
 		gacc.ModuleName = acc.GetName()
-		gacc.ModulePermission = acc.GetPermission()
+		gacc.ModulePermissions = acc.GetPermissions()
 	}
 
 	return gacc, nil
@@ -131,7 +131,7 @@ func (ga *GenesisAccount) ToAccount() auth.Account {
 
 	// module accounts
 	if ga.ModuleName != "" {
-		return supply.NewModuleAccount(bacc, ga.ModuleName, ga.ModulePermission)
+		return supply.NewModuleAccount(bacc, ga.ModuleName, ga.ModulePermissions...)
 	}
 
 	return bacc

x/genaccounts/legacy/v0_36/migrate.go

@@ -20,9 +20,10 @@ const (
 	feeCollectorName  = "fee_collector"
 	mintModuleName    = "mint"
 
-	basic  = "basic"
-	minter = "minter"
-	burner = "burner"
+	basic   = "basic"
+	minter  = "minter"
+	burner  = "burner"
+	staking = "staking"
 )
 
 // Migrate accepts exported genesis state from v0.34 and migrates it to v0.36
@@ -62,7 +63,7 @@ func Migrate(
 				NewGenesisAccount(
 					acc.Address, acc.Coins, acc.Sequence,
 					acc.OriginalVesting, acc.DelegatedFree, acc.DelegatedVesting,
-					acc.StartTime, acc.EndTime, "", "",
+					acc.StartTime, acc.EndTime, "", []string{},
 				),
 			)
 		}
@@ -125,32 +126,32 @@ func Migrate(
 	feeCollectorModuleAcc := NewGenesisAccount(
 		feeCollectorAddr, fees, 0,
 		sdk.Coins{}, sdk.Coins{}, sdk.Coins{},
-		0, 0, feeCollectorName, basic,
+		0, 0, feeCollectorName, []string{basic},
 	)
 	govModuleAcc := NewGenesisAccount(
 		govAddr, govCoins, 0,
 		sdk.Coins{}, sdk.Coins{}, sdk.Coins{},
-		0, 0, govModuleName, burner,
+		0, 0, govModuleName, []string{burner},
 	)
 	distrModuleAcc := NewGenesisAccount(
 		distrAddr, distrCoins, 0,
 		sdk.Coins{}, sdk.Coins{}, sdk.Coins{},
-		0, 0, distrModuleName, basic,
+		0, 0, distrModuleName, []string{basic},
 	)
 	bondedModuleAcc := NewGenesisAccount(
 		bondedAddr, bondedCoins, 0,
 		sdk.Coins{}, sdk.Coins{}, sdk.Coins{},
-		0, 0, bondedPoolName, burner,
+		0, 0, bondedPoolName, []string{burner, staking},
 	)
 	notBondedModuleAcc := NewGenesisAccount(
 		notBondedAddr, notBondedCoins, 0,
 		sdk.Coins{}, sdk.Coins{}, sdk.Coins{},
-		0, 0, notBondedPoolName, burner,
+		0, 0, notBondedPoolName, []string{burner, staking},
 	)
 	mintModuleAcc := NewGenesisAccount(
 		mintAddr, sdk.Coins{}, 0,
 		sdk.Coins{}, sdk.Coins{}, sdk.Coins{},
-		0, 0, mintModuleName, minter,
+		0, 0, mintModuleName, []string{minter},
 	)
 
 	newGenState = append(

x/genaccounts/legacy/v0_36/types.go

@@ -12,19 +12,19 @@ const (
 
 type (
 	GenesisAccount struct {
-		Address       sdk.AccAddress `json:"address"`
-		Coins         sdk.Coins      `json:"coins"`
-		Sequence      uint64         `json:"sequence_number"`
-		AccountNumber uint64         `json:"account_number"`
-
-		OriginalVesting  sdk.Coins `json:"original_vesting"`
-		DelegatedFree    sdk.Coins `json:"delegated_free"`
-		DelegatedVesting sdk.Coins `json:"delegated_vesting"`
-		StartTime        int64     `json:"start_time"`
-		EndTime          int64     `json:"end_time"`
-
-		ModuleName       string `json:"module_name"`
-		ModulePermission string `json:"module_permission"`
+		Address       sdk.AccAddress `json:"address" yaml:"address"`
+		Coins         sdk.Coins      `json:"coins" yaml:"coins"`
+		Sequence      uint64         `json:"sequence_number" yaml:"sequence_number"`
+		AccountNumber uint64         `json:"account_number" yaml:"account_number"`
+
+		OriginalVesting  sdk.Coins `json:"original_vesting" yaml:"original_vesting"`
+		DelegatedFree    sdk.Coins `json:"delegated_free" yaml:"delegated_free"`
+		DelegatedVesting sdk.Coins `json:"delegated_vesting" yaml:"delegated_vesting"`
+		StartTime        int64     `json:"start_time" yaml:"start_time"`
+		EndTime          int64     `json:"end_time" yaml:"end_time"`
+
+		ModuleName        string   `json:"module_name" yaml:"module_name"`
+		ModulePermissions []string `json:"module_permissions" yaml:"module_permissions"`
 	}
 
 	GenesisState []GenesisAccount
@@ -34,20 +34,20 @@ type (
 func NewGenesisAccount(
 	address sdk.AccAddress, coins sdk.Coins, sequence uint64,
 	vestingAmount, delFree, delVesting sdk.Coins, vestingStartTime, vestingEndTime int64,
-	module, permission string,
+	module string, permissions []string,
 ) GenesisAccount {
 
 	return GenesisAccount{
-		Address:          address,
-		Coins:            coins,
-		Sequence:         sequence,
-		AccountNumber:    0, // ignored set by the account keeper during InitGenesis
-		OriginalVesting:  vestingAmount,
-		DelegatedFree:    delFree,
-		DelegatedVesting: delVesting,
-		StartTime:        vestingStartTime,
-		EndTime:          vestingEndTime,
-		ModuleName:       module,
-		ModulePermission: permission,
+		Address:           address,
+		Coins:             coins,
+		Sequence:          sequence,
+		AccountNumber:     0, // ignored set by the account keeper during InitGenesis
+		OriginalVesting:   vestingAmount,
+		DelegatedFree:     delFree,
+		DelegatedVesting:  delVesting,
+		StartTime:         vestingStartTime,
+		EndTime:           vestingEndTime,
+		ModuleName:        module,
+		ModulePermissions: permissions,
 	}
 }

x/gov/test_common.go

@@ -59,8 +59,12 @@ func getMockApp(t *testing.T, numGenAccs int, genState GenesisState, genAccs []a
 
 	bk := bank.NewBaseKeeper(mApp.AccountKeeper, mApp.ParamsKeeper.Subspace(bank.DefaultParamspace), bank.DefaultCodespace)
 
-	supplyKeeper := supply.NewKeeper(mApp.Cdc, keySupply, mApp.AccountKeeper, bk, supply.DefaultCodespace,
-		[]string{}, []string{}, []string{types.ModuleName, staking.NotBondedPoolName, staking.BondedPoolName})
+	maccPerms := map[string][]string{
+		types.ModuleName:          []string{supply.Burner},
+		staking.NotBondedPoolName: []string{supply.Burner, supply.Staking},
+		staking.BondedPoolName:    []string{supply.Burner, supply.Staking},
+	}
+	supplyKeeper := supply.NewKeeper(mApp.Cdc, keySupply, mApp.AccountKeeper, bk, supply.DefaultCodespace, maccPerms)
 	sk := staking.NewKeeper(mApp.Cdc, keyStaking, tKeyStaking, supplyKeeper, pk.Subspace(staking.DefaultParamspace), staking.DefaultCodespace)
 
 	keeper := NewKeeper(mApp.Cdc, keyGov, pk, pk.Subspace("testgov"), supplyKeeper, sk, DefaultCodespace, rtr)
@@ -108,8 +112,8 @@ func getInitChainer(mapp *mock.App, keeper Keeper, stakingKeeper staking.Keeper,
 
 		// set module accounts
 		govAcc := supply.NewEmptyModuleAccount(types.ModuleName, supply.Burner)
-		notBondedPool := supply.NewEmptyModuleAccount(staking.NotBondedPoolName, supply.Burner)
-		bondPool := supply.NewEmptyModuleAccount(staking.BondedPoolName, supply.Burner)
+		notBondedPool := supply.NewEmptyModuleAccount(staking.NotBondedPoolName, supply.Burner, supply.Staking)
+		bondPool := supply.NewEmptyModuleAccount(staking.BondedPoolName, supply.Burner, supply.Staking)
 
 		supplyKeeper.SetModuleAccount(ctx, govAcc)
 		supplyKeeper.SetModuleAccount(ctx, notBondedPool)

x/mint/internal/keeper/test_common.go

@@ -57,8 +57,13 @@ func newTestInput(t *testing.T) testInput {
 	paramsKeeper := params.NewKeeper(types.ModuleCdc, keyParams, tkeyParams, params.DefaultCodespace)
 	accountKeeper := auth.NewAccountKeeper(types.ModuleCdc, keyAcc, paramsKeeper.Subspace(auth.DefaultParamspace), auth.ProtoBaseAccount)
 	bankKeeper := bank.NewBaseKeeper(accountKeeper, paramsKeeper.Subspace(bank.DefaultParamspace), bank.DefaultCodespace)
-	supplyKeeper := supply.NewKeeper(types.ModuleCdc, keySupply, accountKeeper, bankKeeper, supply.DefaultCodespace,
-		[]string{auth.FeeCollectorName}, []string{types.ModuleName}, []string{staking.NotBondedPoolName, staking.BondedPoolName})
+	maccPerms := map[string][]string{
+		auth.FeeCollectorName:     []string{supply.Basic},
+		types.ModuleName:          []string{supply.Minter},
+		staking.NotBondedPoolName: []string{supply.Burner, supply.Staking},
+		staking.BondedPoolName:    []string{supply.Burner, supply.Staking},
+	}
+	supplyKeeper := supply.NewKeeper(types.ModuleCdc, keySupply, accountKeeper, bankKeeper, supply.DefaultCodespace, maccPerms)
 	supplyKeeper.SetSupply(ctx, supply.NewSupply(sdk.Coins{}))
 
 	stakingKeeper := staking.NewKeeper(