Skip to content

Add unit test workflow for pull requests#343

Merged
Jeomon merged 1 commit into
CursorTouch:mainfrom
JezaChen:ci/add-unit-test-workflow
Jul 20, 2026
Merged

Add unit test workflow for pull requests#343
Jeomon merged 1 commit into
CursorTouch:mainfrom
JezaChen:ci/add-unit-test-workflow

Conversation

@JezaChen

Copy link
Copy Markdown
Collaborator

PR #341 fixed problematic test cases introduced by PR #300. To prevent future PRs from introducing invalid or unreliable AI-generated tests, unit tests should run automatically before changes are merged.

  • Run unit tests on pushes to main
  • Run unit tests on PRs targeting main

Copilot AI review requested due to automatic review settings July 20, 2026 14:17
@qodo-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds a GitHub Actions workflow to automatically run the repository’s Python unit tests on Windows for both pushes to main and pull requests targeting main, helping prevent broken/unreliable tests from being merged.

Changes:

  • Introduces a new Tests workflow triggered on push to main and pull_request targeting main.
  • Sets up Python 3.13 + uv, installs dev dependencies, and runs pytest on windows-latest.

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

Comment on lines +23 to +28
- name: Install uv and Python
uses: astral-sh/setup-uv@v5
with:
enable-cache: true
python-version: "3.13"

Comment on lines +32 to +33
- name: Run unit tests
run: uv run --frozen --extra dev pytest
@Jeomon
Jeomon merged commit aeb94c2 into CursorTouch:main Jul 20, 2026
1 check passed
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.

3 participants