mirror of
https://github.com/NixOS/nix
synced 2024-09-20 11:11:03 -04:00
a61e42adb5
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>
(cherry picked from commit 91b6833686
)
32 lines
688 B
Makefile
32 lines
688 B
Makefile
check: libutil-tests_RUN
|
|
|
|
programs += libutil-tests
|
|
|
|
libutil-tests_NAME = libnixutil-tests
|
|
|
|
libutil-tests_ENV := _NIX_TEST_UNIT_DATA=$(d)/data
|
|
|
|
libutil-tests_DIR := $(d)
|
|
|
|
ifeq ($(INSTALL_UNIT_TESTS), yes)
|
|
libutil-tests_INSTALL_DIR := $(checkbindir)
|
|
else
|
|
libutil-tests_INSTALL_DIR :=
|
|
endif
|
|
|
|
libutil-tests_SOURCES := $(wildcard $(d)/*.cc)
|
|
|
|
libutil-tests_EXTRA_INCLUDES = \
|
|
-I tests/unit/libutil-support \
|
|
-I src/libutil
|
|
|
|
libutil-tests_CXXFLAGS += $(libutil-tests_EXTRA_INCLUDES)
|
|
|
|
libutil-tests_LIBS = libutil-test-support libutil
|
|
|
|
libutil-tests_LDFLAGS := -lrapidcheck $(GTEST_LIBS)
|
|
|
|
check: $(d)/data/git/check-data.sh.test
|
|
|
|
$(eval $(call run-test,$(d)/data/git/check-data.sh))
|