Skip to content

Commit 75fec0a

Browse files
fix(idempotency): fix test fixture name and trailing whitespace in test_redis_layer.py
- Fix fixture name from persistence_store_redis to persistence_store_standalone_redis to match existing fixture defined in the test file - Remove trailing whitespace on blank line after test function to pass ruff format check Part of #8090 Signed-off-by: hirenkumar-n-dholariya <hirenkumarnd@gmail.com>
1 parent fec6b64 commit 75fec0a

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

tests/functional/idempotency/_redis/test_redis_layer.py

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -330,24 +330,24 @@ def test_item_to_datarecord_conversion(valid_record):
330330
assert record.in_progress_expiry_timestamp == item[layer.in_progress_expiry_attr]
331331

332332

333-
def test_item_to_datarecord_conversion_missing_optional_attributes(persistence_store_redis):
333+
def test_item_to_datarecord_conversion_missing_optional_attributes(persistence_store_standalone_redis):
334334
"""
335335
When data_attr or validation_key_attr is missing from Redis,
336336
response_data and payload_hash should be None — not the string "None".
337337
Regression test for: https://github.com/aws-powertools/powertools-lambda-python/issues/8090
338338
"""
339339
idempotency_key = "test-func#abc123"
340340
item = {
341-
persistence_store_redis.status_attr: "COMPLETED",
342-
persistence_store_redis.expiry_attr: 9999999999,
341+
persistence_store_standalone_redis.status_attr: "COMPLETED",
342+
persistence_store_standalone_redis.expiry_attr: 9999999999,
343343
# data_attr and validation_key_attr intentionally absent
344344
}
345345

346-
record = persistence_store_redis._item_to_data_record(idempotency_key, item)
346+
record = persistence_store_standalone_redis._item_to_data_record(idempotency_key, item)
347347

348348
assert record.response_data is None
349349
assert record.payload_hash is None
350-
350+
351351

352352
def test_idempotent_function_and_lambda_handler_redis_basic(
353353
persistence_store_standalone_redis: RedisCachePersistenceLayer,

0 commit comments

Comments
 (0)