Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tests: add advanced tests #38

Merged
merged 7 commits into from
Oct 26, 2023
Merged

Tests: add advanced tests #38

merged 7 commits into from
Oct 26, 2023

Conversation

SMoraisAnsys
Copy link
Collaborator

This PR adds the last tests to be extracted and reworked from pyaedt.

@github-actions github-actions bot added the testing Anything related to testing label Oct 25, 2023
@SMoraisAnsys
Copy link
Collaborator Author

There is still an error with test_siwave_add_dc_analysis, I'll look into it asap.

@SMoraisAnsys SMoraisAnsys marked this pull request as draft October 25, 2023 10:47
This test is temporary skipped as it seems to be
not working from pyaedt too.
@SMoraisAnsys SMoraisAnsys marked this pull request as ready for review October 25, 2023 14:53
@SMoraisAnsys
Copy link
Collaborator Author

I did skip the failing test test_siwave_add_dc_analysis as it was also failing in pyaedt.

@svandenb-dev svandenb-dev merged commit 59a0c9c into main Oct 26, 2023
2 of 7 checks passed
@SMoraisAnsys SMoraisAnsys deleted the tests/add_advanced_tests branch January 2, 2024 10:03
svandenb-dev added a commit that referenced this pull request Oct 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
testing Anything related to testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants