Skip to content

chore: fix server fetch test setup - #17290

Merged
teemingc merged 1 commit into
version-3from
elliott/fix-fetch-test-options
Oct 1, 2026
Merged

teemingc merged 1 commit into
version-3from
elliott/fix-fetch-test-options

Conversation

@elliott-with-the-longest-name-on-github

Copy link
Copy Markdown
Contributor

The newly ported self-fetch redirect tests call set_options, which was removed from version-3 in #17008. The suite fails during setup before any of its six regression tests run, as seen in the checks on #17288.

Provide options through the generated-server mock instead, matching the current runtime. Also formats one overlong import in the recently added action tests. No runtime changes or changeset. Verified the failure before the fix; afterward all six regression tests and all 77 Kit unit-test files pass (1,053 tests, 110 skipped), along with format, lint, and full typechecking.

@pkg-svelte-dev

pkg-svelte-dev Bot commented Oct 1, 2026

Copy link
Copy Markdown

Install the latest version of @sveltejs/kit from 1e26014:

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

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

@changeset-bot

changeset-bot Bot commented Oct 1, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 1e26014

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

@teemingc
teemingc merged commit 7e55449 into version-3 Oct 1, 2026
40 of 41 checks passed
@teemingc
teemingc deleted the elliott/fix-fetch-test-options branch October 1, 2026 15:34
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.

3 participants