Hi,

The tests about using custom files (especially input ones) should be present in the example tests. Otherwise, the user is somewhat left to read your mind.

Edit: well, that's actually an issue, when it comes to the use of input_, because... What sane programmer would replace the builtin in the first place?

.

Ok, now it should print a nice error message for your solution.

You need to ensure that the test framework is robustly printing its messages. See my submitted code.

Thanks for the feedback. What's being tested is that sys.stdin is restored, even if an exception is raised inside the function. It's changed now.

There should be some message for the error handling test cases as it is not clear what is being tested here.

Your usage of dicts is very interesting and clear!

Good! I did not know the nested format string and lazy regex was possible.

Very good!