Skip to content

Commit

Permalink
Add Codecov for test coverage reporting (#401)
Browse files Browse the repository at this point in the history
* Added Codecov for unit test coverage reporting

Signed-off-by: saimedhi <[email protected]>

* Added Codecov for unit test coverage reporting

Signed-off-by: saimedhi <[email protected]>

---------

Signed-off-by: saimedhi <[email protected]>
  • Loading branch information
saimedhi authored Sep 30, 2024
1 parent dd20c6f commit 09897ad
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 7 deletions.
7 changes: 6 additions & 1 deletion .github/workflows/build-and-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,12 @@ jobs:
su `id -un 1000` -c "source $NVM_DIR/nvm.sh && nvm use && node -v && yarn -v &&
cd ./plugins/dashboards-flow-framework &&
whoami && yarn osd bootstrap && yarn build && yarn run test:jest --coverage"
- name: Uploads coverage
uses: codecov/codecov-action@v3
with:
verbose: true
env:
CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

# TODO: once github actions supports windows and macos docker containers, we can
# merge these in to the above step's matrix, including adding windows support
Expand Down Expand Up @@ -96,4 +102,3 @@ jobs:
run: |
cd OpenSearch-Dashboards/plugins/dashboards-flow-framework
yarn run test:jest --coverage
10 changes: 4 additions & 6 deletions codecov.yml
Original file line number Diff line number Diff line change
@@ -1,17 +1,15 @@
# disable tracking status entirely until UT is added.
# tracking issue: https://github.com/opensearch-project/dashboards-flow-framework/issues/95
coverage:
# displays different colors depending on below, between, or above the range
range: 50..90
status:
project:
enabled: no
enabled: yes
default:
target: auto
# allows 5% coverage reduction without failing
threshold: 5%
patch: no
changes: no
patch: yes
changes: yes

# disable comments in PRs
comment: no
comment: yes

0 comments on commit 09897ad

Please sign in to comment.