Skip to content

Commit

Permalink
[pytest] Add warning for PRs missing unit test file changes
Browse files Browse the repository at this point in the history
  • Loading branch information
agl29 committed Nov 11, 2024
1 parent 7b54217 commit 89a67ea
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 0 deletions.
18 changes: 18 additions & 0 deletions .github/workflows/commitflow-py3.yml
Original file line number Diff line number Diff line change
Expand Up @@ -70,6 +70,24 @@ jobs:
name: test-reports-${{ matrix.python-version }}
path: test-reports

- name: Check and comment if no unit test files are modified
if: matrix.python-version == '3.8'
run: |
git fetch origin master
changed_files=$(git diff --name-only origin/master)
if echo "$changed_files" | grep -qE '(^test|_test\.py|^tests|_tests\.py|.test)'; then
echo "✅ Unit test files were modified."
else
echo "⚠️ No unit test files modified."
curl -X POST \
-H "Authorization: token ${{ secrets.GITHUB_TOKEN }}" \
-H "Accept: application/vnd.github.v3+json" \
-d '{"body":"⚠️ No unit test files modified. Please ensure that changes are properly tested. ⚠️"}' \
"https://api.github.com/repos/${{ github.repository }}/issues/${{ github.event.pull_request.number }}/comments"
fi
- name: run python lints
run: |
PYTHONWARNINGS=always ./build/env/bin/hue runruff check
Expand Down
1 change: 1 addition & 0 deletions desktop/core/base_requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,7 @@ protobuf==3.20.3
py==1.11.0
pyformance==0.3.2
pytest==8.1.1
pytest-cov==5.0.0
pytest-django==4.8.0
pytest-html==4.1.1
python-dateutil==2.8.2
Expand Down

0 comments on commit 89a67ea

Please sign in to comment.