123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118 |
- from collections.abc import MutableMapping
- import psutil
- import pytest
- import responses
- from django.db import connections
- from sentry.silo import SiloMode
- from sentry.testutils.pytest.sentry import get_default_silo_mode_for_test_cases
- pytest_plugins = ["sentry.testutils.pytest"]
- # XXX: The below code is vendored code from https://github.com/utgwkk/pytest-github-actions-annotate-failures
- # so that we can add support for pytest_rerunfailures
- # retried tests will no longer be annotated in GHA
- #
- # Reference:
- # https://docs.pytest.org/en/latest/writing_plugins.html#hookwrapper-executing-around-other-hooks
- # https://docs.pytest.org/en/latest/writing_plugins.html#hook-function-ordering-call-example
- # https://docs.pytest.org/en/stable/reference.html#pytest.hookspec.pytest_runtest_makereport
- #
- # Inspired by:
- # https://github.com/pytest-dev/pytest/blob/master/src/_pytest/terminal.py
- @pytest.fixture(autouse=True)
- def unclosed_files():
- fds = frozenset(psutil.Process().open_files())
- yield
- assert frozenset(psutil.Process().open_files()) == fds
- @pytest.fixture(autouse=True)
- def validate_silo_mode():
- # NOTE! Hybrid cloud uses many mechanisms to simulate multiple different configurations of the application
- # during tests. It depends upon `override_settings` using the correct contextmanager behaviors and correct
- # thread handling in acceptance tests. If you hit one of these, it's possible either that cleanup logic has
- # a bug, or you may be using a contextmanager incorrectly. Let us know and we can help!
- expected = get_default_silo_mode_for_test_cases()
- message = (
- f"Possible test leak bug! SiloMode was not reset to {expected} between tests. "
- "Please read the comment for validate_silo_mode() in tests/conftest.py."
- )
- if SiloMode.get_current_mode() != expected:
- raise Exception(message)
- yield
- if SiloMode.get_current_mode() != expected:
- raise Exception(message)
- @pytest.fixture(autouse=True)
- def setup_simulate_on_commit(request):
- from sentry.testutils.hybrid_cloud import simulate_on_commit
- with simulate_on_commit(request):
- yield
- @pytest.fixture(autouse=True)
- def setup_enforce_monotonic_transactions(request):
- from sentry.testutils.hybrid_cloud import enforce_no_cross_transaction_interactions
- with enforce_no_cross_transaction_interactions():
- yield
- @pytest.fixture(autouse=True)
- def audit_hybrid_cloud_writes_and_deletes(request):
- """
- Ensure that write operations on hybrid cloud foreign keys are recorded
- alongside outboxes or use a context manager to indicate that the
- caller has considered outbox and didn't accidentally forget.
- Generally you can avoid assertion errors from these checks by:
- 1. Running deletion/write logic within an `outbox_context`.
- 2. Using Model.delete()/save methods that create outbox messages in the
- same transaction as a delete operation.
- Scenarios that are generally always unsafe are using
- `QuerySet.delete()`, `QuerySet.update()` or raw SQL to perform
- writes.
- The User.delete() method is a good example of how to safely
- delete records and generate outbox messages.
- """
- from sentry.testutils.silo import validate_protected_queries
- debug_cursor_state: MutableMapping[str, bool] = {}
- for conn in connections.all():
- debug_cursor_state[conn.alias] = conn.force_debug_cursor
- conn.queries_log.clear()
- conn.force_debug_cursor = True
- try:
- yield
- finally:
- for conn in connections.all():
- conn.force_debug_cursor = debug_cursor_state[conn.alias]
- validate_protected_queries(conn.queries)
- @pytest.fixture(autouse=True)
- def check_leaked_responses_mocks():
- yield
- leaked = responses.registered()
- if leaked:
- responses.reset()
- leaked_s = "".join(f"- {item}\n" for item in leaked)
- raise AssertionError(
- f"`responses` were leaked outside of the test context:\n{leaked_s}"
- f"(make sure to use `@responses.activate` or `with responses.mock:`)"
- )
|