changelog + rabbit

cosmos · Jan 23, 2025 · 50b4550 · 50b4550
1 parent 10d3f21
commit 50b4550
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -45,6 +45,10 @@ Ref: https://keepachangelog.com/en/1.0.0/
 * [180](https://github.com/cosmos/rosetta/pull/180) Update to cosmos-sdk v0.52.0-rc.1.
 * [180](https://github.com/cosmos/rosetta/pull/180) Added bech32 prefix flag.
 
+### Bug Fixes
+
+* [#190](https://github.com/cosmos/rosetta/pull/190) Fixed construction payloads to properly handle transaction signers.
+
 ## [v0.50.11](https://github.com/cosmos/rosetta/releases/tag/v0.50.11) 2024-12-19
 
 ### Improvements

diff --git a/tests/systemtests/construction_test.go b/tests/systemtests/construction_test.go
@@ -103,7 +103,7 @@ func TestPayloads(t *testing.T) {
 
 	op := operation{
 		msgType:  "/cosmos.bank.v1beta1.MsgSend",
-		metadata: fmt.Sprintf(`{"from_address": "%s", "to_address": "%s", "amount":[{"amount":"123", "denom":"stake"}]}`, addr, "cosmos1qcym25uch54xmja4eys35zcyr8npwlm80glwuk"),
+		metadata: fmt.Sprintf(`{"from_address": "%s", "to_address": "%s", "amount":[{"amount":"123", "denom":"stake"}]}`, addr, cli.AddKey("to_address")),
 	}
 	res, err := rosettaRest.constructionPayloads(`"signer_data":[{"account_number":1, "sequence": 0}]`, hexPk, op)
 	assert.NoError(t, err)

diff --git a/tests/systemtests/rest_client.go b/tests/systemtests/rest_client.go
@@ -183,7 +183,7 @@ func (o *operation) String() string {
 func payloadsOperations(ops ...operation) []string {
 	r := make([]string, len(ops))
 	for i, op := range ops {
-		r = append(r, fmt.Sprintf(`{"operation_identifier":{"index": %d}, %s}`, i, op.String()))
+		r[i] = fmt.Sprintf(`{"operation_identifier":{"index": %d}, %s}`, i, op.String())
 	}
 
 	return r