From f3868e2b08d32db8fb2afbbb5cda4ccc1dc27626 Mon Sep 17 00:00:00 2001 From: Dirk Kulawiak Date: Tue, 20 Aug 2024 13:36:04 +0200 Subject: [PATCH 1/3] disable WCS auth tests --- integration/test_auth.py | 12 ++++++------ integration_v3/test_authentication.py | 18 ++++++++++-------- 2 files changed, 16 insertions(+), 14 deletions(-) diff --git a/integration/test_auth.py b/integration/test_auth.py index e3dd81c99..5889a74c0 100644 --- a/integration/test_auth.py +++ b/integration/test_auth.py @@ -168,12 +168,12 @@ def _get_access_token(url: str, user: str, pw: str) -> Dict[str, str]: @pytest.mark.parametrize( "name,user,env_variable_name,port", [ - ( - "WCS", - "ms_2d0e007e7136de11d5f29fce7a53dae219a51458@existiert.net", - "WCS_DUMMY_CI_PW", - WCS_PORT, - ), + # ( # WCS keycloak times out too often + # "WCS", + # "ms_2d0e007e7136de11d5f29fce7a53dae219a51458@existiert.net", + # "WCS_DUMMY_CI_PW", + # WCS_PORT, + # ), ( "okta", "test@test.de", diff --git a/integration_v3/test_authentication.py b/integration_v3/test_authentication.py index cf64bf14b..23bd25ba4 100644 --- a/integration_v3/test_authentication.py +++ b/integration_v3/test_authentication.py @@ -89,14 +89,14 @@ def test_authentication_client_credentials( @pytest.mark.parametrize( "name,user,env_variable_name,port,scope,warning", [ - ( - "WCS", - "ms_2d0e007e7136de11d5f29fce7a53dae219a51458@existiert.net", - "WCS_DUMMY_CI_PW", - WCS_PORT, - None, - False, - ), + # ( # WCS keycloak times out too often + # "WCS", + # "ms_2d0e007e7136de11d5f29fce7a53dae219a51458@existiert.net", + # "WCS_DUMMY_CI_PW", + # WCS_PORT, + # None, + # False, + # ), ( "okta", "test@test.de", @@ -227,6 +227,8 @@ def test_client_with_authentication_with_anon_weaviate(recwarn): def test_bearer_token_without_refresh(recwarn): """Test that the client warns users when only supplying an access token without refresh.""" + pytest.skip("WCS keycloak times out too often") + # testing for warnings can be flaky without this as there are open SSL conections warnings.filterwarnings(action="ignore", message="unclosed", category=ResourceWarning) warnings.filterwarnings(action="ignore", message="Dep005", category=DeprecationWarning) From 2f279e9ef12f5e57d4a3054ef23ca36520fbc68b Mon Sep 17 00:00:00 2001 From: Dirk Kulawiak Date: Tue, 20 Aug 2024 13:48:03 +0200 Subject: [PATCH 2/3] Another one --- integration/test_auth.py | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/integration/test_auth.py b/integration/test_auth.py index 5889a74c0..758580816 100644 --- a/integration/test_auth.py +++ b/integration/test_auth.py @@ -66,14 +66,14 @@ def test_authentication_client_credentials( @pytest.mark.parametrize( "name,user,env_variable_name,port,scope,warning", [ - ( - "WCS", - "ms_2d0e007e7136de11d5f29fce7a53dae219a51458@existiert.net", - "WCS_DUMMY_CI_PW", - WCS_PORT, - None, - False, - ), + # ( # WCS keycloak times out too often + # "WCS", + # "ms_2d0e007e7136de11d5f29fce7a53dae219a51458@existiert.net", + # "WCS_DUMMY_CI_PW", + # WCS_PORT, + # None, + # False, + # ), ( "okta", "test@test.de", From 10b10971c26e40bf9e47aa881a35565265fa2b6e Mon Sep 17 00:00:00 2001 From: Dirk Kulawiak Date: Tue, 20 Aug 2024 14:27:14 +0200 Subject: [PATCH 3/3] another v3 test --- integration_v3/test_authentication.py | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/integration_v3/test_authentication.py b/integration_v3/test_authentication.py index 23bd25ba4..323527d2b 100644 --- a/integration_v3/test_authentication.py +++ b/integration_v3/test_authentication.py @@ -168,12 +168,12 @@ def _get_access_token(url: str, user: str, pw: str) -> Dict[str, str]: @pytest.mark.parametrize( "name,user,env_variable_name,port", [ - ( - "WCS", - "ms_2d0e007e7136de11d5f29fce7a53dae219a51458@existiert.net", - "WCS_DUMMY_CI_PW", - WCS_PORT, - ), + # ( + # "WCS", + # "ms_2d0e007e7136de11d5f29fce7a53dae219a51458@existiert.net", + # "WCS_DUMMY_CI_PW", + # WCS_PORT, + # ), ( "okta", "test@test.de",