Skip to content

CHORE: Make EMIT tests independent#6546

Merged
Samuelopez-ansys merged 22 commits into
mainfrom
chore/emit_test_independent
Sep 5, 2025
Merged

CHORE: Make EMIT tests independent#6546
Samuelopez-ansys merged 22 commits into
mainfrom
chore/emit_test_independent

Conversation

@Samuelopez-ansys

Copy link
Copy Markdown
Member

Description

Make EMIT tests independent each other
Separate EMIT tests in a new set of tests

Issue linked

Please mention the issue number or describe the problem this pull request addresses.

Checklist

  • I have tested my changes locally.
  • I have added necessary documentation or updated existing documentation.
  • I have followed the coding style guidelines of this project.
  • I have added appropriate tests (unit, integration, system).
  • I have reviewed my changes before submitting this pull request.
  • I have linked the issue or issues that are solved by the PR if any.
  • I have agreed with the Contributor License Agreement (CLA).

@codecov

codecov Bot commented Aug 19, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.01%. Comparing base (6477d82) to head (79a644d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6546      +/-   ##
==========================================
+ Coverage   81.15%   82.01%   +0.86%     
==========================================
  Files         243      243              
  Lines       76281    76278       -3     
==========================================
+ Hits        61906    62560     +654     
+ Misses      14375    13718     -657     
🚀 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.

@github-actions github-actions Bot added maintenance Package and maintenance related testing Anything related to testing labels Aug 20, 2025
jsalant22 and others added 2 commits August 22, 2025 12:50
Make sure Revision class grabs the results folder for the current EmitDesignX
@jsalant22
jsalant22 marked this pull request as ready for review August 22, 2025 17:41
@jsalant22
jsalant22 requested a review from a team as a code owner August 22, 2025 17:41
Comment thread .github/workflows/ci_cd.yml Outdated
Comment thread .github/workflows/ci_cd.yml Outdated
@jsalant22

Copy link
Copy Markdown
Contributor

@ansys/pyaedt-maintainers Can we access logs or anything that might give more details on why the test suite (Testing & coverage EMIT (windows) is timing out? I run the full set of tests locally and they complete in <5 minutes. I've tried with both Python 3.10 and 3.11.

Testing and coverage EMIT (Windows)

Comment thread tests/system/emit/test_emit.py
@Samuelopez-ansys

Copy link
Copy Markdown
Member Author

This pop up window shows up in the last test, and for that reason, the test does not finish:

image

Samuelopez-ansys and others added 4 commits September 3, 2025 10:51
@Samuelopez-ansys
Samuelopez-ansys enabled auto-merge (squash) September 5, 2025 07:29
@Samuelopez-ansys
Samuelopez-ansys merged commit d8c8e1a into main Sep 5, 2025
38 checks passed
@Samuelopez-ansys
Samuelopez-ansys deleted the chore/emit_test_independent branch September 5, 2025 17:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Package and maintenance related testing Anything related to testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants