Skip to content

Commit 42ed2f1

Browse files
committed
Fix sonar findings.
1 parent 02532cd commit 42ed2f1

File tree

1 file changed

+10
-10
lines changed
  • powertools-idempotency/powertools-idempotency-core/src/test/java/software/amazon/lambda/powertools/idempotency/persistence

1 file changed

+10
-10
lines changed

powertools-idempotency/powertools-idempotency-core/src/test/java/software/amazon/lambda/powertools/idempotency/persistence/BasePersistenceStoreTest.java

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -240,12 +240,12 @@ void saveSuccess_withCacheEnabled_shouldSaveInCache() throws JsonProcessingExcep
240240

241241
assertThat(status).isEqualTo(2);
242242
assertThat(cache).hasSize(1);
243-
DataRecord dr = cache.get("testFunction#8d6a8f173b46479eff55e0997864a514");
244-
assertThat(dr.getStatus()).isEqualTo(DataRecord.Status.COMPLETED);
245-
assertThat(dr.getExpiryTimestamp()).isEqualTo(now.plus(3600, ChronoUnit.SECONDS).getEpochSecond());
246-
assertThat(dr.getResponseData()).isEqualTo(JsonConfig.get().getObjectMapper().writeValueAsString(product));
247-
assertThat(dr.getIdempotencyKey()).isEqualTo("testFunction#8d6a8f173b46479eff55e0997864a514");
248-
assertThat(dr.getPayloadHash()).isEmpty();
243+
DataRecord cachedDr = cache.get("testFunction#8d6a8f173b46479eff55e0997864a514");
244+
assertThat(cachedDr.getStatus()).isEqualTo(DataRecord.Status.COMPLETED);
245+
assertThat(cachedDr.getExpiryTimestamp()).isEqualTo(now.plus(3600, ChronoUnit.SECONDS).getEpochSecond());
246+
assertThat(cachedDr.getResponseData()).isEqualTo(JsonConfig.get().getObjectMapper().writeValueAsString(product));
247+
assertThat(cachedDr.getIdempotencyKey()).isEqualTo("testFunction#8d6a8f173b46479eff55e0997864a514");
248+
assertThat(cachedDr.getPayloadHash()).isEmpty();
249249
}
250250

251251
@Test
@@ -256,10 +256,10 @@ void getRecord_shouldReturnRecordFromPersistence()
256256
persistenceStore.configure(IdempotencyConfig.builder().build(), "myfunc", cache);
257257

258258
Instant now = Instant.now();
259-
DataRecord dr = persistenceStore.getRecord(JsonConfig.get().getObjectMapper().valueToTree(event), now);
260-
assertThat(dr.getIdempotencyKey()).isEqualTo("testFunction.myfunc#8d6a8f173b46479eff55e0997864a514");
261-
assertThat(dr.getStatus()).isEqualTo(DataRecord.Status.INPROGRESS);
262-
assertThat(dr.getResponseData()).isEqualTo("Response");
259+
DataRecord freshDr = persistenceStore.getRecord(JsonConfig.get().getObjectMapper().valueToTree(event), now);
260+
assertThat(freshDr.getIdempotencyKey()).isEqualTo("testFunction.myfunc#8d6a8f173b46479eff55e0997864a514");
261+
assertThat(freshDr.getStatus()).isEqualTo(DataRecord.Status.INPROGRESS);
262+
assertThat(freshDr.getResponseData()).isEqualTo("Response");
263263
assertThat(status).isZero();
264264
}
265265

0 commit comments

Comments
 (0)