-
Notifications
You must be signed in to change notification settings - Fork 787
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore(fixtures): add d1 fixture tests #8213
Conversation
|
d8109fa
to
92e0594
Compare
A wrangler prerelease is available for testing. You can install this latest build in your project with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-wrangler-8213 You can reference the automatically updated head of this PR with: npm install --save-dev https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/prs/8213/npm-package-wrangler-8213 Or you can use npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-wrangler-8213 dev path/to/script.js Additional artifacts:cloudflare-workers-bindings-extension: wget https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-cloudflare-workers-bindings-extension-8213 -O ./cloudflare-workers-bindings-extension.0.0.0-v8cfa21641.vsix && code --install-extension ./cloudflare-workers-bindings-extension.0.0.0-v8cfa21641.vsix create-cloudflare: npx https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-create-cloudflare-8213 --no-auto-update @cloudflare/kv-asset-handler: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-cloudflare-kv-asset-handler-8213 miniflare: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-miniflare-8213 @cloudflare/pages-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-cloudflare-pages-shared-8213 @cloudflare/unenv-preset: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-cloudflare-unenv-preset-8213 @cloudflare/vite-plugin: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-cloudflare-vite-plugin-8213 @cloudflare/vitest-pool-workers: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-cloudflare-vitest-pool-workers-8213 @cloudflare/workers-editor-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-cloudflare-workers-editor-shared-8213 @cloudflare/workers-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-cloudflare-workers-shared-8213 @cloudflare/workflows-shared: npm install https://prerelease-registry.devprod.cloudflare.dev/workers-sdk/runs/13460546256/npm-package-cloudflare-workflows-shared-8213 Note that these links will no longer work once the GitHub Actions artifact expires.
Please ensure constraints are pinned, and |
92e0594
to
86f3f05
Compare
86f3f05
to
44382d1
Compare
This PR adds a test in the
d1-worker-app
fixture. This automated test will cover changes introduced by #8175. This PR needs to be merged in order for us to land those changes.Fixes #00000
How this was additionally tested
After creating the test, I first ran it against the wrangler version that didn't contain the d1 changes
then ran it again against the wrangler version that contained the changes