Add an extended tag for time-intensive unit tests

[mikeingold]

Changes

• Establishes an :extended @testitem tag for time-intensive tests that can forgo CI (still available for local testing in VS Code)
  • Tagged Box in 4D and Tetrahedron

This comes with the downside that integration of these two geometry types won’t be checked automatically with CI, and a code coverage “reduction”, but I think the actual practical impact is minor. For any changes that would specifically impact these two geometries, we’ll have to make sure to run the tests locally. Slightly inconvenient for a niche case but allows for faster iterations in development of everything else.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.76%. Comparing base (87ac32d) to head (844cffd).
Report is 1 commits behind head on main.

Additional details and impacted files

@@             Coverage Diff             @@
##              main     #109      +/-   ##
===========================================
- Coverage   100.00%   95.76%   -4.24%     
===========================================
  Files           17       17              
  Lines          260      260              
===========================================
- Hits           260      249      -11     
- Misses           0       11      +11     

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

[mikeingold]

We're down to <10 minute CI times just by pushing two geometry types into :extended test sets. 😃

[JoshuaLampert]

Could we put the test for a wrong number of parametric coordinates into another @testitem to keep at least that one covered? It currently is in the 4D Box test (lines 199-200), but could also live somewhere else.

[mikeingold]

Good catch! I just moved it from 4D to 2D

[JoshuaLampert]

Thanks!