Skip to content

Commit

Permalink
Merge pull request #77 from Kava-Labs/yevhenii/integration-tests
Browse files Browse the repository at this point in the history
chore: fix integration tests in CI
  • Loading branch information
evgeniy-scherbina authored Oct 16, 2024
2 parents ad6e768 + 5d32847 commit 367ab06
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 3 deletions.
5 changes: 2 additions & 3 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,9 +37,8 @@ jobs:
- name: Test Unit
run: make test

# TODO(yevhenii): point KAVA_RPC_URL to running kava node and uncomment
# - name: Test Integration
# run: KAVA_RPC_URL=http://50.16.212.18:26658 NETWORK=kava-mainnet PORT=4000 make test-integration
- name: Test Integration
run: KAVA_RPC_URL=https://rpc.data.infra.kava.io:443 NETWORK=kava-mainnet PORT=4000 SKIP_RESOURCE_INTENSIVE_TESTS=true make test-integration

lint:
name: lint
Expand Down
5 changes: 5 additions & 0 deletions testing/account_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,7 @@ import (
"errors"
"fmt"
"math/rand"
"os"
"sort"
"strings"
"testing"
Expand Down Expand Up @@ -61,6 +62,10 @@ func TestAccountBalanceOnlineRetry(t *testing.T) {
t.Skip("skipping account online test")
}

if os.Getenv("SKIP_RESOURCE_INTENSIVE_TESTS") == "true" {
t.Skip("skipping account balance online retry test: it's resource intensive and produces a lot of requests to the node")
}

numJobs := 10
jobCtx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
defer cancel()
Expand Down
4 changes: 4 additions & 0 deletions testing/retry_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,10 @@ func TestBlockRetry(t *testing.T) {
t.Skip("skipping block retry test: it's designed to be run against a live (mainnet) node")
}

if os.Getenv("SKIP_RESOURCE_INTENSIVE_TESTS") == "true" {
t.Skip("skipping block retry test: it's resource intensive and produces a lot of requests to the node")
}

numJobs := 10
jobCtx, cancel := context.WithTimeout(context.Background(), 30*time.Second)
defer cancel()
Expand Down

0 comments on commit 367ab06

Please sign in to comment.