Fix: return response caching

[Bento007]

Reviewers

Functional:

Readability:

---

Changes

• add check when s3_uri is None
• add typing
• add additional testing

[codecov]

Codecov Report

Merging #199 (608b12b) into main (fdda2f4) will increase coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #199      +/-   ##
==========================================
+ Coverage   70.40%   70.41%   +0.01%     
==========================================
  Files         132      132              
  Lines       10022    10026       +4     
==========================================
+ Hits         7056     7060       +4     
  Misses       2966     2966              

Flag	Coverage Δ
frontend	70.41% <100.00%> (+0.01%)	⬆️
javascript	70.41% <100.00%> (+0.01%)	⬆️
smokeTest	?
unitTest	70.41% <100.00%> (+0.01%)	⬆️

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

Impacted Files	Coverage Δ
server/app/api/util.py	100.00% <100.00%> (ø)
server/data_common/dataset_metadata.py	93.75% <100.00%> (+0.09%)	⬆️
app/api/util.py	100.00% <0.00%> (ø)
data_common/dataset_metadata.py	93.75% <0.00%> (+0.09%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fdda2f4...608b12b. Read the comment docs.

[atolopko-czi]

👏 LGTM!