refactor(grpc): simplify download task success handling

[gaius-qi]

Description

This pull request refactors the logic in both the download and upload gRPC server handlers to improve the order in which certain operations are performed after a download task succeeds. The main change is to ensure that the download_finished method is always called before checking for output path handling when the download is not a ranged request.

Key changes:

Refactoring task completion flow:

• The call to task_manager_clone.download_finished is now always executed immediately after a successful download, regardless of whether the download is ranged or not, in both DfdaemonDownloadServerHandler and DfdaemonUploadServerHandler implementations (dragonfly-client/src/grpc/dfdaemon_download.rs, dragonfly-client/src/grpc/dfdaemon_upload.rs). [1] [2]

Conditional output path handling:

• The check for download_clone.range.is_none() and the subsequent output path handling logic have been moved to occur after the download_finished call, ensuring proper task completion notification before any file system operations (dragonfly-client/src/grpc/dfdaemon_download.rs, dragonfly-client/src/grpc/dfdaemon_upload.rs). [1] [2]

Related Issue

Motivation and Context

Screenshots (if appropriate)

[chlins]

lgtm

[codecov]

Codecov Report

❌ Patch coverage is 0% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 42.78%. Comparing base (b06dc0b) to head (a877ce4).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
dragonfly-client/src/grpc/dfdaemon_download.rs	0.00%	7 Missing ⚠️
dragonfly-client/src/grpc/dfdaemon_upload.rs	0.00%	7 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1581      +/-   ##
==========================================
- Coverage   42.79%   42.78%   -0.01%     
==========================================
  Files          75       75              
  Lines       21211    21213       +2     
==========================================
  Hits         9077     9077              
- Misses      12134    12136       +2     

Files with missing lines	Coverage Δ
dragonfly-client/src/grpc/dfdaemon_download.rs	2.50% <0.00%> (-0.01%)	⬇️
dragonfly-client/src/grpc/dfdaemon_upload.rs	0.00% <0.00%> (ø)

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.