<div dir="ltr"><div>Hello!</div><div><br></div><div>I'm writing to suggest that git-review requires to be covered by tests. There are no testing part in it at all, so maybe it is a good idea to cover it.</div><div><br>
</div><div>There are two types of tests to be implemented to cover the project:</div><div><span class="" style="white-space:pre">     </span>* unit tests to verify working of the separated methods</div><div><span class="" style="white-space:pre">    </span>* integrated tests that are using local copy of Gerrit</div>
<div><br></div><div>Unit tests have to mock needed calls to find out working of Python functions used in git-review.</div><div><br></div><div>The integrated tests are supposed to have the following workflow:</div><div><span class="" style="white-space:pre">   </span>* clear (or create) folder with the Gerrit environment for the every test </div>
<div><span class="" style="white-space:pre">    </span>* create new projects and push (--force) everything needed into test branches</div><div><span class="" style="white-space:pre">      </span>* clone project repository and create some new commits there</div>
<div><span class="" style="white-space:pre">    </span>* use git-review to check out how does it work</div><div><br></div><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>