mila-iqia / milatools

Tools to connect to and interact with the Mila cluster
MIT License
63 stars 12 forks source link

[NOT FOR MERGE] chat about tests #57

Closed satyaog closed 7 months ago

satyaog commented 1 year ago

only a discussion to see what and how we should start adding more tests

satyaog commented 1 year ago

@lebrice , @breuleux , This is not for merge, I would only like to have your inputs on what we should test first maybe, how the tests should be implemented and anything I might have missed? The objective here would be to have tests for everything with minimal / quick mocks where we can and local tests with personal account where it's complicated. I'll start adding tests with minimal mocks but let me know if you have any inputs to give