tests: Clean the environment before running tests.

As we start reading more sensitive user configuration (like RT credentials),
this fixture makes it simpler to ensure that the tests never read the user's
real configuration, which could lead to problems like passwords in tracebacks.
This commit is contained in:
Brett Smith 2020-03-23 12:01:02 -04:00
parent d845332c7e
commit 5140ca64f6
2 changed files with 11 additions and 3 deletions

9
tests/conftest.py Normal file
View file

@ -0,0 +1,9 @@
import os
import pytest
from . import testutil
@pytest.fixture(scope='session', autouse=True)
def clean_environment():
os.environ.pop('CONSERVANCY_REPOSITORY', None)

View file

@ -39,6 +39,5 @@ def test_repository_from_environment():
assert config.repository_path().parts == ('/', 'usr', 'local')
def test_no_repository():
with update_environ(CONSERVANCY_REPOSITORY=None):
config = config_mod.Config()
assert config.repository_path() is None
config = config_mod.Config()
assert config.repository_path() is None