Codecov Report

Merging #982 (89aedd8) into main (aab9f14) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #982      +/-   ##
==========================================
+ Coverage   86.13%   86.15%   +0.01%     
==========================================
  Files          92       92              
  Lines        5793     5798       +5     
==========================================
+ Hits         4990     4995       +5     
  Misses        803      803              

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@fepegar changes look good.

only minor comment is access to the private attribute _loaded outside the class, but not really part of this merge request

Do you mean in the tests? I think that's fine, right?

yes, in the test. If python would have real support for private/protected attribute it is not even possible to access it in the test. See also comment here for example: https://stackoverflow.com/questions/1093020/unit-testing-and-checking-private-variable-value

But minor detail, not really important here I think

Alright, thanks for reviewing and for sharing the link. I'll leave it like this as it still is an attribute that shouldn't be needed in the API, and we're just checking that the internals of the class look as they should.

I'll merge in a minute.

Thanks for your contribution, @dmus!