Skip to content

Conversation

@hepheir
Copy link
Member

@hepheir hepheir commented Dec 5, 2025

Pytest를 사용하여 VSCode 에디터에서 인에디터 단위테스트 GUI 를 쉽게 이용할 수 있게 하였습니다.

@hepheir hepheir self-assigned this Dec 5, 2025
@hepheir hepheir changed the base branch from main to develop December 5, 2025 06:45
@hepheir hepheir force-pushed the feature/unittests branch 2 times, most recently from 4b1f51d to ca7ca06 Compare December 5, 2025 06:50
@hepheir hepheir requested a review from Copilot December 5, 2025 06:52
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR configures pytest-based unit testing environment for VSCode integration and adds a GitHub Actions CI workflow for automated testing. The setup enables developers to use VSCode's in-editor test GUI with pytest while also establishing CI/CD for the Django project.

Key Changes:

  • Adds pytest configuration with Django settings module
  • Configures VSCode Python extension for pytest integration
  • Sets up GitHub Actions workflow for CI testing on the develop branch

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
pytest.ini Configures pytest with Django settings module (app.settings)
.vscode/settings.json Enables VSCode pytest integration with proper Python paths and testing configuration
.github/workflows/django.yml Adds CI workflow for Python 3.9-3.13 with automated testing on push/PR to develop branch

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +28 to +31
- name: Run Tests
run: |
cd app
python manage.py test
Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The CI workflow uses Django's built-in test runner (python manage.py test), but this PR configures pytest for local development. This inconsistency means that local tests with pytest might pass while CI tests fail, or vice versa. Consider using pytest in CI as well by changing the test command to pytest and ensuring pytest and pytest-django are installed in the dependencies.

Suggested change
- name: Run Tests
run: |
cd app
python manage.py test
pip install pytest pytest-django
- name: Run Tests
run: |
cd app
pytest

Copilot uses AI. Check for mistakes.
steps:
- uses: actions/checkout@v4
- name: Set up Python ${{ matrix.python-version }}
uses: actions/setup-python@v3
Copy link

Copilot AI Dec 5, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The actions/setup-python@v3 is an older version. Consider updating to actions/setup-python@v5 which is the latest stable version and provides better support for newer Python versions like 3.13, improved caching, and bug fixes.

Suggested change
uses: actions/setup-python@v3
uses: actions/setup-python@v5

Copilot uses AI. Check for mistakes.
@hepheir hepheir merged commit 2a92c83 into develop Dec 5, 2025
5 checks passed
@hepheir hepheir deleted the feature/unittests branch December 5, 2025 07:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants