<div dir="ltr">I may spend some time on this subject, so please share your script if you can so that I can look through it and see what can be done to provide some automated gating.<div class="gmail_extra"><br><br><div class="gmail_quote">
On Thu, Jun 20, 2013 at 11:44 PM, Jeremy Stanley <span dir="ltr"><<a href="mailto:fungi@yuggoth.org" target="_blank">fungi@yuggoth.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 2013-06-20 15:56:22 +0400 (+0400), Dina Belova wrote:<br>
> I'm writing to suggest that git-review requires to be covered by<br>
> tests. There are no testing part in it at all, so maybe it is a<br>
> good idea to cover it.<br>
</div>[...]<br>
<br>
(Cc'ing you since I'm unsure whether you're subscribed, but setting<br>
MFT to the infra list to avoid unnecessary cross-posting)<br>
<br>
We've got an open bug on that topic I've been wanting to address for<br>
a while but haven't been able to prioritize it sufficiently since<br>
git-review is a fairly low-volume project as contributions go:<br>
<br>
    <a href="https://launchpad.net/bugs/1048724" target="_blank">https://launchpad.net/bugs/1048724</a><br>
<br>
In short, right now I manually perform integration tests of every<br>
proposed change using a very hackish shell script that interacts<br>
with a Gerrit sandbox in the ways we're concerned it might break<br>
under Python 2.6, 2.7, 3.2 and 3.3. This is of course very time<br>
consuming, so the current thought is that we should have an<br>
expect-like dummy shell underneath git-review which expects it to<br>
attempt to run particular commands and then returns the correct<br>
responses. This could be initially constructed by inserting a simple<br>
shim to record all shell I/O during one of my integration tests, so<br>
that it could be played back through the dummy shell later in<br>
automated tests.<br>
<span class="HOEnZb"><font color="#888888">--<br>
Jeremy Stanley<br>
</font></span></blockquote></div><br><br clear="all"><div><br></div>-- <br><div style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">Best regards,</div><div style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:13px;background-color:rgb(255,255,255)">
Dina</div>
</div></div>