Skip to content

Commit 0aef100

Browse files
committed
chore(idempotency): remove unused fixtures in tests
1 parent 63d9a9f commit 0aef100

File tree

1 file changed

+4
-23
lines changed

1 file changed

+4
-23
lines changed

tests/functional/idempotency/test_idempotency.py

+4-23
Original file line numberDiff line numberDiff line change
@@ -244,9 +244,6 @@ def test_idempotent_lambda_first_execution(
244244
expected_params_update_item,
245245
expected_params_put_item,
246246
lambda_response,
247-
serialized_lambda_response,
248-
deserialized_lambda_response,
249-
hashed_idempotency_key,
250247
lambda_context,
251248
):
252249
"""
@@ -384,11 +381,9 @@ def test_idempotent_lambda_expired(
384381
idempotency_config: IdempotencyConfig,
385382
persistence_store: DynamoDBPersistenceLayer,
386383
lambda_apigw_event,
387-
timestamp_expired,
388384
lambda_response,
389385
expected_params_update_item,
390386
expected_params_put_item,
391-
hashed_idempotency_key,
392387
lambda_context,
393388
):
394389
"""
@@ -428,8 +423,6 @@ def test_idempotent_lambda_exception(
428423
idempotency_config: IdempotencyConfig,
429424
persistence_store: DynamoDBPersistenceLayer,
430425
lambda_apigw_event,
431-
timestamp_future,
432-
lambda_response,
433426
hashed_idempotency_key,
434427
expected_params_put_item,
435428
lambda_context,
@@ -594,9 +587,6 @@ def test_idempotent_persistence_exception_deleting(
594587
idempotency_config: IdempotencyConfig,
595588
persistence_store: DynamoDBPersistenceLayer,
596589
lambda_apigw_event,
597-
timestamp_future,
598-
lambda_response,
599-
hashed_idempotency_key,
600590
expected_params_put_item,
601591
lambda_context,
602592
):
@@ -638,9 +628,6 @@ def test_idempotent_persistence_exception_updating(
638628
idempotency_config: IdempotencyConfig,
639629
persistence_store: DynamoDBPersistenceLayer,
640630
lambda_apigw_event,
641-
timestamp_future,
642-
lambda_response,
643-
hashed_idempotency_key,
644631
expected_params_put_item,
645632
lambda_context,
646633
):
@@ -682,10 +669,6 @@ def test_idempotent_persistence_exception_getting(
682669
idempotency_config: IdempotencyConfig,
683670
persistence_store: DynamoDBPersistenceLayer,
684671
lambda_apigw_event,
685-
timestamp_future,
686-
lambda_response,
687-
hashed_idempotency_key,
688-
expected_params_put_item,
689672
lambda_context,
690673
):
691674
"""
@@ -727,8 +710,6 @@ def test_idempotent_lambda_first_execution_with_validation(
727710
expected_params_update_item_with_validation,
728711
expected_params_put_item_with_validation,
729712
lambda_response,
730-
hashed_idempotency_key,
731-
hashed_validation_key,
732713
lambda_context,
733714
):
734715
"""
@@ -1107,7 +1088,7 @@ def test_is_missing_idempotency_key():
11071088
indirect=True,
11081089
)
11091090
def test_default_no_raise_on_missing_idempotency_key(
1110-
idempotency_config: IdempotencyConfig, persistence_store: DynamoDBPersistenceLayer, lambda_context
1091+
idempotency_config: IdempotencyConfig, persistence_store: DynamoDBPersistenceLayer
11111092
):
11121093
# GIVEN a persistence_store with use_local_cache = False and event_key_jmespath = "body"
11131094
function_name = "foo"
@@ -1132,7 +1113,7 @@ def test_default_no_raise_on_missing_idempotency_key(
11321113
indirect=True,
11331114
)
11341115
def test_raise_on_no_idempotency_key(
1135-
idempotency_config: IdempotencyConfig, persistence_store: DynamoDBPersistenceLayer, lambda_context
1116+
idempotency_config: IdempotencyConfig, persistence_store: DynamoDBPersistenceLayer
11361117
):
11371118
# GIVEN a persistence_store with raise_on_no_idempotency_key and no idempotency key in the request
11381119
persistence_store.configure(idempotency_config)
@@ -1165,7 +1146,7 @@ def test_raise_on_no_idempotency_key(
11651146
indirect=True,
11661147
)
11671148
def test_jmespath_with_powertools_json(
1168-
idempotency_config: IdempotencyConfig, persistence_store: DynamoDBPersistenceLayer, lambda_context
1149+
idempotency_config: IdempotencyConfig, persistence_store: DynamoDBPersistenceLayer
11691150
):
11701151
# GIVEN an event_key_jmespath with powertools_json custom function
11711152
persistence_store.configure(idempotency_config, "handler")
@@ -1186,7 +1167,7 @@ def test_jmespath_with_powertools_json(
11861167

11871168
@pytest.mark.parametrize("config_with_jmespath_options", ["powertools_json(data).payload"], indirect=True)
11881169
def test_custom_jmespath_function_overrides_builtin_functions(
1189-
config_with_jmespath_options: IdempotencyConfig, persistence_store: DynamoDBPersistenceLayer, lambda_context
1170+
config_with_jmespath_options: IdempotencyConfig, persistence_store: DynamoDBPersistenceLayer
11901171
):
11911172
# GIVEN a persistence store with a custom jmespath_options
11921173
# AND use a builtin powertools custom function

0 commit comments

Comments
 (0)