]> git.bitcoin.ninja Git - rust-lightning/commit
Use `env::temp_dir()` for BP tests
authorDuncan Dean <duncangleeddean@gmail.com>
Wed, 26 Apr 2023 12:57:18 +0000 (14:57 +0200)
committerDuncan Dean <duncangleeddean@gmail.com>
Wed, 26 Apr 2023 12:57:18 +0000 (14:57 +0200)
commit4abf3ba206a72612f46d9f0901be714b9007da6d
tree85bc5ecdd697eebe6f2dbcbda859cc59dbe76742
parentc1825672ed27839e95bb42bf8e75c9068da585d8
Use `env::temp_dir()` for BP tests

Currently `BackgroundProcessor` tests create persister directories in the
current working directory and rely on cleaning up in a `Drop` implementation.

Unfortunately, it seems that in the async tests that nodes are not
`drop()`ed for some reason and so the directories created by those
tests remain behind in the current working directory.

This commit at least ensures that these test directories are created in
a temporary location for the OS using `temp_dir()`. It doesn't aim to
solve the lack of cleanup in the async tests.

Partial fix for #2224 but I believe it's enough to resolve it as these
temp directories that do remain will be purged by the OS at some stage
and are overwritten by subsequent tests if there is a conflict.
lightning-background-processor/src/lib.rs