Skip to content

fix: cleanup the purge uploads as well when running the prune command#138

Merged
gaius-qi merged 1 commit intomainfrom
fix/cleanup-purge
Mar 25, 2025
Merged

fix: cleanup the purge uploads as well when running the prune command#138
gaius-qi merged 1 commit intomainfrom
fix/cleanup-purge

Conversation

@chlins
Copy link
Member

@chlins chlins commented Mar 25, 2025

This pull request introduces several changes to the backend and storage packages, primarily focusing on adding a new method for purging uploads and updating the mock functions accordingly. The most important changes include adding the PerformPurgeUploads method to the storage interface, updating the backend to call this new method, and adding mock functions for testing.

Backend and Storage Enhancements:

Mock Function Updates:

  • test/mocks/backend/backend.go: Added mock functions for the Fetch method, including the Backend_Fetch_Call struct and helper methods.
  • test/mocks/storage/storage.go: Added mock functions for the PerformPurgeUploads method, including the Storage_PerformPurgeUploads_Call struct and helper methods.

@chlins chlins force-pushed the fix/cleanup-purge branch from ac2f784 to b35516e Compare March 25, 2025 08:34
Signed-off-by: chlins <chlins.zhang@gmail.com>
@chlins chlins force-pushed the fix/cleanup-purge branch from b35516e to 83a1d49 Compare March 25, 2025 08:39
Copy link
Member

@gaius-qi gaius-qi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gaius-qi gaius-qi merged commit da51512 into main Mar 25, 2025
5 checks passed
@gaius-qi gaius-qi deleted the fix/cleanup-purge branch March 25, 2025 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants