Implement download callback through job state change

[MrCreosote]

Next is submitting to JAWS

[codecov]

Codecov Report

Attention: Patch coverage is 45.71429% with 19 lines in your changes missing coverage. Please review.

Project coverage is 56.20%. Comparing base (46fc710) to head (9b19ade).
Report is 3 commits behind head on main.

Files with missing lines	Patch %	Lines
cdmtaskservice/job_state.py	0.00%	5 Missing ⚠️
cdmtaskservice/mongo.py	37.50%	5 Missing ⚠️
cdmtaskservice/routes.py	37.50%	5 Missing ⚠️
cdmtaskservice/jobflows/nersc_jaws.py	33.33%	4 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #133   +/-   ##
=======================================
  Coverage   56.20%   56.20%           
=======================================
  Files          31       31           
  Lines        1854     1870   +16     
=======================================
+ Hits         1042     1051    +9     
- Misses        812      819    +7     

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

[Tianhao-Gu]

works. But it might be clearer to validate these parameters first and store them in local variables instead of calling validators inline.

[MrCreosote]

yeah, I used to do it that way, but you wind up with a potentially long list of validators at the start of each method. Inlining them makes the method easier to read imo since the validator calls are so simple