[gdal-dev] Call for discussion: RFC88: Use GoogleTest framework for C/C++ unit tests

[Frank Warmerdam]

On Sun, Nov 20, 2022 at 8:21 PM Even Rouault <even.rouault at spatialys.com>
wrote:

> Hi Frank,
>
>
> I'm not a huge GoogleTest fan but then I'm also not saying it *worse* than
> any alternative, but the reasons are reasonably convincing, and I'm happy
> if those running and contributing tests are happy!
>
> Just curious what you don't like with GoogleTest ?
>

Even,

There is nothing meaningful to offer.  I've been mildly annoyed by the
dependencies it implied in other projects (ie. PixelLoom), but really this
is not material.  My point was more along the line of "I hate every test
environment / ticket system / language / ...", but that given that I do not
find it worse than most. :-)

Should I understand from the RFC that
> https://github.com/rouault/gdal/tree/gtest implements the migration?
> (ie. we have already done the work to migrate, we just need to agree on it?)
>
> Yes the implementation is ready in https://github.com/OSGeo/gdal/pull/6732
> . I find it more convincing when there's an implementation backing a RFC
> (it was partial when I submitted the RFC and I finished it as the initial
> feedback was positive)
>

Awesome.  While sometimes it is a bit "heavy" to implement an idea before
proposing it, it does make it much easier to adopt knowing the
implementation is waiting in the wings.

Best regards,
Frank

> Even
>
> -- http://www.spatialys.com
> My software is free, but my time generally not.
>
>

-- 
---------------------------------------+--------------------------------------
I set the clouds in motion - turn up   | Frank Warmerdam,
warmerdam at pobox.com
light and sound - activate the windows | +1 650-701-7823
<http://voice.google.com/calls?a=nc,%2B16507017823>
and watch the world go round - Rush    | Geospatial Software Developer
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/gdal-dev/attachments/20221120/0cf6a95f/attachment.htm>