sameerpandey17 commented on code in PR #74973:
URL: https://github.com/apache/airflow/pull/74973#discussion_r4308598682
##########
task-sdk/tests/task_sdk/bases/test_notifier.py:
##########
@@ -97,3 +97,35 @@ def test_notifier_call_with_prepared_context(self, caplog):
}
)
assert notifier.message == "task: some_task"
+
+ def test_render_template_fields_with_dag_without_get_template_env(self):
+ """Notifier must not crash when context['dag'] has no get_template_env
(e.g. SerializedDAG)."""
+
+ class _FakeDag:
Review Comment:
Switched both tests to a real `SerializedDAG` via `create_scheduler_dag` and
dropped the stand-in class (3bfb401)
--
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]
