Skip to content

REFACTOR: use use small snp for test - #6596

Merged
Samuelopez-ansys merged 5 commits into
mainfrom
refactor-speed-up-test
Sep 2, 2025
Merged

REFACTOR: use use small snp for test#6596
Samuelopez-ansys merged 5 commits into
mainfrom
refactor-speed-up-test

Conversation

@hui-zhou-a

Copy link
Copy Markdown
Contributor

Description

Please provide a brief description of the changes made in this pull request.

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).

@hui-zhou-a
hui-zhou-a requested a review from a team as a code owner September 1, 2025 20:34
@codecov

codecov Bot commented Sep 1, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81.94%. Comparing base (4be184c) to head (7cf1b1d).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6596      +/-   ##
==========================================
+ Coverage   76.41%   81.94%   +5.53%     
==========================================
  Files         242      242              
  Lines       75961    75961              
==========================================
+ Hits        58042    62245    +4203     
+ Misses      17919    13716    -4203     
🚀 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.

@Samuelopez-ansys

Copy link
Copy Markdown
Member

Where are these files used?

@Samuelopez-ansys
Samuelopez-ansys marked this pull request as draft September 2, 2025 05:30
@hui-zhou-a

Copy link
Copy Markdown
Contributor Author

@Samuelopez-ansys

test_01_get_touchstone_data
test_get_coupling_in_range

@Samuelopez-ansys
Samuelopez-ansys marked this pull request as ready for review September 2, 2025 06:54
@Samuelopez-ansys
Samuelopez-ansys enabled auto-merge (squash) September 2, 2025 06:54
@Samuelopez-ansys
Samuelopez-ansys merged commit 24755c9 into main Sep 2, 2025
38 checks passed
@Samuelopez-ansys
Samuelopez-ansys deleted the refactor-speed-up-test branch September 2, 2025 12:34
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