Skip to content

Conversation

fry69
Copy link
Contributor

@fry69 fry69 commented Sep 26, 2025

This fixes issues I have with running tests on one of my servers, where port 8000 on localhost is occupied.

Note: The current init test will continue to fail, but that will hopefully get replaced by #3524 which does not have this problem.

Before:

[
  "error: Uncaught (in promise) AddrInUse: Address already in use (os error 98)",
  "    at listen (ext:deno_net/01_net.js:594:35)",
  "    at serveInner (ext:deno_http/00_serve.ts:675:16)",
  "    at Object.serve (ext:deno_http/00_serve.ts:591:10)",
  "    at ext:deno_http/00_serve.ts:819:10",
  "    at ext:runtime_main/js/99_main.js:708:13",
  "    at eventLoopTick (ext:core/01_core.js:179:7)"
]
[...]
Builder - write prod routePattern => ./packages/fresh/src/dev/builder_test.ts:284:6
Builder - file:// islands => ./packages/fresh/src/dev/builder_test.ts:319:6
Builder - serves static files in subdir in prod => ./packages/fresh/src/dev/builder_test.ts:474:6
init - can start built project => ./packages/init/src/init_test.ts:237:6

FAILED | 471 passed (14 steps) | 4 failed | 18 ignored

After:

ok | 475 passed (14 steps) | 0 failed | 18 ignored

@fry69 fry69 force-pushed the fry69/fix-tests-port-in-use branch from 49e162f to 9c1d4ca Compare October 6, 2025 06:56
Copy link
Collaborator

@marvinhagemeister marvinhagemeister left a comment

Choose a reason for hiding this comment

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

LGTM

@marvinhagemeister marvinhagemeister merged commit 317f1cb into denoland:main Oct 6, 2025
12 of 13 checks passed
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.

2 participants