You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by sh...@apache.org on 2026/10/19 19:19:00 UTC

(airflow) branch main updated: Mock the simple auth manager in the upstream asset events permission test (#75126)

This is an automated email from the ASF dual-hosted git repository.

shahar1 pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 33e6b5f1fa9 Mock the simple auth manager in the upstream asset events permission test (#75126)
33e6b5f1fa9 is described below

commit 33e6b5f1fa9f0e0a96fd52075e89000d47b781ee
Author: Jarek Potiuk <ja...@potiuk.com>
AuthorDate: Mon Oct 19 21:18:52 2026 +0200

    Mock the simple auth manager in the upstream asset events permission test (#75126)
    
    SimpleAuthManager overrides get_authorized_assets since #72934, so patching the BaseAuthManager method left the real one in place, every asset stayed readable, and the test added in #72924 failed on main.
    
    Generated-by: Claude Opus 5
---
 .../tests/unit/api_fastapi/core_api/routes/public/test_dag_run.py       | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_dag_run.py b/airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_dag_run.py
index 83c243fa98c..0fec3f0fe15 100644
--- a/airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_dag_run.py
+++ b/airflow-core/tests/unit/api_fastapi/core_api/routes/public/test_dag_run.py
@@ -2181,7 +2181,7 @@ class TestGetDagRunAssetTriggerEvents:
         autospec=True,
     )
     @mock.patch(
-        "airflow.api_fastapi.auth.managers.base_auth_manager.BaseAuthManager.get_authorized_assets",
+        "airflow.api_fastapi.auth.managers.simple.simple_auth_manager.SimpleAuthManager.get_authorized_assets",
         autospec=True,
     )
     def test_returns_only_events_of_assets_the_caller_may_read(