The GitHub Actions job "Tests" on airflow.git has failed.
Run started by GitHub user potiuk (triggered by potiuk).

Head commit for run:
ac465e4f10521a128b7bee63686dc55a2534db4a / Jarek Potiuk <ja...@potiuk.com>
Improve detection of tests vs. airflow code for internal API (#41030)

In case of database isolation, some of our tests are running
a number of fixtures that allow for example to create dag runs
as setup code - this code, however sometimes runs methods that
are used for internal_api calls and those calls are not needed
to be run via internal_api.

This PR adds capability of detecting such case - by checking if
any of the "callers" of the internal_api are `conftest.py` which
means that this is a test fixture - and in this case, direct
method call is used rather than internal API call.

This way we better separate test code from "airflow" code in
DB isolation tests - and internal API is only used by the tested
methods and not the test code that manages setup/teardown

Also orm_create_dagrun which should only be used in scheduler is
unmarked as "internal_api" method - it is heavily used in the
fixtures, but neither DAGFileProcessor, Triggerer nor Worker
should create new DAGRuns.

Report URL: https://github.com/apache/airflow/actions/runs/10097016511

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: jobs-unsubscr...@airflow.apache.org
For additional commands, e-mail: jobs-h...@airflow.apache.org

Reply via email to