skeema / skeema / 11749537470

92%

push

github

tests: use tmpfs for containers, with env var for configuring cleanup

This commit adjusts test containers to use tmpfs for the data directory
whenever possible. This reduces the amount of time required for running
integration tests.

By default, test containers are now removed after integration test
completion for each package. This behavior is configurable though the new
SKEEMA_TEST_CLEANUP env var, which optionally can be set to either "none"
(keep containers running) or "stop" (halt but do not remove containers) if
needed. GitHub Actions CI now sets this to "none" so that the same containers
can be re-used across subpackage integration tests.

This commit also modifies two integration tests that previously stopped and
re-started test containers as part of their test logic; this is undesirable
with tmpfs since it inherently requires re-initializing the test database
server. In both cases, indirect code coverage for the underlying logic already
existed elsewhere, and the logic being tested rarely changes anyway.

16 of 22 new or added lines in 3 files covered. (72.73%)

10071 of 10804 relevant lines covered (93.22%)

1.11 hits per line