Skip to content

Maintenance: fix idempotency e2e tests #2621

Closed
@dreamorosi

Description

@dreamorosi

Summary

After merging #2570 some of the integration tests related to the Idempotency utility are failing.

This is because some of the payloads used to seed the tests don't match the control idempotency key due to the former not being deep sorted.

This was raised during the review but I forgot to ask the contributor to include it before merging.

Why is this needed?

So that we can continue running integration tests.

Which area does this relate to?

Tests

Solution

We should make sure that all the payloads in the packages/idempotency/tests/e2e/makeIdempotent.test.ts file are sorted.

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