remove redundancy hermes install for integration tests

ignite · Aug 6, 2024 · 29cf852 · 29cf852
1 parent cee5d4a
commit 29cf852
Showing 1 changed file with 14 additions and 12 deletions.
diff --git a/hermes/integration/app_test.go b/hermes/integration/app_test.go
@@ -285,12 +285,13 @@ func TestCustomIBCTx(t *testing.T) {
 	require.NoError(t, err)
 	pluginPath := filepath.Join(filepath.Dir(filepath.Dir(dir)), "hermes")
 
-	env.Must(env.Exec("install hermes app locally",
-		step.NewSteps(step.New(
-			step.Exec(envtest.IgniteApp, "app", "install", pluginPath),
-			step.Workdir(app.SourcePath()),
-		)),
-	))
+	// We don't need to install Hermes again since it's a builtin extension.
+	// env.Must(env.Exec("install hermes app locally",
+	// 	step.NewSteps(step.New(
+	// 		step.Exec(envtest.IgniteApp, "app", "install", pluginPath),
+	// 		step.Workdir(app.SourcePath()),
+	// 	)),
+	// ))
 
 	// One local plugin expected
 	assertLocalPlugins(t, app, []pluginsconfig.Plugin{{Path: pluginPath}})
@@ -630,12 +631,13 @@ func TestTransferIBCTx(t *testing.T) {
 	require.NoError(t, err)
 	pluginPath := filepath.Join(filepath.Dir(filepath.Dir(dir)), "hermes")
 
-	env.Must(env.Exec("install hermes app locally",
-		step.NewSteps(step.New(
-			step.Exec(envtest.IgniteApp, "app", "install", pluginPath),
-			step.Workdir(app.SourcePath()),
-		)),
-	))
+	// We don't need to install Hermes again since it's a builtin extension.
+	// env.Must(env.Exec("install hermes app locally",
+	// 	step.NewSteps(step.New(
+	// 		step.Exec(envtest.IgniteApp, "app", "install", pluginPath),
+	// 		step.Workdir(app.SourcePath()),
+	// 	)),
+	// ))
 
 	// One local plugin expected
 	assertLocalPlugins(t, app, []pluginsconfig.Plugin{{Path: pluginPath}})