test(sdk): add many tests for InternalApi.upload_file

[speezepearson]

I'm about to start writing an async equivalent for this function, so I want much more thorough unit tests.

[codecov]

Codecov Report

Merging #4539 (1887372) into main (58efdda) will increase coverage by 0.02%.
The diff coverage is 100.00%.

❗ Current head 1887372 differs from pull request most recent head 08fea76. Consider uploading reports for the commit 08fea76 to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4539      +/-   ##
==========================================
+ Coverage   82.98%   83.00%   +0.02%     
==========================================
  Files         261      261              
  Lines       33263    33263              
==========================================
+ Hits        27603    27610       +7     
+ Misses       5660     5653       -7     

Flag	Coverage Δ
functest	56.67% <0.00%> (+0.03%)	⬆️
unittest	73.19% <100.00%> (+<0.01%)	⬆️

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

Impacted Files	Coverage Δ
wandb/sdk/internal/internal_api.py	89.29% <ø> (-0.39%)	⬇️
wandb/sdk/internal/progress.py	82.50% <100.00%> (ø)
wandb/filesync/step_prepare.py	93.67% <0.00%> (-1.27%)	⬇️
wandb/sdk/lib/git.py	79.01% <0.00%> (ø)
wandb/integration/tensorboard/monkeypatch.py	91.35% <0.00%> (ø)
wandb/sdk/wandb_run.py	90.89% <0.00%> (+0.29%)	⬆️
wandb/sdk/lib/printer.py	89.24% <0.00%> (+0.63%)	⬆️
wandb/sdk/internal/system/system_info.py	91.55% <0.00%> (+3.24%)	⬆️

[kptkin]

wow... i'm excited about this feature!

[vanpelt]

Let's try to get these fixes into the next release. I think they are causing real failures in the wild.

[kptkin]

looks great! some great finds!

[kptkin]

@vanpelt @speezepearson I'm unfreezing this branch so merge it when you are ready (probably better sooner, so we could run regression)