add text coverage to PR comment #641
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
name: Run tests | |
on: [pull_request] | |
jobs: | |
test: | |
runs-on: ubuntu-latest | |
strategy: | |
matrix: | |
postgres_version: ['14', '16'] | |
services: | |
postgres: | |
image: postgres:${{ matrix.postgres_version }} | |
env: | |
POSTGRES_DB: ndoh_hub | |
POSTGRES_USER: postgres | |
POSTGRES_PASSWORD: postgres | |
ports: | |
- 5432:5432 | |
redis: | |
image: redis:6.0 | |
ports: | |
- 6379:6379 | |
env: | |
HUB_DATABASE: postgres://postgres:postgres@localhost:5432/ndoh_hub | |
steps: | |
- uses: actions/checkout@v2 | |
- name: Install gettext | |
run: sudo apt-get install gettext | |
- uses: actions/cache@v2 | |
with: | |
path: ~/.cache/pip | |
key: ${{ hashFiles('requirements.txt', 'requirements-dev.txt') }}-pip | |
- uses: actions/setup-python@v2 | |
with: | |
python-version: 3.9 | |
- name: Install dependancies | |
run: | | |
python -m pip install --upgrade pip | |
pip install -r requirements.txt -r requirements-dev.txt | |
django-admin compilemessages | |
- name: Lint | |
run: | | |
flake8 | |
python manage.py makemigrations registrations changes eventstore \ | |
--dry-run | grep 'No changes detected' || \ | |
(echo 'There are changes which require migrations.' && exit 1) | |
black --check . | |
isort -c -rc . | |
- name: Run tests and generate coverage report | |
run: | | |
py.test --cov --cov-report=term-missing | tee coverage.txt | |
- name: Extract coverage percentage | |
id: extract_coverage | |
run: | | |
COVERAGE=$(grep -oP 'TOTAL\s+\d+\s+\d+\s+\d+\s+\d+\s+\K\d+%' coverage.txt) | |
echo "COVERAGE=$COVERAGE" >> $GITHUB_ENV | |
- name: Get changed lines in the PR | |
id: get_diff | |
run: | | |
git fetch origin develop | |
git diff origin/develop --unified=0 > diff.txt | |
current_file="" | |
echo "Changes in files:" > changed_lines.txt | |
while IFS= read -r line; do | |
# Check if the line indicates a new file (diff line) | |
if [[ $line == diff* ]]; then | |
# Extract the filename from the diff line | |
current_file=$(echo "$line" | awk '{print $3}' | sed 's|^b/||') | |
fi | |
# Check if the line indicates changes (hunk header) | |
if [[ $line == @@* ]]; then | |
# Extract line numbers from the hunk header | |
line_numbers=$(echo "$line" | grep -oP '\d+(?=\s)' | xargs | tr ' ' ',') | |
echo "File: $current_file, Changed lines: $line_numbers" >> changed_lines.txt | |
fi | |
done < diff.txt | |
changed_lines=$(cat changed_lines.txt) | |
echo "changed_lines=$changed_lines" >> $GITHUB_ENV | |
cat changed_lines.txt | |
- name: Check for untested lines | |
id: check_coverage | |
run: echo $changed_lines | |
# UNTTESTED_LINES="" | |
# while IFS= read -r line; do | |
# line_number=$(echo "$line" | grep -oP '\d+') | |
# coverage_info=$(grep -E "^ *$line_number " changed_lines.txt || true) | |
# if [[ $coverage_info == *"missed"* ]]; then | |
# UNTTESTED_LINES="$UNTTESTED_LINES\nLine $line_number is not covered by tests." | |
# fi | |
# done < changed_lines.txt | |
# # echo -e "UNTTESTED_LINES=$UNTTESTED_LINES" >> $GITHUB_ENV | |
# # echo "Untested lines: $UNTTESTED_LINES" | |
# - name: Post coverage comment on PR | |
# uses: actions/github-script@v6 | |
# with: | |
# github-token: ${{ secrets.GITHUB_TOKEN }} | |
# script: | | |
# const coverage = process.env.COVERAGE; | |
# const prNumber = context.payload.pull_request.number; | |
# const untestedLines = process.env.UNTTESTED_LINES || "None"; | |
# const commentBody = `PR Untested Lines: ${untestedLines}\n\n\nOverall Test coverage: ${coverage}`; | |
# await github.rest.issues.createComment({ | |
# issue_number: prNumber, | |
# owner: context.repo.owner, | |
# repo: context.repo.repo, | |
# body: commentBody | |
# }); |