fix(campaign): honor acceptance temp selection
This commit is contained in:
@@ -587,6 +587,12 @@ def _runtime_module_versions(registry: Any) -> dict[str, str]:
|
|||||||
return versions
|
return versions
|
||||||
|
|
||||||
|
|
||||||
|
def _create_runtime_root() -> Path:
|
||||||
|
"""Create the isolated runtime under the platform-selected temp root."""
|
||||||
|
|
||||||
|
return Path(tempfile.mkdtemp(prefix="govoplan-campaign-celery-redelivery-"))
|
||||||
|
|
||||||
|
|
||||||
def _bootstrap_and_run(
|
def _bootstrap_and_run(
|
||||||
*,
|
*,
|
||||||
settings: TestbedSettings,
|
settings: TestbedSettings,
|
||||||
@@ -594,9 +600,7 @@ def _bootstrap_and_run(
|
|||||||
redis_url: str,
|
redis_url: str,
|
||||||
visibility_timeout_seconds: int,
|
visibility_timeout_seconds: int,
|
||||||
) -> dict[str, Any]:
|
) -> dict[str, Any]:
|
||||||
runtime_root = Path(
|
runtime_root = _create_runtime_root()
|
||||||
tempfile.mkdtemp(prefix="govoplan-campaign-celery-redelivery-", dir="/tmp")
|
|
||||||
)
|
|
||||||
database = None
|
database = None
|
||||||
try:
|
try:
|
||||||
os.environ.update(
|
os.environ.update(
|
||||||
|
|||||||
@@ -172,6 +172,19 @@ def test_worker_bootstrap_uses_real_late_ack_solo_celery_worker() -> None:
|
|||||||
assert "send_email.run" not in source
|
assert "send_email.run" not in source
|
||||||
|
|
||||||
|
|
||||||
|
def test_runtime_root_uses_platform_temp_selection() -> None:
|
||||||
|
with mock.patch(
|
||||||
|
"govoplan_campaign_celery_redelivery_acceptance.tempfile.mkdtemp",
|
||||||
|
return_value="/selected-temp/govoplan-campaign-celery-redelivery-test",
|
||||||
|
) as mkdtemp:
|
||||||
|
runtime_root = runner._create_runtime_root()
|
||||||
|
|
||||||
|
assert runtime_root == Path(
|
||||||
|
"/selected-temp/govoplan-campaign-celery-redelivery-test"
|
||||||
|
)
|
||||||
|
mkdtemp.assert_called_once_with(prefix="govoplan-campaign-celery-redelivery-")
|
||||||
|
|
||||||
|
|
||||||
def test_worker_log_projection_matches_redelivered_task_without_retaining_id() -> None:
|
def test_worker_log_projection_matches_redelivered_task_without_retaining_id() -> None:
|
||||||
with tempfile.TemporaryDirectory() as temporary_directory:
|
with tempfile.TemporaryDirectory() as temporary_directory:
|
||||||
log_path = Path(temporary_directory) / "worker.log"
|
log_path = Path(temporary_directory) / "worker.log"
|
||||||
|
|||||||
Reference in New Issue
Block a user