Skip to content

Commit e0feaf1

Browse files
committed
test(aws-sdk-extension): add non-EKS JWT raise path test
1 parent 8cae40c commit e0feaf1

1 file changed

Lines changed: 18 additions & 2 deletions

File tree

  • sdk-extension/opentelemetry-sdk-extension-aws/tests/resource

sdk-extension/opentelemetry-sdk-extension-aws/tests/resource/test_eks.py

Lines changed: 18 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ def test_if_no_eks_paths_should_not_raise(
155155
@patch(
156156
"opentelemetry.sdk.extension.aws.resource.eks._get_k8s_cred_value",
157157
return_value=_bearer_jwt(
158-
{"iss": "https://oidc.eks.eu-west-2.amazonaws.com/id/EXAMPLE123"}
158+
{"iss": "https://oidc.eks.eu-west-2.amazonaws.com/id/A1B2C3D4E5F6A1B2C3D4E5F6A1B2C3D4"}
159159
),
160160
)
161161
@patch(
@@ -197,6 +197,22 @@ def test_eks_oidc_jwt_detected(
197197
"opentelemetry.sdk.extension.aws.resource.eks._is_k8s",
198198
return_value=True,
199199
)
200-
def test_non_eks_jwt_returns_empty(self, mock_is_k8s, mock_get_k8s_cred_value):
200+
def test_non_eks_jwt_returns_empty(
201+
self, mock_is_k8s, mock_get_k8s_cred_value
202+
):
201203
actual = AwsEksResourceDetector().detect()
202204
self.assertEqual(actual.attributes, {})
205+
206+
@patch(
207+
"opentelemetry.sdk.extension.aws.resource.eks._get_k8s_cred_value",
208+
return_value=_bearer_jwt({"iss": "https://wrong.jwt.com"}),
209+
)
210+
@patch(
211+
"opentelemetry.sdk.extension.aws.resource.eks._is_k8s",
212+
return_value=True,
213+
)
214+
def test_non_eks_jwt_should_raise(
215+
self, mock_is_k8s, mock_get_k8s_cred_value
216+
):
217+
with self.assertRaises(RuntimeError):
218+
AwsEksResourceDetector(raise_on_error=True).detect()

0 commit comments

Comments
 (0)