The attached patch updates existing tests to use the new get_tu function.<br><br><div class="gmail_quote">On Fri, Feb 17, 2012 at 10:17 AM, Gregory Szorc <span dir="ltr"><<a href="mailto:gregory.szorc@gmail.com">gregory.szorc@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote"><div class="im">On Fri, Feb 17, 2012 at 1:29 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Gregory,<br>
<br>
in test_type.py: use the new get_tu function in the other tests around<br>
the new one, too? (otherwise why pull out a function)<br></blockquote></div><div><br>Sure, I can do that. (I wasn't sure if you would mind the scope creep or would prefer a follow-up).<br> </div><div class="im"><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">


<br>
+    ok_(isinstance(foo.type.is_function_variadic(), bool))<br>
This seems a non-pythonic test to me - why do we care about the<br>
underlying type of the result as long as it evaluates to True / False<br>
in bool context correctly?<br></blockquote></div><div><br>Because it ensures the API contract is behaving properly. If the return type changes accidentally, there would be no check to catch that. I feel that breaking the return type should be a conscientious decision and should be caught via testing. <br>

 </div><div class="im"><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Generally, why use ok_ when all the tests around use normal assert?<br></blockquote></div><div><br>It saves a couple of characters. I also like the style of a function rather than magic of the assert statement. Neither are great reasons, I'll admit.<br>

</div></div>
</blockquote></div><br>