git.iko.soy/typeable/oauth2-proxy
on master help sign in
Merge pull request #825 from oauth2-proxy/test-reporter
f4c292bNick Meves
.github/workflows/ci.yaml+1−1
@@ -45,7 +45,7 @@ jobs:
45 env:
45 env:
46 CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
46 CC_TEST_REPORTER_ID: ${{ secrets.CC_TEST_REPORTER_ID }}
47 run: |
47 run: |
48− ./test.sh
48+ ./.github/workflows/test.sh
49
49
50 docker:
50 docker:
51 runs-on: ubuntu-18.04
51 runs-on: ubuntu-18.04
test.sh → .github/workflows/test.shrenamed+5−6
@@ -2,8 +2,8 @@
2# manually exiting from script, because after-build needs to run always
2# manually exiting from script, because after-build needs to run always
3set +e
3set +e
4
4
5−if [ -z $CC_TEST_REPORT_ID ]; then
5+if [ -z $CC_TEST_REPORTER_ID ]; then
6− echo "1. CC_TEST_REPORT_ID is unset, skipping"
6+ echo "1. CC_TEST_REPORTER_ID is unset, skipping"
7else
7else
8 echo "1. Running before-build"
8 echo "1. Running before-build"
9 ./cc-test-reporter before-build
9 ./cc-test-reporter before-build
@@ -12,13 +12,12 @@ fi
12echo "2. Running test"
12echo "2. Running test"
13make test
13make test
14TEST_STATUS=$?
14TEST_STATUS=$?
15−echo "TEST_STATUS: ${TEST_STATUS}"
16
15
17−if [ -z $CC_TEST_REPORT_ID ]; then
16+if [ -z $CC_TEST_REPORTER_ID ]; then
18− echo "3. CC_TEST_REPORT_ID is unset, skipping"
17+ echo "3. CC_TEST_REPORTER_ID is unset, skipping"
19else
18else
20 echo "3. Running after-build"
19 echo "3. Running after-build"
21− ./cc-test-reporter after-build --exit-code $TEST_STATUS -t gocov
20+ ./cc-test-reporter after-build --exit-code $TEST_STATUS -t gocov --prefix $(basename $(go list -m))
22fi
21fi
23
22
24if [ "$TEST_STATUS" -ne 0 ]; then
23if [ "$TEST_STATUS" -ne 0 ]; then
CHANGELOG.md+1
@@ -25,6 +25,7 @@
25
25
26## Changes since v6.1.1
26## Changes since v6.1.1
27
27
28+- [#825](https://github.com/oauth2-proxy/oauth2-proxy/pull/825) Fix code coverage reporting on GitHub actions(@JoelSpeed)
28- [#705](https://github.com/oauth2-proxy/oauth2-proxy/pull/705) Add generic Header injectors for upstream request and response headers (@JoelSpeed)
29- [#705](https://github.com/oauth2-proxy/oauth2-proxy/pull/705) Add generic Header injectors for upstream request and response headers (@JoelSpeed)
29- [#753](https://github.com/oauth2-proxy/oauth2-proxy/pull/753) Pass resource parameter in login url (@codablock)
30- [#753](https://github.com/oauth2-proxy/oauth2-proxy/pull/753) Pass resource parameter in login url (@codablock)
30- [#789](https://github.com/oauth2-proxy/oauth2-proxy/pull/789) Add `--skip-auth-route` configuration option for `METHOD=pathRegex` based allowlists (@NickMeves)
31- [#789](https://github.com/oauth2-proxy/oauth2-proxy/pull/789) Add `--skip-auth-route` configuration option for `METHOD=pathRegex` based allowlists (@NickMeves)