Tidy up dependencies

[p-j-smith]

Description

Fixes #428

• remove duplicated transitive dependencies and depend on core and core extras (dev and test) where appropriate

Type of change

Please delete options accordingly to the description.

• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
• [ ] This change requires a documentation update

Suggested Checklist

• [ ] I have performed a self-review of my own code.
• [ ] I have made corresponding changes to the documentation.
• [ ] My changes generate no new warnings.
• [ ] I have commented my code, particularly in hard-to-understand areas.
• [ ] I have passed on my local host device. (see further details at the CONTRIBUTING document)
• [ ] Make sure your branch is up-to-date with main branch. See CONTRIBUTING for a general example to syncronise your branch with the main branch.
• [ ] I have requested PR review to UCLH-Foundtry/arc-dev
• [ ] I have adddressed and marked as resolved all the review comments in my PR.
• [ ] Finally, I have selected squash and merge

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 84.95%. Comparing base (5062d20) to head (2f7001d). Report is 1 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #492 +/- ## ======================================= Coverage 84.95% 84.95% ======================================= Files 86 86 Lines 3769 3769 ======================================= Hits 3202 3202 Misses 567 567 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[p-j-smith]

Thanks for the reviews!