Skip to content

[SYSTEMDS-3901] Updated Python CI Test setup#2300

Closed
e-strauss wants to merge 3 commits into
apache:mainfrom
e-strauss:split_python_tests
Closed

[SYSTEMDS-3901] Updated Python CI Test setup#2300
e-strauss wants to merge 3 commits into
apache:mainfrom
e-strauss:split_python_tests

Conversation

@e-strauss

@e-strauss e-strauss commented Jul 30, 2025

Copy link
Copy Markdown
Contributor

Changes:

  • split down of python test runs: w/ env, w/o env, federated
  • separate python formatting test

These changes reduce the runtime of the single tests, and also make the Python test without environment to actually run without the SYSTEMDS_HOME installation.

@e-strauss e-strauss requested a review from Baunsgaard July 30, 2025 21:51
@codecov

codecov Bot commented Jul 30, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.60%. Comparing base (480e9a0) to head (53f71e9).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2300      +/-   ##
============================================
- Coverage     72.61%   72.60%   -0.02%     
+ Complexity    46267    46257      -10     
============================================
  Files          1490     1490              
  Lines        174390   174390              
  Branches      34210    34210              
============================================
- Hits         126634   126614      -20     
- Misses        38194    38211      +17     
- Partials       9562     9565       +3     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Baunsgaard Baunsgaard left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@e-strauss e-strauss closed this in 017af0b Jul 31, 2025
@github-project-automation github-project-automation Bot moved this from In Progress to Done in SystemDS PR Queue Jul 31, 2025
@e-strauss e-strauss deleted the split_python_tests branch July 31, 2025 17:31
j143 pushed a commit to j143/systemds that referenced this pull request Aug 9, 2025
- split down of python test runs: w/ env, w/o env, federated
- separate python formatting test

These changes reduce the runtime of the single tests, and also make the Python test without environment to actually run without the SYSTEMDS_HOME installation.

Closes apache#2300.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants