Skip to content

Commit

Permalink
Remove bash unit tests
Browse files Browse the repository at this point in the history
  • Loading branch information
Malax committed Jan 23, 2025
1 parent b78355d commit 94b2c4f
Show file tree
Hide file tree
Showing 6 changed files with 0 additions and 841 deletions.
41 changes: 0 additions & 41 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -63,46 +63,6 @@ jobs:
- name: Run Hatchet integration tests
run: bundle exec parallel_split_test test/spec/

buildpack-testrunner:
runs-on: ubuntu-22.04
needs: lint
env:
SHUNIT_HOME: /tmp/shunit2-2.1.6
# Note the missing STACK environment variable here. This works since there is a default value in the buildpack
# source. I ported this as-is from the Travis config. Given we're trying to get rid of testrunner entirely,
# it will stay like this. If we, for some reason, decide to keep testrunner, we should look into a fixed STACK env var.
steps:
- uses: actions/checkout@v4
- uses: actions/setup-java@v4
with:
distribution: zulu
java-version: 8
- name: Download and unpack shunit 2.1.6
run: curl -sSf --retry 3 --retry-connrefused --connect-timeout 5 https://storage.googleapis.com/google-code-archive-downloads/v2/code.google.com/shunit2/shunit2-2.1.6.tgz | tar xz -C /tmp/
- name: Clone heroku-buildpack-testrunner
run: git clone https://github.com/heroku/heroku-buildpack-testrunner.git /tmp/testrunner
- name: Apply heroku-buildpack-testrunner patches to enforce bash shell
run: |
cd /tmp/testrunner
git apply <<'EOF'
diff --git a/bin/run b/bin/run
index 0d5b790..a0ff25c 100755
--- a/bin/run
+++ b/bin/run
@@ -101,7 +101,7 @@ for bp in ${@}; do
suite_start_time="$(date +%s)"
echo " TEST SUITE: $(basename ${f})"
- ${SHUNIT_HOME?"'SHUNIT_HOME' environment variable must be set"}/src/shunit2 ${f} | indent
+ /bin/bash ${SHUNIT_HOME?"'SHUNIT_HOME' environment variable must be set"}/src/shunit2 ${f} | indent
exit_code=$(max ${exit_code} ${PIPESTATUS[0]})
suite_end_time="$(date +%s)"
EOF
- name: Execute buildpack-testrunner
run: /tmp/testrunner/bin/run .

unit-tests:
name: "Unit Tests (${{ matrix.stack }})"
runs-on: ubuntu-22.04
Expand All @@ -117,7 +77,6 @@ jobs:
stack: ["heroku-20", "heroku-22", "heroku-24"]
steps:
- uses: actions/checkout@v4
- run: test/v2
- run: test/jdbc.sh

container-test:
Expand Down
79 changes: 0 additions & 79 deletions test/compile_test.sh

This file was deleted.

205 changes: 0 additions & 205 deletions test/unit

This file was deleted.

Loading

0 comments on commit 94b2c4f

Please sign in to comment.