Skip to content

Commit

Permalink
Fix Databricks trigger test (#34272)
Browse files Browse the repository at this point in the history
  • Loading branch information
pankajastro committed Sep 11, 2023
1 parent d3ce442 commit 36eba18
Showing 1 changed file with 22 additions and 23 deletions.
45 changes: 22 additions & 23 deletions tests/providers/databricks/triggers/test_databricks.py
Expand Up @@ -107,23 +107,22 @@ async def test_run_return_success(self, mock_get_run_state, mock_get_run_page_ur
result_state="SUCCESS",
)

generator = self.trigger.run()
actual = await generator.asend(None)
assert actual == TriggerEvent(
{
"run_id": RUN_ID,
"run_state": RunState(
life_cycle_state=LIFE_CYCLE_STATE_TERMINATED, state_message="", result_state="SUCCESS"
).to_json(),
"run_page_url": RUN_PAGE_URL,
}
)
trigger_event = self.trigger.run()
async for event in trigger_event:
assert event == TriggerEvent(
{
"run_id": RUN_ID,
"run_state": RunState(
life_cycle_state=LIFE_CYCLE_STATE_TERMINATED, state_message="", result_state="SUCCESS"
).to_json(),
"run_page_url": RUN_PAGE_URL,
}
)

@pytest.mark.asyncio
@mock.patch("airflow.providers.databricks.triggers.databricks.asyncio.sleep")
@mock.patch("airflow.providers.databricks.hooks.databricks.DatabricksHook.a_get_run_state")
async def test_sleep_between_retries(self, mock_get_run_state, mock_sleep):

mock_get_run_state.side_effect = [
RunState(
life_cycle_state=LIFE_CYCLE_STATE_PENDING,
Expand All @@ -137,16 +136,16 @@ async def test_sleep_between_retries(self, mock_get_run_state, mock_sleep):
),
]

generator = self.trigger.run()
actual = await generator.asend(None)
assert actual == TriggerEvent(
{
"run_id": RUN_ID,
"run_state": RunState(
life_cycle_state=LIFE_CYCLE_STATE_TERMINATED, state_message="", result_state="SUCCESS"
).to_json(),
"run_page_url": RUN_PAGE_URL,
}
)
trigger_event = self.trigger.run()
async for event in trigger_event:
assert event == TriggerEvent(
{
"run_id": RUN_ID,
"run_state": RunState(
life_cycle_state=LIFE_CYCLE_STATE_TERMINATED, state_message="", result_state="SUCCESS"
).to_json(),
"run_page_url": RUN_PAGE_URL,
}
)
mock_sleep.assert_called_once()
mock_sleep.assert_called_with(POLLING_INTERVAL_SECONDS)

0 comments on commit 36eba18

Please sign in to comment.