[Tokenomics] feat: add mint_allocation_source_owner param to tokeno…

…mics module (#915)

## Summary

Add `mint_allocation_supplier` param to the tokenomics module.

## Issue

- `TODO_BETA`

## Type of change

Select one or more from the following:

- [x] New feature, functionality or library
- [ ] Consensus breaking; add the `consensus-breaking` label if so. See
#791 for details
- [ ] Bug fix
- [ ] Code health or cleanup
- [ ] Documentation
- [ ] Other (specify)

## Testing

- [ ] **Documentation**: `make docusaurus_start`; only needed if you
make doc changes
- [x] **Unit Tests**: `make go_develop_and_test`
- [ ] **LocalNet E2E Tests**: `make test_e2e`
- [x] **DevNet E2E Tests**: Add the `devnet-test-e2e` label to the PR.

## Sanity Checklist

- [x] I have tested my changes using the available tooling
- [ ] I have commented my code
- [x] I have performed a self-review of my own code; both comments &
source code
- [ ] I create and reference any new tickets, if applicable
- [ ] I have left TODOs throughout the codebase, if applicable

---------

Co-authored-by: Redouane Lakrache <[email protected]>

config.yml

@@ -282,6 +282,7 @@ genesis:
         mint_allocation_dao: 0.1
         mint_allocation_proposer: 0.05
         mint_allocation_supplier: 0.7
+        mint_allocation_source_owner: 0.15
     shared:
       params:
         num_blocks_per_session: 10

makefiles/params.mk

@@ -26,6 +26,10 @@ params_update_tokenomics_mint_allocation_proposer: ## Update the tokenomics modu
 params_update_tokenomics_mint_allocation_supplier: ## Update the tokenomics module mint_allocation_supplier param
 	poktrolld tx authz exec ./tools/scripts/params/tokenomics_mint_allocation_supplier.json $(PARAM_FLAGS)
 
+.PHONY: params_update_tokenomics_mint_allocation_source_owner
+params_update_tokenomics_mint_allocation_source_owner: ## Update the tokenomics module mint_allocation_source_owner param
+	poktrolld tx authz exec ./tools/scripts/params/tokenomics_mint_allocation_source_owner.json $(PARAM_FLAGS)
+
 ### Service Module Params ###
 .PHONY: params_get_service
 params_get_service: ## Get the service module params

proto/poktroll/tokenomics/params.proto

@@ -21,6 +21,9 @@ message Params {
   // mint_allocation_supplier is the percentage of the minted tokens which are sent
   // to the block supplier account address during claim settlement.
   double mint_allocation_supplier = 3 [(gogoproto.jsontag) = "mint_allocation_supplier", (gogoproto.moretags) = "yaml:\"mint_allocation_supplier\""];
+  // mint_allocation_source_owner is the percentage of the minted tokens which are sent
+  // to the service source owner account address during claim settlement.
+  double mint_allocation_source_owner = 4 [(gogoproto.jsontag) = "mint_allocation_source_owner", (gogoproto.moretags) = "yaml:\"mint_allocation_source_owner\""];
 
   // IMPORTANT: Make sure to update all related files if you're modifying or adding a new parameter.
   // Try the following grep to find all related places: `grep -r compute_units_to_tokens_multiplier`

testutil/integration/suites/param_configs.go

@@ -231,9 +231,10 @@ var (
 			QueryParamsResponse:     tokenomicstypes.QueryParamsResponse{},
 		},
 		ValidParams: tokenomicstypes.Params{
-			MintAllocationDao:      tokenomicstypes.DefaultMintAllocationDao,
-			MintAllocationProposer: tokenomicstypes.DefaultMintAllocationProposer,
-			MintAllocationSupplier: tokenomicstypes.DefaultMintAllocationSupplier,
+			MintAllocationDao:         tokenomicstypes.DefaultMintAllocationDao,
+			MintAllocationProposer:    tokenomicstypes.DefaultMintAllocationProposer,
+			MintAllocationSupplier:    tokenomicstypes.DefaultMintAllocationSupplier,
+			MintAllocationSourceOwner: tokenomicstypes.DefaultMintAllocationSourceOwner,
 		},
 		ParamTypes: map[ParamType]any{
 			ParamTypeFloat64: tokenomicstypes.MsgUpdateParam_AsFloat{},

tools/scripts/params/tokenomics_all.json

@@ -7,7 +7,8 @@
         "params": {
           "mint_allocation_dao": 0.1,
           "mint_allocation_proposer": 0.05,
-          "mint_allocation_supplier": 0.7
+          "mint_allocation_supplier": 0.7,
+          "mint_allocation_source_owner": 0.15
         }
       }
     ]

tools/scripts/params/tokenomics_mint_allocation_source_owner.json

@@ -0,0 +1,12 @@
+{
+  "body": {
+    "messages": [
+      {
+        "@type": "/poktroll.tokenomics.MsgUpdateParam",
+        "authority": "pokt10d07y265gmmuvt4z0w9aw880jnsr700j8yv32t",
+        "name": "mint_allocation_source_owner",
+        "as_float": 0.15
+      }
+    ]
+  }
+}

x/tokenomics/keeper/msg_server_update_param.go

@@ -41,6 +41,9 @@ func (k msgServer) UpdateParam(
 	case tokenomicstypes.ParamMintAllocationSupplier:
 		logger = logger.With("param_value", msg.GetAsFloat())
 		params.MintAllocationSupplier = msg.GetAsFloat()
+	case tokenomicstypes.ParamMintAllocationSourceOwner:
+		logger = logger.With("param_value", msg.GetAsFloat())
+		params.MintAllocationSourceOwner = msg.GetAsFloat()
 	default:
 		return nil, status.Error(
 			codes.InvalidArgument,

x/tokenomics/keeper/msg_server_update_param_test.go

@@ -85,3 +85,28 @@ func TestMsgUpdateParam_UpdateMintAllocationSupplierOnly(t *testing.T) {
 	// Ensure the other parameters are unchanged
 	testkeeper.AssertDefaultParamsEqualExceptFields(t, &defaultParams, res.Params, string(tokenomicstypes.KeyMintAllocationSupplier))
 }
+
+func TestMsgUpdateParam_UpdateMintAllocationSourceOwnerOnly(t *testing.T) {
+	var expectedMintAllocationSourceOwner float64 = 3.14159
+
+	// Set the parameters to their default values
+	k, msgSrv, ctx := setupMsgServer(t)
+	defaultParams := tokenomicstypes.DefaultParams()
+	require.NoError(t, k.SetParams(ctx, defaultParams))
+
+	// Ensure the default values are different from the new values we want to set
+	require.NotEqual(t, expectedMintAllocationSourceOwner, defaultParams.MintAllocationSourceOwner)
+
+	// Update the new parameter
+	updateParamMsg := &tokenomicstypes.MsgUpdateParam{
+		Authority: authtypes.NewModuleAddress(govtypes.ModuleName).String(),
+		Name:      tokenomicstypes.ParamMintAllocationSourceOwner,
+		AsType:    &tokenomicstypes.MsgUpdateParam_AsFloat{AsFloat: expectedMintAllocationSourceOwner},
+	}
+	res, err := msgSrv.UpdateParam(ctx, updateParamMsg)
+	require.NoError(t, err)
+	require.Equal(t, expectedMintAllocationSourceOwner, res.Params.MintAllocationSourceOwner)
+
+	// Ensure the other parameters are unchanged
+	testkeeper.AssertDefaultParamsEqualExceptFields(t, &defaultParams, res.Params, string(tokenomicstypes.KeyMintAllocationSourceOwner))
+}

x/tokenomics/keeper/params_test.go

@@ -109,3 +109,37 @@ func TestParams_ValidateMintAllocationSupplier(t *testing.T) {
 		})
 	}
 }
+
+func TestParams_ValidateMintAllocationSourceOwner(t *testing.T) {
+	tests := []struct {
+		desc                     string
+		mintAllocatioSourceOwner any
+		expectedErr              error
+	}{
+		{
+			desc:                     "invalid type",
+			mintAllocatioSourceOwner: "0",
+			expectedErr:              tokenomicstypes.ErrTokenomicsParamInvalid.Wrap("invalid parameter type: string"),
+		},
+		{
+			desc:                     "invalid MintAllocationSourceOwner (<0)",
+			mintAllocatioSourceOwner: -0.1,
+			expectedErr:              tokenomicstypes.ErrTokenomicsParamInvalid.Wrapf("mint allocation to source owner must be greater than or equal to 0: got %f", -0.1),
+		},
+		{
+			desc:                     "valid MintAllocationSourceOwner",
+			mintAllocatioSourceOwner: tokenomicstypes.DefaultMintAllocationSourceOwner,
+		},
+	}
+
+	for _, test := range tests {
+		t.Run(test.desc, func(t *testing.T) {
+			err := tokenomicstypes.ValidateMintAllocationSourceOwner(test.mintAllocatioSourceOwner)
+			if test.expectedErr != nil {
+				require.ErrorContains(t, err, test.expectedErr.Error())
+			} else {
+				require.NoError(t, err)
+			}
+		})
+	}
+}

x/tokenomics/types/message_update_param.go

@@ -56,6 +56,11 @@ func (msg *MsgUpdateParam) ValidateBasic() error {
 			return err
 		}
 		return ValidateMintAllocationSupplier(msg.GetAsFloat())
+	case ParamMintAllocationSourceOwner:
+		if err := msg.paramTypeIsFloat(); err != nil {
+			return err
+		}
+		return ValidateMintAllocationSourceOwner(msg.GetAsFloat())
 	default:
 		return ErrTokenomicsParamNameInvalid.Wrapf("unsupported param %q", msg.Name)
 	}