Skip to content
This repository was archived by the owner on Sep 11, 2023. It is now read-only.

Write tests for all the Manager methods #322

Open
JackKelly opened this issue Nov 2, 2021 · 3 comments
Open

Write tests for all the Manager methods #322

JackKelly opened this issue Nov 2, 2021 · 3 comments
Labels
enhancement New feature or request

Comments

@JackKelly
Copy link
Member

No description provided.

@JackKelly JackKelly added the enhancement New feature or request label Nov 2, 2021
@JackKelly JackKelly moved this to Todo in Nowcasting Nov 2, 2021
@JackKelly JackKelly mentioned this issue Nov 2, 2021
Merged
30 tasks
@peterdudfield
Copy link
Contributor

I think PR has done it - #363

@peterdudfield
Copy link
Contributor

peterdudfield commented Nov 10, 2021 •
edited
Loading

I would like to see the coverage of the tests, to see if we need to write more - openclimatefix/.github#18

@JackKelly
Copy link
Member Author

Need to write a test for the new Manager.configure_loggers() function. I've added TODOs to manager.py and test_manager.py.

@JackKelly JackKelly mentioned this issue Nov 16, 2021
Merged
7 tasks
@peterdudfield peterdudfield mentioned this issue Nov 18, 2021
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
No open projects
Nowcasting
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JackKelly @peterdudfield