Switch the data directory lookup to use settings.SITE_DATA_DIR.

Review Request #13300 — Created Oct. 2, 2023 and submitted — Latest diff uploaded

Information

Review Board
release-6.x

Reviewers

The new get_data_dir() was returning $HOME, but we actaully already
had a better source for this. It now returns settings.SITE_DATA_DIR,
and also handles the case where this value changes, which can occur
during unit tests.

This fixes unit test failures with CertificateManager.

Unit tests pass.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Switch the data directory lookup to use settings.SITE_DATA_DIR.
The new `get_data_dir()` was returning `$HOME`, but we actaully already had a better source for this. It now returns `settings.SITE_DATA_DIR`, and also handles the case where this value changes, which can occur during unit tests. This fixes unit test failures with `CertificateManager`.
dedb0481e4b6f119a37426faabf1ebb9a1d77d87 Christian Hammond
reviewboard/admin/server.py
Loading...