This is an automated email from the ASF dual-hosted git repository.

vincbeck pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git


The following commit(s) were added to refs/heads/main by this push:
     new 0a81eb08d57 Fix failing compatibility test: importing missing saml for 
old airflows (#42113)
0a81eb08d57 is described below

commit 0a81eb08d57215b7354231b54558173b821497ea
Author: Jarek Potiuk <[email protected]>
AuthorDate: Mon Sep 9 13:15:30 2024 -0700

    Fix failing compatibility test: importing missing saml for old airflows 
(#42113)
    
    The Compatibility tests for AWS are failing after recent changes
    as they attempt to import saml library before skipping the tests
    when the import is missing.
---
 pyproject.toml                                                   | 1 +
 tests/system/providers/amazon/aws/tests/test_aws_auth_manager.py | 4 ++--
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/pyproject.toml b/pyproject.toml
index 3c62b110608..7c466743a93 100644
--- a/pyproject.toml
+++ b/pyproject.toml
@@ -387,6 +387,7 @@ combine-as-imports = true
 "airflow/security/kerberos.py" = ["E402"]
 "airflow/security/utils.py" = ["E402"]
 
"tests/providers/amazon/aws/auth_manager/security_manager/test_aws_security_manager_override.py"
 = ["E402"]
+"tests/system/providers/amazon/aws/tests/test_aws_auth_manager.py" = ["E402"]
 "tests/providers/common/io/xcom/test_backend.py" = ["E402"]
 "tests/providers/elasticsearch/log/elasticmock/__init__.py" = ["E402"]
 "tests/providers/elasticsearch/log/elasticmock/utilities/__init__.py" = 
["E402"]
diff --git a/tests/system/providers/amazon/aws/tests/test_aws_auth_manager.py 
b/tests/system/providers/amazon/aws/tests/test_aws_auth_manager.py
index 44c0bcecc3b..792df7b155d 100644
--- a/tests/system/providers/amazon/aws/tests/test_aws_auth_manager.py
+++ b/tests/system/providers/amazon/aws/tests/test_aws_auth_manager.py
@@ -22,13 +22,13 @@ from unittest.mock import Mock, patch
 import boto3
 import pytest
 
+pytest.importorskip("onelogin")
+
 from airflow.www import app as application
 from tests.system.providers.amazon.aws.utils import set_env_id
 from tests.test_utils.config import conf_vars
 from tests.test_utils.www import check_content_in_response
 
-pytest.importorskip("onelogin")
-
 SAML_METADATA_URL = "/saml/metadata"
 SAML_METADATA_PARSED = {
     "idp": {

Reply via email to