Skip to content

Commit 86cd4e6

Browse files
author
SX
committed
Use set-output instead of GITHUB_ENV for coverage values
1 parent 255220c commit 86cd4e6

File tree

2 files changed

+16
-7
lines changed

2 files changed

+16
-7
lines changed

README.md

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,5 +52,7 @@ Outputs:
5252
* **`steps.mineunit.outputs.badge-color`** Code coverage badge color waiting for issues/#1
5353
* **`steps.mineunit.outputs.mineunit-stdout`** Stdout produced during busted run
5454
* **`steps.mineunit.outputs.mineunit-spec-missing`** Set to true if no spec files found for busted
55-
* **`steps.mineunit.outputs.mineunit-report`** Mineunit detailed test report
55+
* **`steps.mineunit.outputs.mineunit-report`** Mineunit detailed test report
56+
* **`steps.mineunit.outputs.coverage-total`** Total test coverage percentage
57+
* **`steps.mineunit.outputs.coverage-files`** Number of files tested and total number of source files
5658
* **`steps.mineunit.outputs.help-busted-spec-missing`** Help for situation where spec files could not be loaded

action.yml

Lines changed: 13 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -36,7 +36,7 @@ outputs:
3636
value: ${{ steps.badge-wrapper-actions-issue-1.outputs.badge-label }}
3737
badge-status:
3838
description: "Code coverage badge status waiting for issues/#1"
39-
value: ${{ steps.badge-wrapper-actions-issue-1.outputs.badge-status }}
39+
value: "${{ steps.mineunit-coverage.outputs.total }} in ${{ steps.mineunit-coverage.outputs.files }} files"
4040
badge-color:
4141
description: "Code coverage badge color waiting for issues/#1"
4242
value: ${{ steps.badge-wrapper-actions-issue-1.outputs.badge-color }}
@@ -53,6 +53,13 @@ outputs:
5353
description: "Mineunit detailed test report"
5454
value: ${{ steps.mineunit-report.outputs.report }}
5555

56+
coverage-total:
57+
description: "Total test coverage percentage"
58+
value: ${{ steps.mineunit-coverage.outputs.total }}
59+
coverage-files:
60+
description: "Number of files tested and total number of source files"
61+
value: ${{ steps.mineunit-coverage.outputs.files }}
62+
5663
# Help content / on demand documentation
5764
help-busted-spec-missing:
5865
description: "Help for situation where spec files could not be loaded"
@@ -106,18 +113,18 @@ runs:
106113
OUT="$(awk -v p=0 '/^----/{p++;next}p==2{exit}p' luacov.report.out)"
107114
OUT="${OUT//'%'/'%25'}";OUT="${OUT//$'\n'/'%0A'}"
108115
echo "::set-output name=report::${OUT//$'\r'/'%0D'}"
109-
- name: collect coverage data
116+
- id: mineunit-coverage
117+
name: collect coverage data
110118
working-directory: "${{ inputs.working-directory }}"
111119
shell: bash
112120
run: |
113-
(printf coverage=>>$GITHUB_ENV;tail -n 2 luacov.report.out | grep ^Total | grep -o '[0-9.]\+%$'>>$GITHUB_ENV)
114-
(awk -v p=0 '/^----/{p++;next}p==2{exit}p' luacov.report.out | sort -hrk4 > luacov.report.sum)
115-
(echo "files=$(grep -cv '\s0\.00%' luacov.report.sum)/$(wc -l<luacov.report.sum)">>$GITHUB_ENV)
121+
echo "::set-output name=total::$(tail -n 2 luacov.report.out | grep ^Total | grep -o '[0-9.]\+%$')"
122+
awk -v p=0 '/^----/{p++;next}p==2{exit}p' luacov.report.out | sort -hrk4 > luacov.report.sum
123+
echo "::set-output name=files::$(grep -cv '\s0\.00%' luacov.report.sum)/$(wc -l<luacov.report.sum)"
116124
- id: badge-wrapper-actions-issue-1
117125
name: Input wrapper while waiting for actions in actions feature
118126
shell: bash
119127
run: |
120128
echo "::set-output name=badge-name::${{ inputs.badge-name }}"
121129
echo "::set-output name=badge-label::${{ inputs.badge-label }}"
122-
echo "::set-output name=badge-status::${{ env.coverage }} in ${{ env.files }} files"
123130
echo "::set-output name=badge-color::${{ inputs.badge-color }}"

0 commit comments

Comments
 (0)