<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">+1 Kurt<br><div><br></div><div>Awesome!! It's great to see that my work in <a href="https://github.com/schwehr/gdal-autotest2/tree/master/python">https://github.com/schwehr/gdal-autotest2/tree/master/python</a> is obsolete.</div><div><br></div><div>Some very minor suggestions:</div><div><br></div><div>- "Support testing under Python 2 & Python 3 (2.7+)" Move the (2.7+) to be with Python 2.</div><div>- Mention that Fiona and Rasterio also use pytest</div><div>- Add the word "existing" so it's obvious right off that the first example is not new. "<span style="color:rgb(0,0,0);font-family:Verdana,Arial,"Bitstream Vera Sans",Helvetica,sans-serif;font-size:13px">A typical GDAL python unit test:" -> "</span><span style="color:rgb(0,0,0);font-family:Verdana,Arial,"Bitstream Vera Sans",Helvetica,sans-serif;font-size:13px">A typical existing GDAL python unit test:"</span></div><div><div>- Consider including an example with a float comparison.  Those drive me nuts in the old style and I want to make sure people know about how to do that with pytest</div><div>  <a href="https://stackoverflow.com/questions/8560131/pytest-assert-almost-equal">https://stackoverflow.com/questions/8560131/pytest-assert-almost-equal</a></div><div>- Can you give a rough comparison of how long the existing and new styles take to run.  A quick spot check says that they are pretty close with only 1604 looking to get worrisome.  Are there any things that could be listed where people contribute that might speed up the testing?</div><div>- Is there a way for folks with large machines / cloud instances to run the tests quicker without a lot of fiddling?</div><div>- I'm not a fan of GH and PR abbreviations.  Since this is on trac, it would be worth being more verbose </div><div><br></div><div><br></div><br class="gmail-Apple-interchange-newline"></div><div><br></div></div></div></div></div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Dec 5, 2018 at 2:41 PM Craig de Stigter <<a href="mailto:craig.destigter@koordinates.com">craig.destigter@koordinates.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi</div><div><br></div><div>I appreciate your comments on the pytest proposal and all the support to help get it this far. Given no actionable improvements have been suggested, and the feedback thus far seems encouraging...</div><div><br></div><div>I move to adopt RFC 72: Run tests with pytest.<br></div><div><br></div><div><a href="https://trac.osgeo.org/gdal/wiki/rfc72_pytest" target="_blank">https://trac.osgeo.org/gdal/wiki/rfc72_pytest</a></div><div><br></div><div><br></div>Cheers<div>Craig de Stigter<br></div></div>
_______________________________________________<br>
gdal-dev mailing list<br>
<a href="mailto:gdal-dev@lists.osgeo.org" target="_blank">gdal-dev@lists.osgeo.org</a><br>
<a href="https://lists.osgeo.org/mailman/listinfo/gdal-dev" rel="noreferrer" target="_blank">https://lists.osgeo.org/mailman/listinfo/gdal-dev</a></blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature">--<div><a href="http://schwehr.org" target="_blank">http://schwehr.org</a></div></div>