<div dir="ltr">Craig,<div><br></div><div>Thanks for doing this huge lift. Not all heroes wear capes!</div><div><br></div><div>I'm still concerned about using the SWIG bindings to test the C API because subtle defects in the wrappers could hide deeper bugs and wish there were more direct tests, but RFC 72 is a big improvement.</div><div><br></div><div>I use pytest in most of my projects and like how it reduces the friction for writing tests. It uses a lot of magic, which some Python developers don't like, and so it isn't necessarily *the way* to test in Python. I think there is a small risk that the hard to understand patterns of the home-grown approach could be replaced by hard to understand fixtures in a pytest approach. The example shown in the RFC, for example: is making a fixture for every gdal-bin program or adding checker fixtures that make multiple asserts patterns that will help or hurt long term? I'm not super concerned, just want to point out that pytest isn't a panacea and doesn't default to the golden path of testing.</div><div><br></div><div>Thanks again!</div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 3, 2018 at 4:49 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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<br><br>This is a call for discussion on RFC 72: Update autotest suite to use pytest<br><br><a href="https://trac.osgeo.org/gdal/wiki/rfc72_pytest" target="_blank">https://trac.osgeo.org/gdal/wiki/rfc72_pytest</a><br><br>Summary:<br><br>The document proposes and describes conversion of the existing Python autotest suite to use the ​pytest framework.<br><br>Using pytest provides significant productivity gains for writing, reading and debugging python tests, compared with the current home-grown approach. <br><div><br></div><div>--</div><div><br></div>Please submit feedback soon, as we'd like to have this merged in time for the imminent GDAL 2.4.0 RC if possible.<br><br>Thanks<br>Craig de Stigter<br><br><br></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" data-smartmail="gmail_signature"><div dir="ltr">Sean Gillies</div></div>