Task/WP-728: Mutation hook: Copy file

[jalowe13]

Overview

Updating hooks to use React Query mutations with Typescript. This mutation covers copying a file.

Related

• WP-728

Changes

• useCopy.js to useCopy.ts
• Identify types of payloads
• Wraps dispatch actions inside a mutation

Testing

• Navigate to the folder where the file is that you would like to copy
• Select the checkbox of the folder
• Click the copy button
• Select the destination directory to copy to

UI

[codecov]

Codecov Report

Attention: Patch coverage is 16.79389% with 109 lines in your changes missing coverage. Please review.

Project coverage is 72.88%. Comparing base (b01a86a) to head (71709a7).
Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
client/src/hooks/datafiles/mutations/useCopy.ts	16.79%	109 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1000       +/-   ##
===========================================
+ Coverage   60.44%   72.88%   +12.44%     
===========================================
  Files         211      534      +323     
  Lines        6285    33456    +27171     
  Branches      730     2981     +2251     
===========================================
+ Hits         3799    24385    +20586     
- Misses       2329     8876     +6547     
- Partials      157      195       +38     

Flag	Coverage Δ
javascript	75.76% <16.79%> (?)
unittests	60.44% <ø> (ø)

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

Files with missing lines	Coverage Δ
client/src/hooks/datafiles/mutations/useCopy.ts	16.79% <16.79%> (ø)

... and 322 files with indirect coverage changes

---- 🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests
• JS Bundle Analysis - Avoid shipping oversized bundles

[jmcmillenmusic]

This works 100% as intended! Looks good to me!

[jarosenb]

This dispatch will occur immediately after copyCalls is constructed. You might need to use mutateAsync and await each mutation before dispatching the toast. mutateAsync returns a promise so you could use Promise.all() here.

[jarosenb]

LGTM! The final callback now doesn't fire until all operations are complete.