Merge pull request #825 from oauth2-proxy/test-reporter

Fix code coverage reporting on GitHub actions

.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

.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

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)