The GitHub Actions job "Tests AMD" on airflow.git/fix-pluginmgr-and-asset-test-leaking-state has succeeded. Run started by GitHub user ashb (triggered by ashb).
Head commit for run: f2c65d4a69e36663fc4d41d8798747fca2efef1e / Ash Berlin-Taylor <[email protected]> Remove side-effects in models/tests_dags affecting plugin manager tests I guess CI must not run this exact combination of tests together, but prior to this change if you ran `pytest airflow-core/tests/unit/models/test_dag.py::TestDag::test_bulk_write_to_db_assets airflow-core/tests/unit/plugins/test_plugins_manager.py::TestPluginsManager::test_registering_plugin_listeners` you would get a test failure. The issue was caused by having two fixtures of the same name, a module level `clean_plugins`, and a class level one. This is by design in Pytest and is how to override plugins at different scopes. This also explains why we had `listener_manager.clear()` in a finally block when it should have been handled by the fixture Report URL: https://github.com/apache/airflow/actions/runs/15882192816 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
