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

Head commit for run:
20719d839ef3dc6adc79b82933b92e9b6f828f05 / Jarek Potiuk <[email protected]>
Improve detection of tests vs. airflow code for internal API

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/10096017982

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to