You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "benbuckman (via GitHub)" <gi...@apache.org> on 2023/09/11 15:14:41 UTC

[GitHub] [airflow] benbuckman commented on pull request #32820: Support task mapping with setup teardown

benbuckman commented on PR #32820:
URL: https://github.com/apache/airflow/pull/32820#issuecomment-1714093966

   This PR appears to have unintentionally(?), or at least without being documented in the changelog, changed the way `dag.test()` handles exceptions.
   (See this line for example – https://github.com/apache/airflow/pull/32820/files#diff-62c8e300ee91e0d59f81e0ea5d30834f04db71ae74f2e155a10b51056b00b59bR2677 – `dag.test()` now returns a `DagRun` not `None`.)
   
   This might actually help with writing unit tests for exceptions in DAGs – https://github.com/apache/airflow/discussions/32831 – I'm exploring that now.
   
   Can this be added retroactively to the changelog for clarity?


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org