Skip to content

Conversation

@Mia-Cross
Copy link
Contributor

This PR duplicates the acctest.CompressCassette function into acctest.CompressCassetteV3 and acctest.CompressCassetteV4 in order to be able to handle both v3 and v4 cassettes.

It also re-records the cassettes from the instance package that were previously overwritten in v3 when they should have been using v4.

@Mia-Cross Mia-Cross self-assigned this Oct 30, 2025
@Mia-Cross Mia-Cross added instance Instance issues, bugs and feature requests tests labels Oct 30, 2025
@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 68.11594% with 22 lines in your changes missing coverage. Please review.
✅ Project coverage is 1.77%. Comparing base (ed9d79b) to head (f1ea79b).
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
internal/acctest/vcr_compress.go 79.31% 7 Missing and 5 partials ⚠️
cmd/vcr-compressor/main.go 0.00% 9 Missing ⚠️
internal/acctest/acctest.go 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff            @@
##           master   #3430      +/-   ##
=========================================
+ Coverage    1.66%   1.77%   +0.11%     
=========================================
  Files         391     391              
  Lines       43262   43325      +63     
=========================================
+ Hits          720     770      +50     
- Misses      42459   42467       +8     
- Partials       83      88       +5     

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

@Mia-Cross Mia-Cross marked this pull request as ready for review October 30, 2025 16:47
@Mia-Cross Mia-Cross requested review from a team and remyleone as code owners October 30, 2025 16:47
@remyleone remyleone added this pull request to the merge queue Oct 30, 2025
Merged via the queue into scaleway:master with commit cbf408e Oct 30, 2025
78 checks passed
@Mia-Cross Mia-Cross deleted the vcr_compressor_split_versions branch October 30, 2025 17:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

instance Instance issues, bugs and feature requests tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants