Re: [PR] Add tests to `airflow/auth/managers/fab/decorators/auth.py` [airflow]

2023-11-02 Thread via GitHub
vincbeck merged PR #35358: URL: https://github.com/apache/airflow/pull/35358 -- 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:

Re: [PR] Add tests to `airflow/auth/managers/fab/decorators/auth.py` [airflow]

2023-11-02 Thread via GitHub
vincbeck commented on code in PR #35358: URL: https://github.com/apache/airflow/pull/35358#discussion_r1380188321 ## tests/auth/managers/fab/decorators/test_auth.py: ## @@ -0,0 +1,141 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] Add tests to `airflow/auth/managers/fab/decorators/auth.py` [airflow]

2023-11-02 Thread via GitHub
potiuk commented on PR #35358: URL: https://github.com/apache/airflow/pull/35358#issuecomment-1790298521 I think first case where you need to mark a test with DB marker :). -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and

Re: [PR] Add tests to `airflow/auth/managers/fab/decorators/auth.py` [airflow]

2023-11-01 Thread via GitHub
ephraimbuddy commented on code in PR #35358: URL: https://github.com/apache/airflow/pull/35358#discussion_r1379337932 ## tests/auth/managers/fab/decorators/test_auth.py: ## @@ -0,0 +1,141 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] Add tests to `airflow/auth/managers/fab/decorators/auth.py` [airflow]

2023-11-01 Thread via GitHub
ephraimbuddy commented on code in PR #35358: URL: https://github.com/apache/airflow/pull/35358#discussion_r1379337932 ## tests/auth/managers/fab/decorators/test_auth.py: ## @@ -0,0 +1,141 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[PR] Add tests to `airflow/auth/managers/fab/decorators/auth.py` [airflow]

2023-11-01 Thread via GitHub
vincbeck opened a new pull request, #35358: URL: https://github.com/apache/airflow/pull/35358 Add tests to the module `airflow/auth/managers/fab/decorators/auth.py`. Part of the effort of #35127 --- **^ Add meaningful description above** Read the