amoghrajesh commented on code in PR #53020:
URL: https://github.com/apache/airflow/pull/53020#discussion_r2195265500
##########
task-sdk/tests/task_sdk/execution_time/test_task_runner.py:
##########
@@ -1402,6 +1406,7 @@ def mock_send_side_effect(*args, **kwargs):
mock_supervisor_comms.send.side_effect = mock_send_side_effect
run(runtime_ti, context=runtime_ti.get_template_context(),
log=mock.MagicMock())
+ spy_agency.assert_spy_called_with(deserialize, ser_value)
Review Comment:
Yeah, i didn't like the test either, pushed a better test here: dc87132d78
It only tests a minimal case if Xcom.get_all is called or not and if
deserialized is called or not :)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
