chore: optimize defer and test

[jim3ma]

Signed-off-by: Jim Ma [email protected]

Description

1. Optimize defer in peerTaskConductor.Done and peerTaskConductor.Fail.
2. Fix concurrent test failed in runPeerTaskConductor

Related Issue

Motivation and Context

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
• Documentation Update (if none of the other choices apply)

Checklist:

• Code compiles correctly.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov-commenter]

Codecov Report

Merging #1010 (734016f) into main (5083963) will decrease coverage by 0.04%.
The diff coverage is 55.17%.

@@            Coverage Diff             @@
##             main    #1010      +/-   ##
==========================================
- Coverage   45.60%   45.56%   -0.05%     
==========================================
  Files         128      128              
  Lines        9107     9122      +15     
==========================================
+ Hits         4153     4156       +3     
- Misses       4483     4493      +10     
- Partials      471      473       +2     

Flag	Coverage Δ
Object-compatibility-e2etests	?
e2etests	?
unittests	45.56% <55.17%> (-0.05%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
client/daemon/storage/storage_manager.go	19.20% <ø> (ø)
client/daemon/peer/peertask_conductor.go	56.72% <40.00%> (-0.63%)	⬇️
client/daemon/peer/peertask_manager.go	43.75% <66.66%> (+1.85%)	⬆️
client/daemon/peer/peertask_file.go	57.60% <100.00%> (ø)
client/daemon/peer/peertask_stream.go	65.41% <100.00%> (ø)
client/daemon/peer/piece_broker.go	81.48% <0.00%> (-3.71%)	⬇️

[gaius-qi]

LGTM