+
Skip to content

Conversation

rafacanton
Copy link
Contributor

@rafacanton rafacanton commented Jul 24, 2025

This test needs to be temporarily disabled to ensure pipeline stability until servers are updated to be able to readd all fluid variables constained in CFF files.

#2468 has the actual changes when the server is up-to-date

@rafacanton rafacanton requested a review from PProfizi July 24, 2025 07:53
@rafacanton rafacanton self-assigned this Jul 24, 2025
@rafacanton rafacanton added the CI/CD Related to CI/CD label Jul 24, 2025
@PProfizi
Copy link
Contributor

PProfizi commented Jul 24, 2025

Hey @rafacanton could we actually instead make the tests better to start with, like was done in #2436 for MAPDL result info?
The idea being that we stop having to do this kind of manipulation.
The problem here is test design.
I can take the ownership of the PR and do it if you want.

@rafacanton
Copy link
Contributor Author

Hey @rafacanton could we actually instead make the tests better to start with, like was done in #2436 for MAPDL result info? The idea being that we stop having to do this kind of manipulation. The problem here is test design. I can take the ownership of the PR and do it if you want.

Yes, see the updated test here: #2468

@PProfizi PProfizi merged commit bf20995 into master Jul 24, 2025
31 of 58 checks passed
@PProfizi PProfizi deleted the rcanton/disable_cff_test branch July 24, 2025 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI/CD Related to CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

点击 这是indexloc提供的php浏览器服务,不要输入任何密码和下载