missing integration stuff

[nocollier]

Description

Realized that I had missed some things needed for full integration.

Checklist

Please confirm that this pull request has done the following:

• Tests added
• Documentation added (where applicable)
• Changelog item added to changelog/

[nocollier]

@lewisjared I discovered that I missed a good bit in the integration which meant tests ran but didn't include ilamb things. Now I have some errors, but these last ones I don't understand. Do you have any idea of what I have missed?

[lewisjared]

Yup it turns out the tests are sensitive to new metrics and sample data so need to be tweaked each time either changes. I've got a fix for this in a pr I'm about to merge so hopefully that will resolve this #68

[lewisjared]

The changes look good though so feel free to assign it to me so I can merge once I fix the flakey test

[lewisjared]

I don't think that the slug had to be globally unique. The intention was that it is unique for a provider. The constraint is that the combination of the provider slug + the metric slug was unique

[nocollier]

I had a bunch (~10) failed tests due to some errors which reported 'unique slug' problems that were fixed when I made the change. But I will check this again.

[lewisjared]

Testing this locally after merging main looks to work

git checkout missing-ilamb
git merge origin/main
make virtual-environment
make test

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Files with missing lines	Coverage Δ
...etrics-ilamb/src/cmip_ref_metrics_ilamb/example.py	100.00% <100.00%> (ø)
packages/ref/src/cmip_ref/provider_registry.py	92.59% <100.00%> (+0.59%)	⬆️