Skip to content

Commit

Permalink
Merge pull request #77 from acsone/master-distributed-run
Browse files Browse the repository at this point in the history
Add support for distributed run
  • Loading branch information
yvaucher authored Oct 7, 2024
2 parents b4e05a5 + 0168a0e commit 6e925f3
Show file tree
Hide file tree
Showing 2 changed files with 28 additions and 1 deletion.
2 changes: 2 additions & 0 deletions README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -51,3 +51,5 @@ You can use the ``ODOO_RC`` environment variable using an odoo configuration fil
export ODOO_RC=/path/to/odoo/config.cfg
pytest ...

The plugin is also compatible with distributed run provided by the `pytest-xdist <https://pypi.org/project/pytest-xdist/>`_ library. When tests are distributed, a copy of the database is created for each worker at the start of the test session.
This is useful to avoid concurrent access to the same database, which can lead to deadlocks. The provided database is therefore used only as template. At the end of the tests, all the created databases are dropped.
27 changes: 26 additions & 1 deletion pytest_odoo.py
Original file line number Diff line number Diff line change
Expand Up @@ -8,6 +8,7 @@
import os
import signal
import threading
from contextlib import contextmanager
from pathlib import Path
from typing import Optional

Expand Down Expand Up @@ -110,6 +111,28 @@ def load_http(request):
signal.signal(signal.SIGINT, signal.default_int_handler)


@contextmanager
def _worker_db_name():
# This method ensure that if tests are ran in a distributed way
# thanks to the use of pytest-xdist addon, each worker will use
# a specific copy of the initial database to run their tests.
# In this way we prevent deadlock errors.
xdist_worker = os.getenv("PYTEST_XDIST_WORKER")
original_db_name = db_name = odoo.tests.common.get_db_name()
try:
if xdist_worker:
db_name = f"{original_db_name}-{xdist_worker}"
os.system(f"dropdb {db_name} --if-exists")
os.system(f"createdb -T {original_db_name} {db_name}")
odoo.tools.config["db_name"] = db_name
yield db_name
finally:
if db_name != original_db_name:
odoo.sql_db.close_db(db_name)
os.system(f"dropdb {db_name}")
odoo.tools.config["db_name"] = original_db_name


@pytest.fixture(scope='session', autouse=True)
def load_registry():
# Initialize the registry before running tests.
Expand All @@ -121,7 +144,9 @@ def load_registry():
# Finally we enable `testing` flag on current thread
# since Odoo sets it when loading test suites.
threading.current_thread().testing = True
odoo.registry(odoo.tests.common.get_db_name())
with _worker_db_name() as db_name:
odoo.registry(db_name)
yield


@pytest.fixture(scope='module', autouse=True)
Expand Down

0 comments on commit 6e925f3

Please sign in to comment.