async def test_deleter_run(config, mocker):
"""Test the Deleter does the work the deleter should do."""
logger_mock = mocker.MagicMock()
p = Deleter(config, logger_mock)
p._do_work = AsyncMock()
await p.run()
p._do_work.assert_called()
async def test_deleter_run_exception(config, mocker):
"""Test an error doesn't kill the Deleter."""
logger_mock = mocker.MagicMock()
p = Deleter(config, logger_mock)
p.last_work_end_timestamp = None
p._do_work = AsyncMock()
p._do_work.side_effect = [Exception("bad thing happen!")]
await p.run()
p._do_work.assert_called()
assert p.last_work_end_timestamp
async def test_deleter_run_exception(config, mocker):
"""Test an error doesn't kill the Deleter."""
logger_mock = mocker.MagicMock()
dw_mock = mocker.patch(
"lta.deleter.Deleter._do_work") # , new_callable=AsyncMock)
dw_mock.side_effect = [Exception("bad thing happen!")]
p = Deleter(config, logger_mock)
p.last_work_end_timestamp = None
await p.run()
assert p.last_work_end_timestamp
dw_mock.assert_called()
async def test_deleter_quarantine_bundle_with_reason(config, mocker):
"""Test that _do_work_claim attempts to quarantine a Bundle that fails to get deleted."""
logger_mock = mocker.MagicMock()
lta_rc_mock = mocker.patch("rest_tools.client.RestClient", new_callable=AsyncMock)
p = Deleter(config, logger_mock)
await p._quarantine_bundle(lta_rc_mock, {"uuid": "c4b345e4-2395-4f9e-b0eb-9cc1c9cdf003"}, "Rucio caught fire, then we roasted marshmellows.")
lta_rc_mock.request.assert_called_with("PATCH", "/Bundles/c4b345e4-2395-4f9e-b0eb-9cc1c9cdf003", mocker.ANY)
async def test_deleter_delete_bundle_raises(config, mocker):
"""Test that _do_work_claim both calls _quarantine_bundle and re-raises when _delete_bundle raises."""
logger_mock = mocker.MagicMock()
lta_rc_mock = mocker.patch("rest_tools.client.RestClient.request",
new_callable=AsyncMock)
lta_rc_mock.return_value = {
"bundle": {
"one": 1,
},
}
db_mock = mocker.patch("lta.deleter.Deleter._delete_bundle",
new_callable=AsyncMock)
qb_mock = mocker.patch("lta.deleter.Deleter._quarantine_bundle",
new_callable=AsyncMock)
db_mock.side_effect = Exception(
"LTA DB unavailable; currently safer at home")
p = Deleter(config, logger_mock)
with pytest.raises(Exception):
await p._do_work_claim()
lta_rc_mock.assert_called_with(
"POST", '/Bundles/actions/pop?source=WIPAC&dest=NERSC&status=detached',
{'claimant': f'{p.name}-{p.instance_uuid}'})
db_mock.assert_called_with(mocker.ANY, {"one": 1})
qb_mock.assert_called_with(mocker.ANY, {"one": 1},
"LTA DB unavailable; currently safer at home")
async def test_deleter_do_work_yes_results(config, mocker):
"""Test that _do_work keeps working until the LTA DB has no work."""
logger_mock = mocker.MagicMock()
dwc_mock = mocker.patch("lta.deleter.Deleter._do_work_claim", new_callable=AsyncMock)
dwc_mock.side_effect = [True, True, False]
p = Deleter(config, logger_mock)
await p._do_work()
dwc_mock.assert_called()
async def test_deleter_do_work_no_results(config, mocker):
"""Test that _do_work goes on vacation when the LTA DB has no work."""
logger_mock = mocker.MagicMock()
dwc_mock = mocker.patch("lta.deleter.Deleter._do_work_claim", new_callable=AsyncMock)
dwc_mock.return_value = False
p = Deleter(config, logger_mock)
await p._do_work()
dwc_mock.assert_called()
async def test_deleter_do_work_pop_exception(config, mocker):
"""Test that _do_work raises when the RestClient can't pop."""
logger_mock = mocker.MagicMock()
lta_rc_mock = mocker.patch("rest_tools.client.RestClient.request", new_callable=AsyncMock)
lta_rc_mock.side_effect = HTTPError(500, "LTA DB on fire. Again.")
p = Deleter(config, logger_mock)
with pytest.raises(HTTPError):
await p._do_work()
lta_rc_mock.assert_called_with("POST", '/Bundles/actions/pop?source=WIPAC&dest=NERSC&status=detached', {'claimant': f'{p.name}-{p.instance_uuid}'})
async def test_deleter_delete_bundle(config, mocker):
"""Test that _delete_bundle attempts to delete a Bundle."""
logger_mock = mocker.MagicMock()
lta_rc_mock = mocker.patch("rest_tools.client.RestClient", new_callable=AsyncMock)
remove_mock = mocker.patch("os.remove", new_callable=MagicMock)
p = Deleter(config, logger_mock)
await p._delete_bundle(lta_rc_mock, {
"uuid": "c4b345e4-2395-4f9e-b0eb-9cc1c9cdf003",
"bundle_path": "/icecube/datawarehouse/path/to/c4b345e4-2395-4f9e-b0eb-9cc1c9cdf003.zip",
})
remove_mock.assert_called()
lta_rc_mock.request.assert_called_with("PATCH", "/Bundles/c4b345e4-2395-4f9e-b0eb-9cc1c9cdf003", mocker.ANY)
async def test_deleter_do_work_claim_no_result(config, mocker):
"""Test that _do_work_claim does not work when the LTA DB has no work."""
logger_mock = mocker.MagicMock()
lta_rc_mock = mocker.patch("rest_tools.client.RestClient.request", new_callable=AsyncMock)
lta_rc_mock.return_value = {
"bundle": None
}
db_mock = mocker.patch("lta.deleter.Deleter._delete_bundle", new_callable=AsyncMock)
p = Deleter(config, logger_mock)
await p._do_work_claim()
lta_rc_mock.assert_called_with("POST", '/Bundles/actions/pop?source=WIPAC&dest=NERSC&status=detached', {'claimant': f'{p.name}-{p.instance_uuid}'})
db_mock.assert_not_called()
async def test_deleter_do_work_claim_yes_result(config, mocker):
"""Test that _do_work_claim processes the Bundle that it gets from the LTA DB."""
logger_mock = mocker.MagicMock()
lta_rc_mock = mocker.patch("rest_tools.client.RestClient.request", new_callable=AsyncMock)
lta_rc_mock.return_value = {
"bundle": {
"one": 1,
},
}
db_mock = mocker.patch("lta.deleter.Deleter._delete_bundle", new_callable=AsyncMock)
p = Deleter(config, logger_mock)
assert await p._do_work_claim()
lta_rc_mock.assert_called_with("POST", '/Bundles/actions/pop?source=WIPAC&dest=NERSC&status=detached', {'claimant': f'{p.name}-{p.instance_uuid}'})
db_mock.assert_called_with(mocker.ANY, {"one": 1})
def test_constructor_config(config, mocker):
"""Test that a Deleter can be constructed with a configuration object and a logging object."""
logger_mock = mocker.MagicMock()
p = Deleter(config, logger_mock)
assert p.name == "testing-deleter"
assert p.dest_site == "NERSC"
assert p.disk_base_path == "/path/to/rucio/rse/root"
assert p.heartbeat_patch_retries == 3
assert p.heartbeat_patch_timeout_seconds == 30
assert p.heartbeat_sleep_duration_seconds == 60
assert p.input_status == "detached"
assert p.lta_rest_token == "fake-lta-rest-token"
assert p.lta_rest_url == "http://RmMNHdPhHpH2ZxfaFAC9d2jiIbf5pZiHDqy43rFLQiM.com/"
assert p.output_status == "source-deleted"
assert p.source_site == "WIPAC"
assert p.work_retries == 3
assert p.work_sleep_duration_seconds == 60
assert p.work_timeout_seconds == 30
assert p.logger == logger_mock
async def test_deleter_logs_configuration(mocker):
"""Test to make sure the Deleter logs its configuration."""
logger_mock = mocker.MagicMock()
deleter_config = {
"COMPONENT_NAME": "logme-testing-deleter",
"DEST_SITE": "NERSC",
"DISK_BASE_PATH": "/path/to/rucio/rse/root",
"HEARTBEAT_PATCH_RETRIES": "1",
"HEARTBEAT_PATCH_TIMEOUT_SECONDS": "20",
"HEARTBEAT_SLEEP_DURATION_SECONDS": "30",
"INPUT_STATUS": "detached",
"LTA_REST_TOKEN": "logme-fake-lta-rest-token",
"LTA_REST_URL":
"logme-http://zjwdm5ggeEgS1tZDZy9l1DOZU53uiSO4Urmyb8xL0.com/",
"OUTPUT_STATUS": "source-deleted",
"RUN_ONCE_AND_DIE": "False",
"SOURCE_SITE": "WIPAC",
"WORK_RETRIES": "5",
"WORK_SLEEP_DURATION_SECONDS": "70",
"WORK_TIMEOUT_SECONDS": "90",
}
Deleter(deleter_config, logger_mock)
EXPECTED_LOGGER_CALLS = [
call("deleter 'logme-testing-deleter' is configured:"),
call('COMPONENT_NAME = logme-testing-deleter'),
call('DEST_SITE = NERSC'),
call('DISK_BASE_PATH = /path/to/rucio/rse/root'),
call('HEARTBEAT_PATCH_RETRIES = 1'),
call('HEARTBEAT_PATCH_TIMEOUT_SECONDS = 20'),
call('HEARTBEAT_SLEEP_DURATION_SECONDS = 30'),
call('INPUT_STATUS = detached'),
call('LTA_REST_TOKEN = logme-fake-lta-rest-token'),
call(
'LTA_REST_URL = logme-http://zjwdm5ggeEgS1tZDZy9l1DOZU53uiSO4Urmyb8xL0.com/'
),
call('OUTPUT_STATUS = source-deleted'),
call('RUN_ONCE_AND_DIE = False'),
call('SOURCE_SITE = WIPAC'),
call('WORK_RETRIES = 5'),
call('WORK_SLEEP_DURATION_SECONDS = 70'),
call('WORK_TIMEOUT_SECONDS = 90')
]
logger_mock.info.assert_has_calls(EXPECTED_LOGGER_CALLS)
def test_do_status(config, mocker):
"""Verify that the Deleter has no additional state to offer."""
logger_mock = mocker.MagicMock()
p = Deleter(config, logger_mock)
assert p._do_status() == {}