From 82007ccfa574276e2ae6edca1820129e2df2c491 Mon Sep 17 00:00:00 2001 From: emidev98 Date: Fri, 10 Nov 2023 08:50:33 +0200 Subject: [PATCH 1/2] feat: change allowed length for gov proposals title and descr --- app/keepers/keepers.go | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/app/keepers/keepers.go b/app/keepers/keepers.go index 9c9cef4f..83216088 100644 --- a/app/keepers/keepers.go +++ b/app/keepers/keepers.go @@ -508,7 +508,9 @@ func NewTerraAppKeepers( keepers.BankKeeper, keepers.StakingKeeper, baseApp.MsgServiceRouter(), - govtypes.DefaultConfig(), + govtypes.Config{ + MaxMetadataLen: 5100, // define the length of the governance proposal's title and description + }, authtypes.NewModuleAddress(govtypes.ModuleName).String(), ) From bc0284a1d7c9673b092aa0b137aec17788cf2417 Mon Sep 17 00:00:00 2001 From: emidev98 Date: Fri, 10 Nov 2023 09:02:06 +0200 Subject: [PATCH 2/2] tests: modify tests to reflect changes on gov module config --- integration-tests/src/modules/gov.test.ts | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/integration-tests/src/modules/gov.test.ts b/integration-tests/src/modules/gov.test.ts index 79b2a808..66877cd1 100644 --- a/integration-tests/src/modules/gov.test.ts +++ b/integration-tests/src/modules/gov.test.ts @@ -147,9 +147,9 @@ describe("Governance Module (https://github.com/terra-money/cosmos-sdk/tree/rele [], Coins.fromString("1000000000uluna"), val2WalletAddress, - "metadata", - "title", - "summary" + "METADATA Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue", + "TITLE Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue", + "SUMMARY Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue" ); // Create an alliance proposal sign and submit on chain-2 let tx = await val2Wallet.createAndSignTx({ @@ -205,9 +205,9 @@ describe("Governance Module (https://github.com/terra-money/cosmos-sdk/tree/rele proposal = proposal as Proposal; expect(proposal.id).toBe(proposalId); expect(proposal.proposer).toBe(val2WalletAddress); - expect(proposal.metadata).toBe("metadata"); - expect(proposal.title).toBe("title"); - expect(proposal.summary).toBe("summary"); + expect(proposal.metadata).toBe("METADATA Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue"); + expect(proposal.title).toBe("TITLE Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue"); + expect(proposal.summary).toBe("SUMMARY Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue Lorem ipsum dolor sit amet, consectetur adipiscing elit. Donec nec varius odio. Phasellus tellus felis, varius ut sapien sit amet, imperdiet vehicula metus. Nullam convallis, erat sit amet ultrices ornare, quam metus ornare elit, quis sollicitudin dolor lorem non risus. Pellentesque pretium augue"); expect(proposal.messages.length).toBe(0); expect(proposal.total_deposit).toStrictEqual(Coins.fromString("1000000000uluna")); expect(proposal.submit_time.getTime()).toBeLessThan(Date.now());