zed icon indicating copy to clipboard operation
zed copied to clipboard

Refactor backupCreate for testability

Open ivanauth opened this issue 2 months ago • 1 comments

This PR makes the backup creation code easier to test by extracting dependencies.

Changes

  • Extract backupCreateImpl that accepts a client and progress tracker as dependencies
  • Add ProgressTracker interface to separate progress file logic from backup logic
  • Now possible to test edge cases like PermissionDenied without spinning up a full server

Also fixes a time.Tick goroutine leak along the way.

Resolves #569

ivanauth avatar Dec 19 '25 21:12 ivanauth