Follow-up from "Enable Project Import to import from remote object storage [RUN ALL RSPEC] [RUN AS-IF-FOSS]"
<!--IssueSummary start-->
<details>
<summary>
Everyone can contribute. [Help move this issue forward](https://handbook.gitlab.com/handbook/marketing/developer-relations/contributor-success/community-contributors-workflows/#contributor-links) while earning points, leveling up and collecting rewards.
</summary>
- [Close this issue](https://contributors.gitlab.com/manage-issue?action=close&projectId=278964&issueIid=330021)
</details>
<!--IssueSummary end-->
The following discussion from !59033 should be addressed:
- [ ] @alexkalderimis started a [discussion](https://gitlab.com/gitlab-org/gitlab/-/merge_requests/59033#note_567972248):
> This is probably OK, but I can't help but feel that the cleaner OO solution here is to have the `import_export_upload` know how to save itself to a file - then we can just call `import_export_upload.save_to(@archive_file)`, without any of this branching.
>
> Thoughts?
issue