diff --git a/.github/workflows/ci.yaml b/.github/workflows/ci.yaml index 1b0d26c4..9aab298c 100644 --- a/.github/workflows/ci.yaml +++ b/.github/workflows/ci.yaml @@ -45,7 +45,7 @@ jobs: env: CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }} run: | - ./test.sh + ./.github/workflows/test.sh docker: runs-on: ubuntu-18.04 diff --git a/test.sh b/.github/workflows/test.sh similarity index 66% rename from test.sh rename to .github/workflows/test.sh index c4f6ef85..e17aa713 100755 --- a/test.sh +++ b/.github/workflows/test.sh @@ -2,8 +2,8 @@ # manually exiting from script, because after-build needs to run always set +e -if [ -z $CC_TEST_REPORT_ID ]; then - echo "1. CC_TEST_REPORT_ID is unset, skipping" +if [ -z $CC_TEST_REPORTER_ID ]; then + echo "1. CC_TEST_REPORTER_ID is unset, skipping" else echo "1. Running before-build" ./cc-test-reporter before-build @@ -12,13 +12,12 @@ fi echo "2. Running test" make test TEST_STATUS=$? -echo "TEST_STATUS: ${TEST_STATUS}" -if [ -z $CC_TEST_REPORT_ID ]; then - echo "3. CC_TEST_REPORT_ID is unset, skipping" +if [ -z $CC_TEST_REPORTER_ID ]; then + echo "3. CC_TEST_REPORTER_ID is unset, skipping" else echo "3. Running after-build" - ./cc-test-reporter after-build --exit-code $TEST_STATUS -t gocov + ./cc-test-reporter after-build --exit-code $TEST_STATUS -t gocov --prefix $(basename $(go list -m)) fi if [ "$TEST_STATUS" -ne 0 ]; then diff --git a/CHANGELOG.md b/CHANGELOG.md index 62a9c5ac..eaca5c86 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -25,6 +25,7 @@ ## Changes since v6.1.1 +- [#825](https://github.com/oauth2-proxy/oauth2-proxy/pull/825) Fix code coverage reporting on GitHub actions(@JoelSpeed) - [#705](https://github.com/oauth2-proxy/oauth2-proxy/pull/705) Add generic Header injectors for upstream request and response headers (@JoelSpeed) - [#753](https://github.com/oauth2-proxy/oauth2-proxy/pull/753) Pass resource parameter in login url (@codablock) - [#789](https://github.com/oauth2-proxy/oauth2-proxy/pull/789) Add `--skip-auth-route` configuration option for `METHOD=pathRegex` based allowlists (@NickMeves)