Skip to content

FIX: Add all PyAEDT Extension tests#1949

Merged
svandenb-dev merged 2 commits intomainfrom
fix/pyaedt_extension_issue
Mar 23, 2026
Merged

FIX: Add all PyAEDT Extension tests#1949
svandenb-dev merged 2 commits intomainfrom
fix/pyaedt_extension_issue

Conversation

@Samuelopez-ansys
Copy link
Member

This pull request updates the continuous integration workflow to expand test coverage for the external/pyaedt extensions. The main change is the addition of several new system-level tests to the CI pipeline, which will help catch issues earlier and improve overall code quality.

Testing improvements:

  • Added the following system tests to the ci-release.yml workflow: test_export_to_3d.py, test_export_layout.py, test_post_layout_design.py, and test_via_clustering.py in the external/pyaedt/tests/system/extensions/ directory.

@github-actions github-actions bot added maintenance Package and maintenance related ci/cd labels Mar 23, 2026
@codecov-commenter
Copy link

codecov-commenter commented Mar 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.84%. Comparing base (5cbae21) to head (2dfeb6c).
⚠️ Report is 43 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1949      +/-   ##
==========================================
+ Coverage   70.20%   70.84%   +0.63%     
==========================================
  Files         261      263       +2     
  Lines       42416    42876     +460     
==========================================
+ Hits        29779    30376     +597     
+ Misses      12637    12500     -137     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@SMoraisAnsys SMoraisAnsys left a comment

Choose a reason for hiding this comment

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

LGTM, please @hui-zhou-a and @svandenb-dev could you ensure that you cover the pyaedt tests involving pyedb when releasing ? It's not the first time that a new release of pyedb is breaking pyaedt's CI :/

Copy link
Collaborator

@svandenb-dev svandenb-dev left a comment

Choose a reason for hiding this comment

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

LGTM

@svandenb-dev svandenb-dev merged commit 8d2bce4 into main Mar 23, 2026
25 checks passed
@svandenb-dev svandenb-dev deleted the fix/pyaedt_extension_issue branch March 23, 2026 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/cd maintenance Package and maintenance related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants