You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by el...@apache.org on 2022/08/17 05:23:50 UTC

[airflow] branch main updated: Fix SystemsManagerParameterStoreBackend test (#25751)

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

eladkal 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 dc738cde04 Fix SystemsManagerParameterStoreBackend test (#25751)
dc738cde04 is described below

commit dc738cde04d91084bf79b1a601395b7abd41d8ca
Author: Tzu-ping Chung <ur...@gmail.com>
AuthorDate: Wed Aug 17 13:23:37 2022 +0800

    Fix SystemsManagerParameterStoreBackend test (#25751)
    
    This has been failing since the backend moved use_ssl to a separated
    attribute instead of bundling it in kwargs.
---
 tests/secrets/test_secrets.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/tests/secrets/test_secrets.py b/tests/secrets/test_secrets.py
index f8d8a6717c..d4a2bfa7fe 100644
--- a/tests/secrets/test_secrets.py
+++ b/tests/secrets/test_secrets.py
@@ -69,13 +69,13 @@ class TestConnectionsFromSecrets(unittest.TestCase):
     )
     def test_backends_kwargs(self):
         backends = initialize_secrets_backends()
-        systems_manager = [
+        systems_manager = next(
             backend
             for backend in backends
             if backend.__class__.__name__ == 'SystemsManagerParameterStoreBackend'
-        ][0]
-
-        assert systems_manager.kwargs == {'use_ssl': False}
+        )
+        assert systems_manager.kwargs == {}
+        assert systems_manager.use_ssl is False
 
     @conf_vars(
         {