Skip to content

Minor refactoring and start of test coverage for zoo#260

Open
tennlee wants to merge 2 commits intoACCESS-Community-Hub:developfrom
tennlee:test_zoo
Open

Minor refactoring and start of test coverage for zoo#260
tennlee wants to merge 2 commits intoACCESS-Community-Hub:developfrom
tennlee:test_zoo

Conversation

@tennlee
Copy link
Copy Markdown
Collaborator

@tennlee tennlee commented Apr 29, 2026

Work still under way
Some refactoring of classes within Zoo
Some unit tests added (more work required)

Final Checks:

  • If no generative AI was used, then tick this box

Alternatively, if generative AI was used, then confirm you have:

  • Attributed any generative AI (such as GitHub Copilot) that was used in this PR. See our contributing guide for more information.
  • Included the name and version of the tool or system in the pull request
  • Described the scope of that use

Finally,

  • Mark the PR as ready to review. Note - we encourage you to ask for feedback at the outset or at any time during the work.

@tennlee tennlee requested a review from edoyango April 29, 2026 12:32
@tennlee
Copy link
Copy Markdown
Collaborator Author

tennlee commented Apr 29, 2026

Hi @edoyango I have started on some work to add a bit of test coverage to the zoo package. This includes some minor refactoring to start to simplify the import structure. I'm not quite sure whether to merge early and then keep going, or keep working away at this until the tests are more rigorous and provide more coverage. Let me know what you think of things, I could go either way.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant