Move more test code to test/
Submitted by Simon McVittie
Assigned to Simon McVittie
Link to original bug (#108178)
Description
dbus currently has too much test code in bus/ and dbus/. This is awkward for several reasons:
- we need to compile the test binaries alongside the rest of bus/ and dbus/, but then run them from test/
- the test code gets counted as part of bus/ and dbus/ when computing coverage statistics, distorting our ideas about the coverage of the actual production code
- the "embedded tests" should not be compiled into libdbus and dbus-daemon when building secure, efficient production dbus binaries, but then we lose test coverage by disabling them
- the test framework used for the "embedded tests" makes it hard to skip tests that take forever and might not even be relevant to the bug/feature you're actually working on
This branch attempts to clean that up, removing something like half of the test code from dbus/, along with all the test executables.
Because I'm experimenting with freedesktop.org Gitlab ahead of dbus' migration, here's a merge request to review: smcv/dbus-temp!2
(That's in a temporary clone of dbus, which I'll archive when we have the real dbus/dbus.git in Gitlab, and replace with a proper fork in smcv/dbus.)
Version: git master