Skip to content

Conversation

@edenhaus
Copy link
Member

No description provided.

Copilot AI review requested due to automatic review settings September 17, 2025 20:32
Copy link
Contributor

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 introduces pytest-xdist to improve test execution performance through parallel test running. The main purpose is to enable concurrent test execution using the -n auto flag, which automatically detects the optimal number of worker processes.

  • Added pytest-xdist dependency to enable parallel test execution
  • Modified test commands across CI workflows to use -n auto flag for automatic worker count detection
  • Sorted parametrized test sets to ensure deterministic test execution order in parallel environments

Reviewed Changes

Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pyproject.toml Adds pytest-xdist==3.8.0 dependency for parallel test execution
ci/entrypoint.sh Updates pytest command to use -n auto for parallel execution
.github/workflows/ci.yml Modifies CI workflows to run tests in parallel using -n auto
tests/messages/xml/test_pos.py Wraps parametrized test sets in sorted() for deterministic order
tests/messages/xml/test_map.py Wraps parametrized test sets in sorted() for deterministic order
tests/messages/xml/test_clean.py Wraps parametrized test sets in sorted() for deterministic order

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@edenhaus edenhaus added the pr: refactor PR with code refactoring label Sep 17, 2025
@codecov
Copy link

codecov bot commented Sep 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.86%. Comparing base (9171330) to head (78e0128).
⚠️ Report is 3 commits behind head on dev.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##              dev    #1171   +/-   ##
=======================================
  Coverage   93.86%   93.86%           
=======================================
  Files         135      135           
  Lines        5149     5149           
  Branches      336      336           
=======================================
  Hits         4833     4833           
  Misses        252      252           
  Partials       64       64           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@codspeed-hq
Copy link

codspeed-hq bot commented Sep 17, 2025

CodSpeed Performance Report

Merging #1171 will not alter performance

Comparing pytest-xdist (78e0128) with dev (685a7ef)

Summary

✅ 6 untouched

@edenhaus edenhaus merged commit a2d3211 into dev Sep 17, 2025
27 checks passed
@edenhaus edenhaus deleted the pytest-xdist branch September 17, 2025 20:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr: refactor PR with code refactoring

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants