Re: [PATCH 1/4] cmake: also build unit tests
- From
Johannes Schindelin <johannes.schindelin@gmx.de>
- Date
- Sep 18, 2023, 20:58 UTC
- Message-ID
- <4e13e2ce-9cca-eee4-788a-bf47fc29bc07@gmx.de>
- In-Reply-To
- <d7d1505c-8d65-4b64-8814-3e3b1e46f8ac@gmail.com>
Hi Phillip,
On Mon, 11 Sep 2023, Phillip Wood wrote:
Show 8 quoted lines
> On 31/08/2023 07:15, Johannes Schindelin via GitGitGadget wrote: > > From: Johannes Schindelin <johannes.schindelin@gmx.de> > > > > A new, better way to run unit tests was just added to Git. This adds > > support for building those unit tests via CMake. > > This patch builds the unit tests but does not add them to the list of tests > run by CTest - how are the tests typically run on the CMake build?
You're right, I missed that the unit tests are run as part not of t0080, but as a separate target in `t/Makefile`.
I've added a couple of patches to clean up the CTest part and then run the unit test (t-strbuf, and whatever is added to the `UNIT_TEST_PROGRAMS` variable in `Makefile`).
Thank you, Johannes