Skip to content

fix output comparisons for e2e tests - #14884

Merged
emily-shen merged 1 commit into
mainfrom
pbd/fix-e2e-tests
Jul 28, 2026
Merged

fix output comparisons for e2e tests#14884
emily-shen merged 1 commit into
mainfrom
pbd/fix-e2e-tests

Conversation

@petebacondarwin

@petebacondarwin petebacondarwin commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

#14806 introduced a change that broke the remote e2e tests.


  • Tests
    • Tests included/updated
    • Automated tests not possible - manual testing has been completed as follows:
    • Additional testing not necessary because:
  • Public documentation
    • Cloudflare docs PR(s):
    • Documentation not necessary because: test change

A picture of a cute animal (not mandatory, but encouraged)


Open in Devin Review

@petebacondarwin petebacondarwin added the ci:no-changeset-required Skip pull request checks for a valid changeset label Jul 27, 2026
@changeset-bot

changeset-bot Bot commented Jul 27, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 5bfdf44

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

@github-project-automation github-project-automation Bot moved this to Untriaged in workers-sdk Jul 27, 2026
@workers-devprod
workers-devprod requested review from a team and dario-piotrowicz and removed request for a team July 27, 2026 21:18
@workers-devprod

workers-devprod commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Codeowners approval required for this PR:

  • ✅ @cloudflare/wrangler
Show detailed file reviewers

@petebacondarwin petebacondarwin added the ci:run-remote-tests Run remote/E2E tests that require Cloudflare API credentials label Jul 27, 2026
@ask-bonk

ask-bonk Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

UnknownError: ProviderInitError

github run

@ask-bonk

ask-bonk Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

@petebacondarwin Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

devin-ai-integration[bot]

This comment was marked as resolved.

@pkg-pr-new

pkg-pr-new Bot commented Jul 27, 2026

Copy link
Copy Markdown
@cloudflare/autoconfig

npm i https://pkg.pr.new/@cloudflare/autoconfig@14884

@cloudflare/config

npm i https://pkg.pr.new/@cloudflare/config@14884

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@14884

@cloudflare/deploy-helpers

npm i https://pkg.pr.new/@cloudflare/deploy-helpers@14884

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@14884

miniflare

npm i https://pkg.pr.new/miniflare@14884

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@14884

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@14884

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@14884

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@14884

@cloudflare/workers-auth

npm i https://pkg.pr.new/@cloudflare/workers-auth@14884

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@14884

@cloudflare/workers-utils

npm i https://pkg.pr.new/@cloudflare/workers-utils@14884

wrangler

npm i https://pkg.pr.new/wrangler@14884

commit: 5bfdf44

@emily-shen
emily-shen enabled auto-merge (squash) July 28, 2026 08:14

@workers-devprod workers-devprod left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Codeowners reviews satisfied

@emily-shen
emily-shen merged commit e353367 into main Jul 28, 2026
68 of 69 checks passed
@emily-shen
emily-shen deleted the pbd/fix-e2e-tests branch July 28, 2026 08:14
@github-project-automation github-project-automation Bot moved this from Untriaged to Done in workers-sdk Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:no-changeset-required Skip pull request checks for a valid changeset ci:run-remote-tests Run remote/E2E tests that require Cloudflare API credentials

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants