Skip to content

Commit 3dee69e

Browse files
committed
Fix failing test
1 parent b9955e8 commit 3dee69e

File tree

1 file changed

+8
-21
lines changed

1 file changed

+8
-21
lines changed

tests/unit/test_lambda_helper.py

Lines changed: 8 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,7 @@ def s3_client(sagemaker_session):
5454

5555

5656
def test_lambda_object_with_arn_happycase():
57-
lambda_obj = lambda_helper.Lambda(
58-
function_arn=LAMBDA_ARN, session=sagemaker_session
59-
)
57+
lambda_obj = lambda_helper.Lambda(function_arn=LAMBDA_ARN, session=sagemaker_session)
6058
assert lambda_obj.function_arn == LAMBDA_ARN
6159

6260

@@ -378,6 +376,7 @@ def test_update_lambda_s3bucket_not_provided(s3_upload, sagemaker_session):
378376
FunctionName=LAMBDA_ARN,
379377
S3Bucket=sagemaker_session.default_bucket(),
380378
S3Key=s3_upload.return_value,
379+
Architectures=None,
381380
)
382381

383382

@@ -492,9 +491,7 @@ def test_upsert_lambda_client_error(sagemaker_session):
492491

493492

494493
def test_invoke_lambda_happycase(sagemaker_session):
495-
lambda_obj = lambda_helper.Lambda(
496-
function_arn=LAMBDA_ARN, session=sagemaker_session
497-
)
494+
lambda_obj = lambda_helper.Lambda(function_arn=LAMBDA_ARN, session=sagemaker_session)
498495
lambda_obj.invoke()
499496

500497
sagemaker_session.lambda_client.invoke.assert_called_with(
@@ -503,9 +500,7 @@ def test_invoke_lambda_happycase(sagemaker_session):
503500

504501

505502
def test_invoke_lambda_client_error(sagemaker_session):
506-
lambda_obj = lambda_helper.Lambda(
507-
function_arn=LAMBDA_ARN, session=sagemaker_session
508-
)
503+
lambda_obj = lambda_helper.Lambda(function_arn=LAMBDA_ARN, session=sagemaker_session)
509504

510505
sagemaker_session.lambda_client.invoke.side_effect = ClientError(
511506
{"Error": {"Code": "InvalidCodeException", "Message": "invoke failed"}},
@@ -518,19 +513,13 @@ def test_invoke_lambda_client_error(sagemaker_session):
518513

519514

520515
def test_delete_lambda_happycase(sagemaker_session):
521-
lambda_obj = lambda_helper.Lambda(
522-
function_arn=LAMBDA_ARN, session=sagemaker_session
523-
)
516+
lambda_obj = lambda_helper.Lambda(function_arn=LAMBDA_ARN, session=sagemaker_session)
524517
lambda_obj.delete()
525-
sagemaker_session.lambda_client.delete_function.assert_called_with(
526-
FunctionName=LAMBDA_ARN
527-
)
518+
sagemaker_session.lambda_client.delete_function.assert_called_with(FunctionName=LAMBDA_ARN)
528519

529520

530521
def test_delete_lambda_client_error(sagemaker_session):
531-
lambda_obj = lambda_helper.Lambda(
532-
function_arn=LAMBDA_ARN, session=sagemaker_session
533-
)
522+
lambda_obj = lambda_helper.Lambda(function_arn=LAMBDA_ARN, session=sagemaker_session)
534523

535524
sagemaker_session.lambda_client.delete_function.side_effect = ClientError(
536525
{"Error": {"Code": "Invalid", "Message": "Delete failed"}}, "Invoke"
@@ -542,9 +531,7 @@ def test_delete_lambda_client_error(sagemaker_session):
542531

543532

544533
def test_upload_to_s3(s3_client):
545-
key = lambda_helper._upload_to_s3(
546-
s3_client, FUNCTION_NAME, ZIPPED_CODE_DIR, S3_BUCKET
547-
)
534+
key = lambda_helper._upload_to_s3(s3_client, FUNCTION_NAME, ZIPPED_CODE_DIR, S3_BUCKET)
548535
s3_client.upload_file.assert_called_with(ZIPPED_CODE_DIR, S3_BUCKET, key)
549536
assert key == S3_KEY
550537

0 commit comments

Comments
 (0)