You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/05/06 05:31:43 UTC

[GitHub] [airflow] jhtimmins commented on pull request #15385: [WIP] Added table to view providers in airflow ui under admin tab

jhtimmins commented on pull request #15385:
URL: https://github.com/apache/airflow/pull/15385#issuecomment-833237576


   > Made readability changes and rebased but am uncertain how to check for permissions in the views_test.py
   
   Thanks @ShakaibKhan. You can call `.create_user_and_login` (as seen [here](https://github.com/apache/airflow/blob/master/tests/www/test_views.py#L2162)) to create a user with specific permissions. So if you can call that in `test_should_list_providers_on_page_with_details` to give them the necessary permissions, and then add another test where you create a user but don't give them any permissions, so they've authenticated but not authorized.


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org