Thanks for the code! It looks pretty good in general. I'm going to make a few stylistic fixes (indentation, parameter order, multiline import formatting), and I'll add a few more "bad path" tests to ensure the exception still gets raised when expected. Then I'll merge and commit.
Thanks for the code! It looks pretty good in general. I'm going to make a few stylistic fixes (indentation, parameter order, multiline import formatting), and I'll add a few more "bad path" tests to ensure the exception still gets raised when expected. Then I'll merge and commit.
Cheers!