kube icon indicating copy to clipboard operation
kube copied to clipboard

Upgrade tarpaulin for new features

Open clux opened this issue 1 year ago • 0 comments

Want to use a less ancient version of tarpaulin and get new features.

  • [ ] Bump to latest (currently 0.25.0) from our pin at 0.18.5 from Nov 2021
  • [ ] Find a better GH Action approach (see below for the issues with actions-rs/tarpaulin)
  • [ ] Try the proc macro coverage so that we can get credit for tests in kube-derive (currently it looks bad but it is tested)
  • [ ] Do an all-features build in coverage (to ensure numbers are the most accurate and the most tests run)
  • [ ] Try to include one or two examples if it can be done quickly (like we do for unit tests atm)

A lot of this can be attempted locally by simply running cargo install cargo-tarpaulin then running just coverage inside the root of this repo (with a local k3d / non-important kubernetes cluster at your context).

Some of this has been attempted before. #1143 most recently. But it's annoying because: a) changes have surprising results, b) iteration cycles are slow, and c) we never got around many of the problems.

Problems encountered

It is possible that there is a lot more work to be done upstream before we need to really do anything here - in particular, see the master issue about Misses in coverage. So in the interest of having a place to resume from down the line, I am stashing knowledge here for a potentially ambitious future me, or anyone who wants to play with this.

It's a minor nice-to-have CI issue (tarpaulin and the actions are already great helpers!) and it's not really costing us anything real by lagging behind for another year or three. At least as long as the nodejs 12 deprecation issue on CI from the action becomes breaking.

clux avatar Feb 18 '23 14:02 clux