I'm also tempted to ask that the test conditions match the name. It seems backwards that the test "forbids" and yet the assertion is assertTrue as in the tests that "allows".
The change itself looks fine though. Approving the MP with these suggestions.
Thanks for the update. Per the test style guide, please add docstrings to the tests. /dev.launchpad. net/TestsStyleG uide#line- 416
https:/
I'm also tempted to ask that the test conditions match the name. It seems backwards that the test "forbids" and yet the assertion is assertTrue as in the tests that "allows".
The change itself looks fine though. Approving the MP with these suggestions.