🐛 Add taskgroup when uploading binary

[ibolton336]

• Improve handling of taskgroup state by using contextAPI
• Async create the taskgroup before uploading the custom binary in the single app use case. This is currently broken because we require a task group to exist before navigating to the upload file screen. This regression was caused by the move away from useEffect here 🐛 Reset wizard form after analysis run #1267
  Resolves https://issues.redhat.com/browse/MTA-1175

[codecov]

Codecov Report

Patch coverage: 29.16% and project coverage change: -0.14% ⚠️

Comparison is base (435b73a) 43.09% compared to head (f0ed7a4) 42.96%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1280      +/-   ##
==========================================
- Coverage   43.09%   42.96%   -0.14%     
==========================================
  Files         143      144       +1     
  Lines        4297     4317      +20     
  Branches      998      999       +1     
==========================================
+ Hits         1852     1855       +3     
- Misses       2364     2381      +17     
  Partials       81       81              

Flag	Coverage Δ
client	42.96% <29.16%> (-0.14%)	⬇️
server	∅ <ø> (∅)

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

Files Changed	Coverage Δ
...tions/analysis-wizard/components/upload-binary.tsx	13.18% <15.00%> (-0.33%)	⬇️
...s/applications/analysis-wizard/analysis-wizard.tsx	64.16% <25.00%> (+1.17%)	⬆️
...ns/analysis-wizard/components/TaskGroupContext.tsx	40.00% <40.00%> (ø)
...ages/applications/analysis-wizard/custom-rules.tsx	15.27% <50.00%> (+0.48%)	⬆️
...pp/pages/applications/analysis-wizard/set-mode.tsx	86.20% <100.00%> (+2.33%)	⬆️

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

[sjd78]

This is the only bit that looks suspicious to me under brain compile.

What happens if the taskGroup changes? Maybe something like from undefined to { id: 5 }?

I'll run through some smoke tests in a bit. Maybe that will clear up my confusion.

[ibolton336]

+1 Fair concern. The taskgroup should always exist by the time the user arrives at the custom-rules step as it is created either 1) when onMove to step 2 of the wizard or 2) user uploads a binary on step 1 of the wizard.

[sjd78]

https://react.dev/learn/reusing-logic-with-custom-hooks#passing-reactive-values-between-hooks

Ok so I'm not concerned about that anymore. 😄

[sjd78]

LGTM