Refactor

[bio-boris]

Description of PR purpose/changes

• Please include a summary of the change and which issue is fixed.
• Please also include relevant motivation and context.
• List any dependencies that are required for this change.

Jira Ticket / Github Issue

• [ ] Added the Jira Ticket to the title of the PR e.g. (DATAUP-69 Adds a PR template)

Testing Instructions

• Details for how to test the PR:
• [ ] Tests pass in Github Actions and locally
• [ ] Changes available by spinning up a local test suite and doing X

Dev Checklist:

• [ ] My code follows the guidelines at https://sites.google.com/truss.works/kbasetruss/data-upload-project/development
• [ ] I have performed a self-review of my own code
• [ ] I have commented my code, particularly in hard-to-understand areas
• [ ] I have made corresponding changes to the documentation
• [ ] My changes generate no new warnings
• [ ] I have added tests that prove my fix is effective or that my feature works
• [ ] New and existing unit tests pass locally with my changes
• [ ] Any dependent changes have been merged and published in downstream modules
• [ ] I have run Black and Flake8 on changed Python Code manually or with git precommit (and the Github Actions build passes)

Updating Version and Release Notes (if applicable)

• [ ] Version has been bumped for each release
• [ ] Release notes have been updated for each release (and during the merge of feature branches)

[lgtm-com[bot]]

This pull request introduces 5 alerts when merging 9496e48ade0be5c300b48496adf9bfa926fc7b34 into 77231e196fdb0e59b2c26ed903826b5da9ebf29a - view on LGTM.com

new alerts:

• 3 for Module-level cyclic import
• 1 for Unused import
• 1 for Wrong number of arguments in a class instantiation