Reset processedCount when downloading all artifacts

[campersau]

Currently the totalNumberOfFilesToProcess was reset for each artifact but not the processedCount which lead to percentage values higher than 100% when downloading all artifacts. Also resetting the processedCount to 0 fixes that and scopes the counts to a single artifact.

Fixes actions/download-artifact#95

Here is another output with > 100% percent in log output https://github.com/campersau/CefSharp/runs/3389459433?check_suite_focus=true#step:2:3. There are two artifacts with 4 files each:

No artifact name specified, downloading all artifacts
Creating an extra directory for each artifact that is being downloaded
Total number of files that will be downloaded: 4
Total file count: 4 ---- Processed file #3 (75.0%)
Total number of files that will be downloaded: 4
Total file count: 4 ---- Processed file #6 (150.0%) <!--- Total file count 4 but file 6!
There were 2 artifacts downloaded
Artifact netcore was downloaded to D:\a\CefSharp\CefSharp\netcore
Artifact netframework was downloaded to D:\a\CefSharp\CefSharp\netframework
Artifact download has finished successfully

[brcrista]

@konradpabjan could you take a look when you get a chance since you have context on this code?

[brcrista]

Also looks like we need to do another npm audit fix PR: https://github.com/actions/toolkit/pull/889/checks?check_run_id=3400902928

[konradpabjan]

Looks good!

(the failing audit test are unrelated to these changes so we're good).

I'll work on pushing out a new version of the @actions/artifact package with these changes

[TWiStErRob]

No tests? 😮

[konradpabjan]

@TWiStErRob

The changes here are relatively simple so it doesn't hurt though I'm running some extra tests locally to confirm behavior. There are a lot of tests for other parts of the upload/download process but the status reporter is something that is currently lacking https://github.com/actions/toolkit/tree/main/packages/artifact/__tests__