Lee-W commented on code in PR #62463:
URL: https://github.com/apache/airflow/pull/62463#discussion_r2870919722
##########
airflow-core/tests/unit/timetables/test_trigger_timetable.py:
##########
@@ -657,29 +659,30 @@ def test_latest_run_with_run(dag_maker, session):
dag_maker.sync_dag_to_db()
session.commit()
dm = session.scalar(select(DagModel))
- assert dm.next_dagrun == start_date + datetime.timedelta(days=4)
+ assert dm.next_dagrun is None
Review Comment:
So the next_dagrun refers to next_dagrun_logical_date. Given that we already
have lots of next_dagrun_*, it might be confusing. Not in this PR, but We might
want to change it at some point 🤔
--
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]
