From 83e08799d6b55bc2119aff4f9749bad103b5f574 Mon Sep 17 00:00:00 2001 From: Daniel Schadt Date: Tue, 16 Apr 2024 22:48:23 +0200 Subject: add test for fietsupdate bootstrapping --- tests/bootstrap/test_new_instance.py | 70 ++++++++++++++++++++++++++++++++++++ 1 file changed, 70 insertions(+) create mode 100644 tests/bootstrap/test_new_instance.py (limited to 'tests/bootstrap') diff --git a/tests/bootstrap/test_new_instance.py b/tests/bootstrap/test_new_instance.py new file mode 100644 index 0000000..e136579 --- /dev/null +++ b/tests/bootstrap/test_new_instance.py @@ -0,0 +1,70 @@ +"""Test to ensure that bootstrapping a new instance works with the fietsupdate +script, as described in the documentation. +""" + +import contextlib +import logging +import os +import shutil +import subprocess +import venv +from pathlib import Path + +LOGGER = logging.getLogger(__name__) +REPO_BASE = Path(__file__).parent.parent.parent + + +@contextlib.contextmanager +def chdir(path: Path): + """Change path as context manager. + + Changes back to the previous path on ``__exit__``. + """ + cwd = Path.cwd() + os.chdir(path) + try: + yield + finally: + os.chdir(cwd) + + +def install_fietsboek(venv_path: Path) -> Path: + """Creates a virtual env in the given path and installs fietsboek. + + :param venv_path: Path in which the virtual environment should be created. + :return: The path to the ``bin`` directory in which all scripts reside. + """ + venv.create(venv_path, with_pip=True) + subprocess.check_call([venv_path / "bin" / "pip", "install", str(REPO_BASE)]) + return venv_path / "bin" + + +def test_setup_via_fietsupdate(tmpdir): + with chdir(tmpdir): + # We create a new temporary virtual environment with a fresh install, just + # to be sure there's as little interference as possible. + LOGGER.info("Installing Fietsboek into clean env") + binaries_path = install_fietsboek(tmpdir / "venv") + + LOGGER.info("Creating a test configuration") + shutil.copy(REPO_BASE / "testing.ini", "testing.ini") + Path("data").mkdir() + + # Try to run the migrations + subprocess.check_call( + [binaries_path / "fietsupdate", "update", "-c", "testing.ini", "-f"] + ) + + # Also try to add an administrator + subprocess.check_call([ + binaries_path / "fietsctl", + "user", + "add", + "-c", "testing.ini", + "--email", "foobar@example.com", + "--name", "Foo Bar", + "--password", "raboof", + "--admin", + ]) + + assert True -- cgit v1.2.3 From 7f526efeed7b807a3b56f465637bee5dc8a27e3d Mon Sep 17 00:00:00 2001 From: Daniel Schadt Date: Thu, 18 Apr 2024 22:28:55 +0200 Subject: have data_dir as a placeholder in testing.ini --- tests/bootstrap/test_new_instance.py | 15 +++++++++++++-- 1 file changed, 13 insertions(+), 2 deletions(-) (limited to 'tests/bootstrap') diff --git a/tests/bootstrap/test_new_instance.py b/tests/bootstrap/test_new_instance.py index e136579..dc3076e 100644 --- a/tests/bootstrap/test_new_instance.py +++ b/tests/bootstrap/test_new_instance.py @@ -39,6 +39,18 @@ def install_fietsboek(venv_path: Path) -> Path: return venv_path / "bin" +def create_config(config_name: Path): + """Copies the testing.ini config and fills in the placeholders. + + :param config_name: Path to the resulting config. + """ + shutil.copy(REPO_BASE / "testing.ini", config_name) + config = config_name.read_text() + config = config.replace("# %% fietsboek.data_dir %%", "fietsboek.data_dir = %(here)s/data") + config_name.write_text(config) + Path("data").mkdir() + + def test_setup_via_fietsupdate(tmpdir): with chdir(tmpdir): # We create a new temporary virtual environment with a fresh install, just @@ -47,8 +59,7 @@ def test_setup_via_fietsupdate(tmpdir): binaries_path = install_fietsboek(tmpdir / "venv") LOGGER.info("Creating a test configuration") - shutil.copy(REPO_BASE / "testing.ini", "testing.ini") - Path("data").mkdir() + create_config(Path("testing.ini")) # Try to run the migrations subprocess.check_call( -- cgit v1.2.3