Unit testing for workspace uploader

[Xiangs18]

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: +1.61% 🎉

Comparison is base (5958b2c) 41.35% compared to head (6389c7b) 42.97%.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #393      +/-   ##
==========================================
+ Coverage   41.35%   42.97%   +1.61%     
==========================================
  Files          68       68              
  Lines        5445     5445              
==========================================
+ Hits         2252     2340      +88     
+ Misses       3193     3105      -88     

see 2 files with indirect coverage changes

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

[Tianhao-Gu]

LGTM

[MrCreosote]

So usually unit tests shouldn't be accessing internal APIs [1], but given that real testing would require contacting kbase services, using real assembly files, etc. this seems reasonable for now, although not comprehensive.

When we get to the serious test writing stage we'll probably want to stand up the data services in a text fixture and run full tests, although to my mind we might want to do the database conversion first.

[1] https://blog.ploeh.dk/2015/09/22/unit-testing-internals/

approved accidentally

[MrCreosote]

LGTM

[MrCreosote]

&)&_&))^^Y&%(