Skip to content

Move test of DagRun.update_state to better place#42845

Merged
dstandish merged 2 commits intoapache:mainfrom
astronomer:aip-78-small-test-refactor
Oct 9, 2024
Merged

Move test of DagRun.update_state to better place#42845
dstandish merged 2 commits intoapache:mainfrom
astronomer:aip-78-small-test-refactor

Conversation

@dstandish
Copy link
Contributor

Previously this lived in test_scheduler_job.py

It only really tested the behavior of DagRun.update_state.

As far as I can tell, it checks that if you null out the state on a TI of a finished dag, and then you call update_state, then the DR will be set to running.

@boring-cyborg boring-cyborg bot added the area:Scheduler including HA (high availability) scheduler label Oct 9, 2024
Previously this lived in test_scheduler_job.py

It only really tested the behavior of DagRun.update_state.

As far as I can tell, it checks that if you null out the state on a TI of a finished dag, and then you call ``update_state``, then the DR will be set to running.
@dstandish dstandish force-pushed the aip-78-small-test-refactor branch from 94e9870 to 244e8b3 Compare October 9, 2024 13:42
@dstandish dstandish merged commit 26d7919 into apache:main Oct 9, 2024
@dstandish dstandish deleted the aip-78-small-test-refactor branch October 9, 2024 15:15
kunaljubce pushed a commit to kunaljubce/airflow that referenced this pull request Oct 13, 2024
Previously this lived in test_scheduler_job.py

It only really tested the behavior of DagRun.update_state.

As far as I can tell, it checks that if you null out the state on a TI of a finished dag, and then you call ``update_state``, then the DR will be set to running.
pavansharma36 pushed a commit to pavansharma36/airflow that referenced this pull request Oct 14, 2024
Previously this lived in test_scheduler_job.py

It only really tested the behavior of DagRun.update_state.

As far as I can tell, it checks that if you null out the state on a TI of a finished dag, and then you call ``update_state``, then the DR will be set to running.
R7L208 pushed a commit to R7L208/airflow that referenced this pull request Oct 17, 2024
Previously this lived in test_scheduler_job.py

It only really tested the behavior of DagRun.update_state.

As far as I can tell, it checks that if you null out the state on a TI of a finished dag, and then you call ``update_state``, then the DR will be set to running.
harjeevanmaan pushed a commit to harjeevanmaan/airflow that referenced this pull request Oct 23, 2024
Previously this lived in test_scheduler_job.py

It only really tested the behavior of DagRun.update_state.

As far as I can tell, it checks that if you null out the state on a TI of a finished dag, and then you call ``update_state``, then the DR will be set to running.
PaulKobow7536 pushed a commit to PaulKobow7536/airflow that referenced this pull request Oct 24, 2024
Previously this lived in test_scheduler_job.py

It only really tested the behavior of DagRun.update_state.

As far as I can tell, it checks that if you null out the state on a TI of a finished dag, and then you call ``update_state``, then the DR will be set to running.
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
Previously this lived in test_scheduler_job.py

It only really tested the behavior of DagRun.update_state.

As far as I can tell, it checks that if you null out the state on a TI of a finished dag, and then you call ``update_state``, then the DR will be set to running.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:Scheduler including HA (high availability) scheduler

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments