Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.08%. Comparing base (0775624) to head (b382289).
⚠️ Report is 10 commits behind head on main.

@@            Coverage Diff             @@
##             main      #55      +/-   ##
==========================================
- Coverage   97.09%   97.08%   -0.02%     
==========================================
  Files           2        2              
  Lines         448      446       -2     
  Branches       52       48       -4     
==========================================
- Hits          435      433       -2     
- Misses          9       10       +1     
+ Partials        4        3       -1     

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

If _boundary_prefix and _material_prefix use the same name (e.g. _metadata_prefix) we might be able to streamline the code further by removing some argument passing, but maybe at a loss of clarity. Thoughts @gonuke?

If _boundary_prefix and _material_prefix use the same name (e.g. _metadata_prefix) we might be able to streamline the code further by removing some argument passing, but maybe at a loss of clarity. Thoughts @gonuke?

I want to leave open the possibility of other metadata being stored on these groups such that there may be additional prefixes.

If _boundary_prefix and _material_prefix use the same name (e.g. _metadata_prefix) we might be able to streamline the code further by removing some argument passing, but maybe at a loss of clarity. Thoughts @gonuke?

I want to leave open the possibility of other metadata being stored on these groups such that there may be additional prefixes.

Good call. I think I know what you have in mind.

Thanks @gonuke!

I'm going to bypass the coverage check and merge. This improvement is worth the decrease of 0.02% in our 97% coverage I think 😃

Thanks!

I'm actually a little confused by the coverage test. There appear to be multiple metrics and I added tests that addressed the first metric. I can't for the life of me figure out where coverage is now reduced by this....