xfail `tests/integration/cmor/_fixes/test_common.py::test_cl_hybrid_height_coord_fix_metadata` while Iris folk fix behaviour

[valeriupredoi]

Description

Issue described in https://github.com/SciTools/iris/issues/5806 and reported internally in #2357

Remember to plug intended behaviour after Iris folk fix that, or, un-xfail the test

@schlunma gives a nice overview of the test: This test tests a fix that converts a non-CF-compliant netCDF file into a CF-compliant file. One issue of this netCDF file is that the units of the bounds differ to those of the points.

Since 3.8, iris now ignores bounds with different (incompatible) units than the points. I think we need to wait for a reply of the iris people before we can properly fix this. It would be especially helpful to know if we can somehow retrieve the bounds after loading the cube.

---

Before you get started

• [x] ☝ Create an issue to discuss what you are going to do

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

• [x] 🛠 This pull request has a descriptive title and labels
• [x] 🛠 All checks below this pull request were successful

---

To help with the number pull requests:

• 🙏 We kindly ask you to review two other open pull requests in this repository

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.12%. Comparing base (bf04feb) to head (4b8ae81).

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #2363 +/- ## ======================================= Coverage 94.12% 94.12% ======================================= Files 241 241 Lines 13393 13393 ======================================= Hits 12606 12606 Misses 787 787 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[bouweandela]

Could over of you please give a bit more detail about why this test is failing?

[schlunma]

This test tests a fix that converts a non-CF-compliant netCDF file into a CF-compliant file. One issue of this netCDF file is that the units of the bounds differ to those of the points.

Since 3.8, iris now ignores bounds with different (incompatible) units than the points. I think we need to wait for a reply of the iris people before we can properly fix this. It would be especially helpful to know if we can somehow retrieve the bounds after loading the cube.

[valeriupredoi]

thanks, Manu! And yes, valid question, Bouwe :beer:

[bouweandela]

Just to make it clear that this is broken with iris 3.8, could you please open an issue in our own repository too with this description and which datasets can now now longer be fixed by ESMValCore? I'm not sure if the iris folks have plans to support all kinds of non-compliant data, so the fix may need an update if we want to use iris 3.8.