Stop caching gotestsum

Author: jannotti

.github/actions/setup-test/action.yml

@@ -28,13 +28,3 @@ runs:
         sudo dpkg-reconfigure -f noninteractive man-db
         sudo apt-get install -y --no-upgrade expect
       shell: bash
-    - name: Cache gotestsum
-      id: cache-gotestsum
-      uses: actions/cache@v4
-      with:
-        path: ~/go/bin/gotestsum
-        key: gotestsum-${{ runner.os }}-${{ runner.arch }}-${{ hashFiles('scripts/buildtools/versions') }}
-    - name: Install gotestsum
-      if: steps.cache-gotestsum.outputs.cache-hit != 'true'
-      run: ./scripts/buildtools/install_buildtools.sh -o "gotest.tools/gotestsum"
-      shell: bash

.github/workflows/ci-nightly.yml

@@ -102,7 +102,6 @@ jobs:
       - name: Run tests
         run: |
           ./scripts/configure_dev.sh
-          ./scripts/buildtools/install_buildtools.sh -o "gotest.tools/gotestsum"
           PACKAGES="$(go list ./... | grep -v /go-algorand/test/)"
           export PACKAGE_NAMES=$(echo $PACKAGES | tr -d '\n')          
           mkdir -p test_results/${{ matrix.platform }}_test_nightly/${PARTITION_ID}
@@ -184,7 +183,6 @@ jobs:
       - name: Run integration tests
         run: |
           ./scripts/configure_dev.sh
-          ./scripts/buildtools/install_buildtools.sh -o "gotest.tools/gotestsum"
           mkdir -p ~/test_results/${{ matrix.platform }}_integration_nightly/${PARTITION_ID}
           TEST_RESULTS=~/test_results/${{ matrix.platform }}_integration_nightly/${PARTITION_ID} \
           test/scripts/run_integration_tests.sh
@@ -241,7 +239,6 @@ jobs:
       - name: Run E2E expect tests
         run: |
           scripts/configure_dev.sh
-          scripts/buildtools/install_buildtools.sh -o "gotest.tools/gotestsum"
           mkdir -p ~/test_results/${{ matrix.platform }}_e2e_expect_nightly/${PARTITION_ID}
           TEST_RESULTS=~/test_results/${{ matrix.platform }}_e2e_expect_nightly/${PARTITION_ID} \
           test/scripts/run_integration_tests.sh
@@ -308,7 +305,6 @@ jobs:
       - name: Run E2E subs tests
         run: |
           scripts/configure_dev.sh
-          scripts/buildtools/install_buildtools.sh -o "gotest.tools/gotestsum"
           mkdir -p ~/test_results/${{ matrix.platform }}_e2e_subs_nightly
           TEST_RESULTS=~/test_results/${{ matrix.platform }}_e2e_subs_nightly \
           test/scripts/run_integration_tests.sh

test/scripts/e2e_go_tests.sh

@@ -7,10 +7,6 @@ set -o pipefail
 
 export GOPATH=$(go env GOPATH)
 
-# Needed for now because circleci doesn't use makefile yet
-TEST_RESULTS=${TEST_RESULTS:="$(pwd)"}
-GOTESTCOMMAND=${GOTESTCOMMAND:="go tool -modfile=${REPO_ROOT}/tool.mod gotestsum --format testname --junitfile ${TEST_RESULTS}/results.xml --jsonfile ${TEST_RESULTS}/testresults.json --"}
-
 # If one or more -t <pattern> are specified, use GOTESTCOMMAND -run <pattern> for each
 
 TESTPATTERNS=()
@@ -48,6 +44,10 @@ fi
 # Anchor our repo root reference location
 REPO_ROOT="$( cd "$(dirname "$0")" ; pwd -P )"/../..
 
+# Needed for now because circleci doesn't use makefile yet
+TEST_RESULTS=${TEST_RESULTS:="$(pwd)"}
+GOTESTCOMMAND=${GOTESTCOMMAND:="go tool -modfile=${REPO_ROOT}/tool.mod gotestsum --format testname --junitfile ${TEST_RESULTS}/results.xml --jsonfile ${TEST_RESULTS}/testresults.json --"}
+
 if [ "${NORACEBUILD}" = "" ]; then
     # Need bin-race binaries for e2e tests
     if [ "${NO_BUILD}" != "true" ]; then