fix(namada): Update Namada example test with Namada v1.0.0 (#1316)

strangelove-ventures · Dec 10, 2024 · 4bd4690 · 4bd4690
1 parent 912e4e1
commit 4bd4690
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 11 deletions.
diff --git a/chain/namada/namada_chain.go b/chain/namada/namada_chain.go
@@ -477,7 +477,7 @@ func (c *NamadaChain) SendIBCTransfer(ctx context.Context, channelID, keyName st
 		return ibc.Tx{}, fmt.Errorf("the transaction failed: %s", outputStr)
 	}
 
-	re = regexp.MustCompile(`Transaction ([0-9A-F]+) was successfully applied at height (\d+), consuming (\d+) gas units`)
+	re = regexp.MustCompile(`Transaction batch ([0-9A-F]+) was applied at height (\d+), consuming (\d+) gas units`)
 	matchesAll := re.FindAllStringSubmatch(outputStr, -1)
 	if len(matches) == 0 {
 		return ibc.Tx{}, fmt.Errorf("the transaction failed: %s", outputStr)

diff --git a/chain/namada/namada_node.go b/chain/namada/namada_node.go
@@ -183,7 +183,8 @@ func (n *NamadaNode) CreateContainer(ctx context.Context) error {
 
 	mvCmd := "echo 'starting a validator node'"
 	if !n.Validator {
-		mvCmd = fmt.Sprintf(`mv %s/wallet.toml %s/%s`, n.HomeDir(), n.HomeDir(), n.Chain.Config().ChainID)
+		baseDir := fmt.Sprintf("%s/%s", n.HomeDir(), n.Chain.Config().ChainID)
+		mvCmd = fmt.Sprintf(`mv %s/wallet.toml %s && sed -i 's/127.0.0.1:26657/0.0.0.0:26657/g' %s/config.toml`, n.HomeDir(), baseDir, baseDir)
 	}
 
 	ledgerCmd := fmt.Sprintf(`%s node --base-dir %s ledger`, n.Chain.Config().Bin, n.HomeDir())

diff --git a/examples/namada/namada_chain_test.go b/examples/namada/namada_chain_test.go
@@ -28,8 +28,8 @@ func TestNamadaNetwork(t *testing.T) {
 	t.Parallel()
 	client, network := interchaintest.DockerSetup(t)
 
-	nv := 1
-	fn := 0
+	nv := 2
+	fn := 1
 
 	coinDecimals := namadachain.NamTokenDenom
 	chains, err := interchaintest.NewBuiltinChainFactory(zaptest.NewLogger(t), []*interchaintest.ChainSpec{
@@ -38,11 +38,12 @@ func TestNamadaNetwork(t *testing.T) {
 		}},
 		{
 			Name:    "namada",
-			Version: "v0.44.1",
+			Version: "v1.0.0",
 			ChainConfig: ibc.ChainConfig{
 				ChainID:      "namada-test",
 				Denom:        namadachain.NamAddress,
-				Gas:          "250000",
+				Gas:          "50000",
+				GasPrices:    "0.00001",
 				CoinDecimals: &coinDecimals,
 				Bin:          "namada",
 			},
@@ -58,8 +59,8 @@ func TestNamadaNetwork(t *testing.T) {
 	// Relayer Factory
 	r := interchaintest.NewBuiltinRelayerFactory(ibc.Hermes, zaptest.NewLogger(t),
 		relayer.CustomDockerImage(
-			"ghcr.io/heliaxdev/hermes",
-			"v1.10.4-namada-beta17-rc2@sha256:a95ede57f63ebb4c70aa4ca0bfb7871a5d43cd76d17b1ad62f5d31a9465d65af",
+			"ghcr.io/informalsystems/hermes",
+			"luca_joss-test-namada-docker-action@sha256:3bae7e68fe76647d5dba19982d01d4c33e2eebef17e3506da17129f4e9483b11",
 			"2000:2000",
 		)).
 		Build(t, client, network)
@@ -103,9 +104,12 @@ func TestNamadaNetwork(t *testing.T) {
 
 	initBalance := math.NewInt(1_000_000)
 
-	gasSpent, _ := strconv.ParseInt(namada.Config().Gas, 10, 64)
-	namadaGasSpent := math.NewInt(gasSpent)
-	tokenDenom := math.NewInt(int64(stdmath.Pow(10, float64(*namada.Config().CoinDecimals))))
+	gasSpent, _ := strconv.ParseFloat(namada.Config().Gas, 64)
+	gasPrice, _ := strconv.ParseFloat(namada.Config().GasPrices, 64)
+	tokenDenomVal := stdmath.Pow(10, float64(*namada.Config().CoinDecimals))
+	// GasSpent should be an integer
+	namadaGasSpent := math.NewInt(int64(gasSpent * gasPrice * tokenDenomVal))
+	tokenDenom := math.NewInt(int64(tokenDenomVal))
 	namadaInitBalance := initBalance.Mul(tokenDenom)
 
 	users := interchaintest.GetAndFundTestUsers(t, ctx, "user", initBalance, chain, namada)