kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #37643
Re: [PATCH] Make generic 'common' test and add UTF8 tests
-
To:
kicad-developers@xxxxxxxxxxxxxxxxxxx
-
From:
Wayne Stambaugh <stambaughw@xxxxxxxxx>
-
Date:
Wed, 26 Sep 2018 09:41:34 -0400
-
In-reply-to:
<CAG1r56+p+sA+_YsbsMXphyjXTjOivWT+Knac=+7x2+vD55G0zw@mail.gmail.com>
-
User-agent:
Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101 Thunderbird/60.0
John,
I merged your patches into the dev branch. Thank you for your contribution.
Cheers,
Wayne
On 9/25/2018 10:32 AM, John Beard wrote:
> Hi,
>
> These patches make a place for unit tests of stuff in /common under
> /qa. This has to move some existing stuff around a little bit:
>
> * /qa/common -> /qa/qa_utils
> * /qa/geometry is now part of the new /qa/common
>
> This makes a place to put libcommon tests and keeps /qa as an analogue
> of the main tree, just under /qa.
>
> Followed up by the first (non-geom) unit test of common code - UTF8,
> which replaces some very old and ad-hoc manual testing code in
> common/utf8.cpp with a proper Boost unit test.
>
> This should not change any aspect of production code, as all changes
> are to #if 0 code and /qa.
>
> The alternative to this method is to distribute tests into the various
> subdirs. In this case, creating /common/qa. This is how eeschema
> currently does it. It would be functionally the same to do it that
> way, just needs to move files around. `make test` would still work the
> same. Let me know if that way is preferred.
>
> Cheers,
>
> John
>
>
> _______________________________________________
> Mailing list: https://launchpad.net/~kicad-developers
> Post to : kicad-developers@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~kicad-developers
> More help : https://help.launchpad.net/ListHelp
>
References