Only call size once per data provider

[Vampire]

No description provided.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.84%. Comparing base (2c7db77) to head (b6ede76).
Report is 155 commits behind head on master.

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #2032      +/-   ##
============================================
+ Coverage     80.44%   81.84%   +1.39%     
- Complexity     4337     4609     +272     
============================================
  Files           441      448       +7     
  Lines         13534    14445     +911     
  Branches       1707     1827     +120     
============================================
+ Hits          10888    11822     +934     
+ Misses         2008     1946      -62     
- Partials        638      677      +39     

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

[AndreasTu]

Thanks for the fix!

[Vampire]

Argh, damn codecov.
Now that I change less lines, it doesn't like the null case not being tested.
Strange actually because when doing the changes it definitely was null at some point which is why I added the null case. :-/

[Vampire]

I was not able to reproduce the case with the current code, maybe it was in some intermediate state.
I also from a cursory look do not see how it should be able to happen.
So a hopefully last review please. :-)