Skip to content

test: stabilize async remote mutation tests - #16896

Draft
svelte-triage-bot[bot] wants to merge 1 commit into
version-3from
flake/kit-pr-16892-remote-mutation-tests
Draft

test: stabilize async remote mutation tests#16896
svelte-triage-bot[bot] wants to merge 1 commit into
version-3from
flake/kit-pr-16892-remote-mutation-tests

Conversation

@svelte-triage-bot

Copy link
Copy Markdown

Fixes two pre-existing async e2e flakes exposed by the CI run for #16892.

The form reconnect test previously polled a server-side iterator cleanup counter. The failed traces show that the form response succeeded and the targeted get_count connection restarted, but transport cleanup had not updated the counter within the assertion window. Assert the observable reconnect requests directly instead, including that get_finite_count is not reconnected.

The serial batch tests also retained module-level todo mutations across retries. Reset the todos before asserting the baseline state so reruns are idempotent.

Failed workflow: https://github.com/sveltejs/kit/actions/runs/32541364509

Verification:

  • primary test: 100 dev repetitions and 100 build repetitions
  • batch mutation sequence: 40 tests
  • relevant dev tests with two workers: 120 tests
  • relevant build tests: 60 tests
  • pnpm -F test-async check
  • formatting and diff checks

@pkg-svelte-dev

Copy link
Copy Markdown

Install the latest version of @sveltejs/kit from a425290:

pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/a4252909c77a1117b01a588c653a6919f3fcbca2

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/kit/pr/16896

Note

This PR is from a fork. A maintainer must approve approve each commit before it can be built and installed.

@changeset-bot

changeset-bot Bot commented Aug 22, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: a425290

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

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.

0 participants