potiuk commented on PR #50449: URL: https://github.com/apache/airflow/pull/50449#issuecomment-2897572327
https://github.com/apache/airflow/blob/2442b215db2e800b72af06bba80b327fce5a41d1/providers/amazon/tests/unit/amazon/aws/auth_manager/routes/test_login.py#L29 > > but this import should be available when the `apache-airflow-providers-amazon[python3-saml]` is installed in the CI. So I guess I am missing something to make this extra installed in the CI @eladkal - Yes. The problem is that in the compatibility tests and amazon tests with lowest dependencies we do not use the extra when installing the provider. In those tests we are installing just `provider` - no optional extras. Since this is now effectively an optional feature we can make the tests optional via - `importorskip` -- 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]
