Fix bug when reading domain IDs from .vtus

[javijv4]

Current situation

Resolves #520

Release Notes

• Fixed bug
• Added an FSI and an CEP test that checks the functionality to read domain IDs from a .vtu file.

Testing

New tests are:

• tests/cases/fsi/pipe_3d_domains
• tests/cases/cep/slab_domains

Code of Conduct & Contributing Guidelines

• I agree to follow the Code of Conduct and Contributing Guidelines.

[codecov]

Codecov Report

❌ Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 67.84%. Comparing base (fbf70b4) to head (17e1d13).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
Code/Source/solver/VtkData.cpp	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #521      +/-   ##
==========================================
+ Coverage   67.79%   67.84%   +0.05%     
==========================================
  Files         168      168              
  Lines       32798    32797       -1     
  Branches     5767     5768       +1     
==========================================
+ Hits        22234    22252      +18     
+ Misses      10426    10408      -18     
+ Partials      138      137       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ktbolt]

Looks good !

[ktbolt]

@javijv4 Testing reading domain IDs from a file should really be a unit test; does not make sense to have two pipe_3d tests that differ only by how they read domain IDs.

Adding a unit test is probably too much to do for this right now so just remove the pipe_3d_domains test.

[ktbolt]

@javijv4 There are two solver XML files in svMultiPhysics/tests/cases/cep/slab_domains. Can we run both of them ?

[javijv4]

@ktbolt, yep. I removed the pipe test and updated the cep slab_domains test to run both .xml.

[kko27]

Looks good to me!