<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Jul 14, 2015 at 3:22 PM Robert Collins <<a href="mailto:robertc@robertcollins.net">robertc@robertcollins.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 15 July 2015 at 10:05, Ethan Furman <<a href="mailto:ethan@stoneleaf.us" target="_blank">ethan@stoneleaf.us</a>> wrote:<br>
> On 07/14/2015 02:53 PM, Robert Collins wrote:<br>
...<br>
>>> I don't think unittest can protect its users from such things.<br>
>><br>
>><br>
>> It can't, but there is a sliding scale of API usability, and we should<br>
>> try to be up the good end of that :).<br>
><br>
><br>
> I hope you're not suggesting that supporting misspellings, and thereby<br>
> ruling out the proper use of an otherwise fine variable name, is at the good<br>
> end of that scale?<br>
<br>
I'm not supporting the misspelling thing - see my suggestion earlier<br>
in this thread to move the mock assertions to standalone functions,<br>
removing the bug in that area *entirely* and eventually removing the<br>
check for method names starting with assert from mock entirely.<br>
<br>
What I am doing is rejecting the argument that because we can't fix<br>
every mis-use users might make, we therefore should not fix the cases<br>
where we can fix it.<br>
<br>
For clarity, I think we should:<br>
 - remove the assret check, it is I think spurious.<br>
 - add a set of functions to the mock module that should be used in<br>
preference to Mock.assert*<br>
 - mark the Mock.assert* functions as PendingDeprecation<br>
 - in 3.6 move the PendingDeprecation to Deprecated<br>
 - in 3.7 remove the Mock.assert* functions and the check for method<br>
names beginning with assert entirely.<br></blockquote><div><br></div><div>+1 from me </div></div></div>