Skip to content

test: add race-condition e2e#1896

Merged
dai-shi merged 7 commits intowakujs:mainfrom
Master-Hash:race-condition-e2e
Feb 27, 2026
Merged

test: add race-condition e2e#1896
dai-shi merged 7 commits intowakujs:mainfrom
Master-Hash:race-condition-e2e

Conversation

@Master-Hash
Copy link
Contributor

@Master-Hash Master-Hash commented Jan 11, 2026

fix #1880

@vercel
Copy link

vercel bot commented Jan 11, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
waku Ready Ready Preview Feb 27, 2026 9:57am

Request Review

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 11, 2026

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Copy link
Member

@dai-shi dai-shi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks. I'll have a closer look later.

@dai-shi
Copy link
Member

dai-shi commented Jan 11, 2026

marking draft for now.

@dai-shi dai-shi marked this pull request as draft January 11, 2026 06:25
Master-Hash and others added 2 commits January 11, 2026 14:25
@pkg-pr-new
Copy link

pkg-pr-new bot commented Jan 13, 2026

Open in StackBlitz

npm i https://pkg.pr.new/waku@1896

commit: c6d482c

Copy link
Member

@dai-shi dai-shi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay. It looks like a good test. We should fix it. Thanks.

Comment on lines +9 to +10
void push('/about');
void push('/bar');
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If the reproduction is such a simple way, I would probably consider putting the test case in an existing spec and fixture.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dai-shi dai-shi marked this pull request as ready for review February 27, 2026 10:29
@dai-shi dai-shi merged commit 0963dae into wakujs:main Feb 27, 2026
48 of 49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Race condition in router

2 participants