cscli hubtest: extract method finalizeRun()

[mmetc]

there's no reason for it to be a persistent hook, "run" does not have subcommands

[github-actions]

@mmetc: There are no area labels on this PR. You can add as many areas as you see fit.

• /area agent
• /area local-api
• /area cscli
• /area appsec
• /area security
• /area configuration

Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

[codecov]

Codecov Report

❌ Patch coverage is 0% with 61 lines in your changes missing coverage. Please review.
✅ Project coverage is 62.88%. Comparing base (5fcc066) to head (da0847c).
⚠️ Report is 4 commits behind head on master.

Files with missing lines	Patch %	Lines
cmd/crowdsec-cli/clihubtest/run.go	0.00%	61 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4181      +/-   ##
==========================================
- Coverage   62.89%   62.88%   -0.01%     
==========================================
  Files         467      467              
  Lines       33280    33282       +2     
==========================================
- Hits        20932    20931       -1     
- Misses      10225    10227       +2     
- Partials     2123     2124       +1     

Flag	Coverage Δ
bats	46.10% <0.00%> (-0.02%)	⬇️
unit-linux	35.69% <0.00%> (-0.03%)	⬇️
unit-windows	24.93% <0.00%> (-0.09%)	⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

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

[sabban]

lgtm