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.

Sign up for GitHub

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

fix node:http events regression and add another test #10566

Merged

Jarred-Sumner merged 3 commits into main from nektro-patch-38119

Apr 27, 2024

Contributor

nektro commented Apr 27, 2024

Followup from #10447

nektro added 2 commits

April 26, 2024 22:32


revert property lookup here

83809c4


fix node:http events regression and add another test

49fe16c

Jarred-Sumner approved these changes

View reviewed changes


ci: expose env var needed for stripe test

e70caaf

github-actions bot commented Apr 27, 2024 •

edited

❌ @nektro, your commit has failing tests :(

💪 3 failing tests Darwin AARCH64

test/cli/install/registry/bun-install-registry.test.ts 1 failing
test/js/bun/http/serve.test.ts 1 failing
test/js/node/http2/node-http2.test.js 1 failing

💻 2 failing tests Darwin x64 baseline

test/cli/install/registry/bun-install-registry.test.ts 1 failing
test/js/web/workers/worker.test.ts 1 failing

💻 2 failing tests Darwin x64

test/cli/install/registry/bun-install-registry.test.ts 1 failing
test/js/node/http2/node-http2.test.js 1 failing

🐧💪 3 failing tests Linux AARCH64

test/cli/install/registry/bun-install-registry.test.ts 1 failing
test/js/node/http2/node-http2.test.js 2 failing
test/js/node/os/os.test.js 1 failing

🐧🖥 3 failing tests Linux x64 baseline

test/cli/install/registry/bun-install-registry.test.ts 1 failing
test/js/node/http2/node-http2.test.js 1 failing
test/js/web/timers/setInterval.test.js 1 failing

🐧🖥 2 failing tests Linux x64

test/cli/install/registry/bun-install-registry.test.ts 1 failing
test/js/node/http2/node-http2.test.js 2 failing

🪟💻 10 failing tests Windows x64 baseline

test/cli/install/bunx.test.ts 1 failing
test/cli/install/registry/bun-install-registry.test.ts 1 failing
test/cli/test/bun-test.test.ts 1 failing
test/integration/next-pages/test/dev-server-ssr-100.test.ts 1 failing
test/integration/next-pages/test/dev-server.test.ts 1 failing
test/integration/next-pages/test/next-build.test.ts 1 failing
test/js/bun/shell/bunshell.test.ts 2 failing
test/js/node/dns/node-dns.test.js 2 failing
test/js/node/worker_threads/worker_threads.test.ts STATUS_SEVERITY_ERROR
test/js/web/workers/worker.test.ts SIGTERM

🪟💻 10 failing tests Windows x64

test/cli/install/bunx.test.ts 1 failing
test/cli/install/registry/bun-install-registry.test.ts 2 failing
test/integration/next-pages/test/dev-server-ssr-100.test.ts 1 failing
test/integration/next-pages/test/dev-server.test.ts 1 failing
test/integration/next-pages/test/next-build.test.ts 1 failing
test/js/bun/shell/bunshell.test.ts 2 failing
test/js/bun/shell/commands/rm.test.ts 1 failing
test/js/node/child_process/child_process.test.ts 1 failing
test/js/node/dns/node-dns.test.js 2 failing
test/js/node/watch/fs.watchFile.test.ts 1 failing

Jarred-Sumner merged commit adc631c into main

44 of 52 checks passed

Jarred-Sumner deleted the nektro-patch-38119 branch

April 27, 2024 07:53

nektro mentioned this pull request

test: node:http: don't hardcode this value #10671

Merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment