Skip to content

Conversation

@rafacanton
Copy link
Contributor

@rafacanton rafacanton commented Jul 24, 2025

Enhance test for CFF result info to add support for internal variables

@rafacanton rafacanton marked this pull request as draft July 24, 2025 07:49
@codecov
Copy link

codecov bot commented Jul 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.18%. Comparing base (bd177cb) to head (2ff9424).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2468      +/-   ##
==========================================
+ Coverage   84.15%   84.18%   +0.02%     
==========================================
  Files          91       91              
  Lines       10683    10683              
==========================================
+ Hits         8990     8993       +3     
+ Misses       1693     1690       -3     

@PProfizi PProfizi changed the base branch from master to main July 28, 2025 09:15
@rafacanton rafacanton changed the title Modify test for new fluid variables Modify test for internal fluid variables Jul 29, 2025
@rafacanton rafacanton marked this pull request as ready for review July 29, 2025 14:13
@rafacanton rafacanton requested a review from PProfizi July 29, 2025 15:50
@rafacanton rafacanton self-assigned this Jul 29, 2025
@rafacanton rafacanton added the tests Related to tests label Jul 29, 2025
@rafacanton rafacanton closed this Jul 30, 2025
@rafacanton rafacanton reopened this Jul 30, 2025
@rafacanton rafacanton merged commit cf099e4 into main Jul 30, 2025
92 of 119 checks passed
@PProfizi PProfizi changed the title Modify test for internal fluid variables test: Modify test for internal fluid variables Jul 30, 2025
@rafacanton rafacanton deleted the rcanton/cff_new_vars branch July 30, 2025 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests Related to tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants