Add all test files to git LFS

[Schamper]

The decision on when to include a file in LFS was getting a bit annoying, and sometimes we forget. Just put it all in LFS.

We should update either the README or the docs at some point to include a small explanation on LFS too.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.24%. Comparing base (d98a557) to head (51f46b5).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #562      +/-   ##
==========================================
- Coverage   74.28%   74.24%   -0.04%     
==========================================
  Files         285      285              
  Lines       23605    23605              
==========================================
- Hits        17535    17526       -9     
- Misses       6070     6079       +9     

Flag	Coverage Δ
unittests	74.24% <ø> (-0.04%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[Schamper]

Hm this does remove the ability to preview a (text based) test file in a PR, which is also very annoying.

[pyrco]

Hm this does remove the ability to preview a (text based) test file in a PR, which is also very annoying.

That's an unfortunate side effect. Putting tests/_data/* in LFS is right thing to do I think. This is a shortcoming of GitHub and I'm leaning to not letting that stand in the way of the this.

[Schamper]

Hm this does remove the ability to preview a (text based) test file in a PR, which is also very annoying.

That's an unfortunate side effect. Putting tests/_data/* in LFS is right thing to do I think. This is a shortcoming of GitHub and I'm leaning to not letting that stand in the way of the this.

Yes, agreed.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 74.90%. Comparing base (ab8104f) to head (c427b90).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #562   +/-   ##
=======================================
  Coverage   74.90%   74.90%           
=======================================
  Files         288      288           
  Lines       24115    24115           
=======================================
  Hits        18064    18064           
  Misses       6051     6051           

Flag	Coverage Δ
unittests	74.90% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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