feat: add end-to-end testing workflow and comment out skipped tests

TERITORI · Jan 4, 2025 · 19ced45 · 19ced45
1 parent 848c881
commit 19ced45
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 16 deletions.
diff --git a/.github/workflows/integration.yml b/.github/workflows/integration.yml
@@ -34,3 +34,17 @@ jobs:
         run: make integration-tests
         env:
           TERITORI_DAPP_REPO: teritori-dapp
+  test-e2e:
+    runs-on: ubuntu-latest
+    timeout-minutes: 45
+    steps:
+      - uses: actions/setup-go@v5
+        with:
+          go-version: 1.21.x
+      - uses: actions/checkout@v4
+      - name: Build Teritori Docker Image
+        run: make docker.build.e2e
+      - name: Build Hermes Docker Image
+        run: make docker.build.hermes
+      - name: Test E2E
+        run: make test-e2e
diff --git a/tests/e2e/e2e_test.go b/tests/e2e/e2e_test.go
@@ -74,20 +74,20 @@ func (s *IntegrationTestSuite) TestSlashing() {
 }
 
 // todo add fee test with wrong denom order
-func (s *IntegrationTestSuite) TestStakingAndDistribution() {
-	if !runStakingAndDistributionTest {
-		s.T().Skip()
-	}
-	s.testStaking()
-	s.testDistribution()
-}
+// func (s *IntegrationTestSuite) TestStakingAndDistribution() {
+// 	if !runStakingAndDistributionTest {
+// 		s.T().Skip()
+// 	}
+// 	s.testStaking()
+// 	s.testDistribution()
+// }
 
-func (s *IntegrationTestSuite) TestVesting() {
-	if !runVestingTest {
-		s.T().Skip()
-	}
-	chainAAPI := fmt.Sprintf("http://%s", s.valResources[s.chainA.id][0].GetHostPort("1317/tcp"))
-	s.testDelayedVestingAccount(chainAAPI)
-	s.testContinuousVestingAccount(chainAAPI)
-	// s.testPeriodicVestingAccount(chainAAPI) TODO: add back when v0.45 adds the missing CLI command.
-}
+// func (s *IntegrationTestSuite) TestVesting() {
+// 	if !runVestingTest {
+// 		s.T().Skip()
+// 	}
+// 	chainAAPI := fmt.Sprintf("http://%s", s.valResources[s.chainA.id][0].GetHostPort("1317/tcp"))
+// 	s.testDelayedVestingAccount(chainAAPI)
+// 	s.testContinuousVestingAccount(chainAAPI)
+// 	// s.testPeriodicVestingAccount(chainAAPI) TODO: add back when v0.45 adds the missing CLI command.
+// }