potiuk opened a new pull request, #75126: URL: https://github.com/apache/airflow/pull/75126
`TestGetDagRunAssetTriggerEvents::test_returns_only_events_of_assets_the_caller_may_read`, added in #72924, fails on `main`: the endpoint returns all three events instead of the one on the readable asset. The test patches `BaseAuthManager.get_authorized_assets`, but since #72934 `SimpleAuthManager` (the auth manager the API test client uses) overrides that method, so the patch never takes effect and every asset stays readable. #72924's CI ran on a base without #72934. The production filter is fine; this patches `SimpleAuthManager.get_authorized_assets`, as the other asset permission tests in `routes/public/test_assets.py` and `routes/ui/test_assets.py` do. `TestGetDagRunAssetTriggerEvents`: 13 passed (previously 1 failed on `main`). --- ##### Was generative AI tooling used to co-author this PR? - [X] Yes — Claude Code (Opus 5) Generated-by: Claude Code (Opus 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
