nix-super/tests/unit/libstore-support/local.mk

libraries += libstore-test-support

libstore-test-support_NAME = libnixstore-test-support

libstore-test-support_DIR := $(d)

ifeq ($(INSTALL_UNIT_TESTS), yes)
  libstore-test-support_INSTALL_DIR := $(checklibdir)
else
  libstore-test-support_INSTALL_DIR :=
endif

libstore-test-support_SOURCES := $(wildcard $(d)/tests/*.cc)

libstore-test-support_CXXFLAGS += $(libstore-tests_EXTRA_INCLUDES)

libstore-test-support_LIBS = \
    libutil-test-support \
    libstore libutil

libstore-test-support_LDFLAGS := -pthread -lrapidcheck
Move tests to separate directories, and document Today, with the tests inside a `tests` intermingled with the corresponding library's source code, we have a few problems: - We have to be careful that wildcards don't end up with tests being built as part of Nix proper, or test headers being installed as part of Nix proper. - Tests in libraries but not executables is not right: - It means each executable runs the previous unit tests again, because it needs the libraries. - It doesn't work right on Windows, which doesn't want you to load a DLL just for the side global variable . It could be made to work with the dlopen equivalent, but that's gross! This reorg solves these problems. There is a remaining problem which is that sibbling headers (like `hash.hh` the test header vs `hash.hh` the main `libnixutil` header) end up shadowing each other. This PR doesn't solve that. That is left as future work for a future PR. Co-authored-by: Valentin Gagarin <valentin.gagarin@tweag.io> 2023-08-25 17:20:28 +03:00			`libraries += libstore-test-support`

			`libstore-test-support_NAME = libnixstore-test-support`

			`libstore-test-support_DIR := $(d)`

			`ifeq ($(INSTALL_UNIT_TESTS), yes)`
			`libstore-test-support_INSTALL_DIR := $(checklibdir)`
			`else`
			`libstore-test-support_INSTALL_DIR :=`
			`endif`

			`libstore-test-support_SOURCES := $(wildcard $(d)/tests/*.cc)`

			`libstore-test-support_CXXFLAGS += $(libstore-tests_EXTRA_INCLUDES)`

			`libstore-test-support_LIBS = \`
			`libutil-test-support \`
			`libstore libutil`

Add missing `-pthread` for test support libraries This is good in general (see how the other libraries also have long had it, since 49fe9592a47e7819179c2de4fd6068e897e944c7) but in particular needed to fix the NetBSD build. 2023-12-06 01:10:37 +02:00			`libstore-test-support_LDFLAGS := -pthread -lrapidcheck`