Skip to content

Commit 36eba18

Browse files
authored
Fix Databricks trigger test (#34272)
1 parent d3ce442 commit 36eba18

File tree

1 file changed

+22
-23
lines changed

1 file changed

+22
-23
lines changed

tests/providers/databricks/triggers/test_databricks.py

Lines changed: 22 additions & 23 deletions
Original file line numberDiff line numberDiff line change
@@ -107,23 +107,22 @@ async def test_run_return_success(self, mock_get_run_state, mock_get_run_page_ur
107107
result_state="SUCCESS",
108108
)
109109

110-
generator = self.trigger.run()
111-
actual = await generator.asend(None)
112-
assert actual == TriggerEvent(
113-
{
114-
"run_id": RUN_ID,
115-
"run_state": RunState(
116-
life_cycle_state=LIFE_CYCLE_STATE_TERMINATED, state_message="", result_state="SUCCESS"
117-
).to_json(),
118-
"run_page_url": RUN_PAGE_URL,
119-
}
120-
)
110+
trigger_event = self.trigger.run()
111+
async for event in trigger_event:
112+
assert event == TriggerEvent(
113+
{
114+
"run_id": RUN_ID,
115+
"run_state": RunState(
116+
life_cycle_state=LIFE_CYCLE_STATE_TERMINATED, state_message="", result_state="SUCCESS"
117+
).to_json(),
118+
"run_page_url": RUN_PAGE_URL,
119+
}
120+
)
121121

122122
@pytest.mark.asyncio
123123
@mock.patch("airflow.providers.databricks.triggers.databricks.asyncio.sleep")
124124
@mock.patch("airflow.providers.databricks.hooks.databricks.DatabricksHook.a_get_run_state")
125125
async def test_sleep_between_retries(self, mock_get_run_state, mock_sleep):
126-
127126
mock_get_run_state.side_effect = [
128127
RunState(
129128
life_cycle_state=LIFE_CYCLE_STATE_PENDING,
@@ -137,16 +136,16 @@ async def test_sleep_between_retries(self, mock_get_run_state, mock_sleep):
137136
),
138137
]
139138

140-
generator = self.trigger.run()
141-
actual = await generator.asend(None)
142-
assert actual == TriggerEvent(
143-
{
144-
"run_id": RUN_ID,
145-
"run_state": RunState(
146-
life_cycle_state=LIFE_CYCLE_STATE_TERMINATED, state_message="", result_state="SUCCESS"
147-
).to_json(),
148-
"run_page_url": RUN_PAGE_URL,
149-
}
150-
)
139+
trigger_event = self.trigger.run()
140+
async for event in trigger_event:
141+
assert event == TriggerEvent(
142+
{
143+
"run_id": RUN_ID,
144+
"run_state": RunState(
145+
life_cycle_state=LIFE_CYCLE_STATE_TERMINATED, state_message="", result_state="SUCCESS"
146+
).to_json(),
147+
"run_page_url": RUN_PAGE_URL,
148+
}
149+
)
151150
mock_sleep.assert_called_once()
152151
mock_sleep.assert_called_with(POLLING_INTERVAL_SECONDS)

0 commit comments

Comments
 (0)