<div>Hello List, I am reviewing my very first cubicweb's patch and </div><div>I'd like to know the best practices at this point to complete the </div><div>revision correctly.</div><div><br></div><div>Syt has already given me an explanation of what steps to follow, but I </div>
<div>would like to know how a deeper tests can be done (in case the </div><div>developer has the time to do some extra revisions of the patch, of course).</div><div>This take us to another topics that as a cubicweb developer I would </div>
<div>really like to know and understand better.</div><div><br></div><div>So following the instructions more or less what I should do is:</div><div><br></div><div>1.- Check if the patch is linked to a ticket. Not in my case. I guess </div>
<div>this is to get more information and understand better what </div><div>the patch does.</div><div>2.- Check if the commit messages and comments are enough to understand </div><div>the change. In case it isn't clear enough, we should write some </div>
<div>comments to ask for more information, right?</div><div>3.- Review the code looking for pitfall/bugs. I see this can be done </div><div>in cubicweb forge, looking at the version content of each patch.</div><div><br></div>
<div>At this point, I have checked the patch at first sight, but what I </div><div>would really like to do is to test it. Here is where it comes some </div><div>other topics.</div><div><br></div><div>So I go looking for the patch, and apply it on cubicweb.</div>
<div><br></div><div>1.- How do I know what branch (stable, default, other) should I use to </div><div>apply the patch?</div><div><br></div><div>In my case I used the stable branch, trying to have some luck.  Then </div><div>
run an application instance, just to see the screen messages or to </div><div>find out if something there is a critical error.</div><div><br></div><div>My patch's name is "[session] provide a simple transaction context </div>
<div>manager for session ", the name is clear, but due my lack of </div><div>experience in some cubicweb areas, I really would like to do some tests.</div><div><br></div><div>2.- I would think running cubiweb's tests with pytest tool would be </div>
<div>the least I can do, right? </div><div>Then I go to the test folder and start to run the test with pytest, </div><div>but I am not sure I am doing this right, because the tests don't </div><div>return messages</div>
<div><br></div><div>carlos@debian:~/src/fcubicweb/cubicweb/test$ pytest unittest_req.py </div><div>=========================  unittest_req.py  ==========================</div><div>**********************************************************************</div>
<div>*********</div><div><br></div><div>carlos@debian:~/src/fcubicweb/cubicweb/test$</div><div><br></div><div>I think I am missing some parameters or configuration here, am I?</div><div><br></div><div>3.- Well, let's say that tests ran ok, but at this point the tests I made could've not </div>
<div>even touch the code of the patch yet, nor to see if there is any </div><div>problem to run it, nor to test the logic of the change. This left me </div><div>the feeling of the patch hasn't been checked at this time.</div>
<div><br></div><div>What would be your recomendations about this situation?.</div><div><br></div><div>Best Regards</div><div>Carlos Balderas</div>