ashb commented on code in PR #60108:
URL: https://github.com/apache/airflow/pull/60108#discussion_r2980434615


##########
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_task_instances.py:
##########
@@ -3226,6 +3256,44 @@ def test_invalid_scope_value_rejected(self, client, 
session, create_task_instanc
         assert resp.status_code == 403
         assert "Invalid token scope" in resp.json()["detail"]
 
+    def test_workload_scope_accepted_on_run_endpoint(

Review Comment:
   Maybe we already have a test for this, but I think the negative test is 
important too -- that the workload scoped token is _not_ accepted on other 
endpoints. You don't have to exhaustively check all endpoints, but testing at 
least one other to ensure it gives a 4xx is worthwhile I think



-- 
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]

Reply via email to