start work on delete_cassettes()

[maelle]

Fix #552

Not ready, will need a second pass

[sckott]

@maelle nice! should i look for the current state of this branch or do you want to do more first?

[maelle]

I will do more first!

[maelle]

@sckott this PR was sort of an experiment, I worked with Claude CLI! I think it's ready for review. 😸

[maelle]

A bit confused by the test error that is unrelated to my changes?!

 ══ Failed tests ════════════════════════════════════════════════════════════════
── Failure ('test-lightswitch.R:16:5'): turned_off works if and only if no cassettes active ──
Adding new snapshot:
Code
  turned_off(5 + 5)
Condition
  Error in `turn_off()`:
  ! You must eject all cassettes before you can turn vcr off.
Backtrace:
    ▆
 1. └─base::eval(...)
 2.   └─base::eval(...)
 3.     └─testthat::expect_snapshot(turned_off(5 + 5), error = TRUE) at test-lightswitch.R:16:5

[ FAIL 1 | WARN 0 | SKIP 1 | PASS 598 ]
Error:
! Test failures.
Execution halted

[maelle]

Same failure in #554 that has no real edit, maybe due to the new testthat version?

[maelle]

Ah yes

Additionally, expect_snapshot() and friends will now fail when creating a new snapshot on CI, as that’s usually a signal that you’ve forgotten to run the snapshot code locally before committing.

I'll make a commit to main!

[sckott]

great i'll take a look soon!

[sckott]

Looks good so far. I'm still poking around.

Shouldn't there be a man/delete_cassettes.Rd file? Can you add that please

[maelle]

@sckott done, sorry about the delay 😸