Skip to content

DF-24087, OPDATA-6745, OPDATA-7912: Add test for request deduping bug#840

Merged
dskloetc merged 1 commit into
mainfrom
kloet/dedup-bug-test
Jun 29, 2026
Merged

DF-24087, OPDATA-6745, OPDATA-7912: Add test for request deduping bug#840
dskloetc merged 1 commit into
mainfrom
kloet/dedup-bug-test

Conversation

@dskloetc

@dskloetc dskloetc commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

Description

We previously had a bug in request deduping described in DF-24087.
This bug got unintentionally fixed in #805, which therefore didn't add a regression test for it.

This PR adds a regression test for the bug that is already fixed.

Changes

  1. Add a test that tries to make 2 identical requests right after each other.

Testing

I verified that while this test currently passes, it fails when added before #805

@dskloetc dskloetc changed the title DF-24087, OPDATA-6745: Add test for request deduping bug DF-24087, OPDATA-6745, OPDATA-7912: Add test for request deduping bug Jun 29, 2026
@github-actions

github-actions Bot commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

NPM Publishing labels 🏷️

🔵 This PR has the none label set and it will not cause a version bump.

@dskloetc dskloetc added the none label Jun 29, 2026
@dskloetc dskloetc marked this pull request as ready for review June 29, 2026 11:23
@dskloetc dskloetc requested a review from a team as a code owner June 29, 2026 11:23
@dskloetc dskloetc merged commit 31ed5a9 into main Jun 29, 2026
18 of 19 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

❌ failed to create version bump PR
Failure message: No message provided
Failed job: createVersionBumpPR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants