Skip to content

Maintenance: remove duplication in Idempotency tests #2918

Closed
@dreamorosi

Description

@dreamorosi

Summary

The unit tests for the makeIdempotent function wrapper and the makeHandlerIdempotent Middy.js middleware are very similar except for some lines.

According to SonarCloud, the two test files have 68% of duplication. We should work to reduce this to a minimum.

Why is this needed?

So that we can drop redundant code and reduce maintenance burden and cognitive load for maintainers and contributors.

Which area does this relate to?

Tests

Solution

No response

Acknowledgment

Future readers

Please react with 👍 and your use case to help us understand customer demand.

Metadata

Metadata

Assignees

Labels

completedThis item is complete and has been merged/shippedidempotencyThis item relates to the Idempotency UtilityinternalPRs that introduce changes in governance, tech debt and chores (linting setup, baseline, etc.)testsPRs that add or change tests

Type

No type

Projects

Status

Shipped

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions