From 103ba194630f5f32df0a69bc8ee85f8a2e90ce93 Mon Sep 17 00:00:00 2001
From: Bjorn Lu <bjornlu.dev@gmail.com>
Date: Tue, 28 Nov 2023 00:33:10 +0800
Subject: Clean up tests (#9183)

---
 packages/integrations/node/test/headers.test.js | 147 ++++++++++++++++++++++++
 1 file changed, 147 insertions(+)
 create mode 100644 packages/integrations/node/test/headers.test.js

(limited to 'packages/integrations/node/test/headers.test.js')

diff --git a/packages/integrations/node/test/headers.test.js b/packages/integrations/node/test/headers.test.js
new file mode 100644
index 000000000..17cfd3701
--- /dev/null
+++ b/packages/integrations/node/test/headers.test.js
@@ -0,0 +1,147 @@
+import nodejs from '../dist/index.js';
+import { loadFixture, createRequestAndResponse } from './test-utils.js';
+import { expect } from 'chai';
+
+describe('Node Adapter Headers', () => {
+	/** @type {import('./test-utils').Fixture} */
+	let fixture;
+
+	before(async () => {
+		fixture = await loadFixture({
+			root: './fixtures/headers/',
+			output: 'server',
+			adapter: nodejs({ mode: 'middleware' }),
+		});
+		await fixture.build();
+	});
+
+	it('Endpoint Simple Headers', async () => {
+		await runTest('/endpoints/simple', {
+			'content-type': 'text/plain;charset=utf-8',
+			'x-hello': 'world',
+		});
+	});
+
+	it('Endpoint Astro Single Cookie Header', async () => {
+		await runTest('/endpoints/astro-cookies-single', {
+			'content-type': 'text/plain;charset=utf-8',
+			'set-cookie': 'from1=astro1',
+		});
+	});
+
+	it('Endpoint Astro Multi Cookie Header', async () => {
+		await runTest('/endpoints/astro-cookies-multi', {
+			'content-type': 'text/plain;charset=utf-8',
+			'set-cookie': ['from1=astro1', 'from2=astro2'],
+		});
+	});
+
+	it('Endpoint Response Single Cookie Header', async () => {
+		await runTest('/endpoints/response-cookies-single', {
+			'content-type': 'text/plain;charset=utf-8',
+			'set-cookie': 'hello1=world1',
+		});
+	});
+
+	it('Endpoint Response Multi Cookie Header', async () => {
+		await runTest('/endpoints/response-cookies-multi', {
+			'content-type': 'text/plain;charset=utf-8',
+			'set-cookie': ['hello1=world1', 'hello2=world2'],
+		});
+	});
+
+	it('Endpoint Complex Headers Kitchen Sink', async () => {
+		await runTest('/endpoints/kitchen-sink', {
+			'content-type': 'text/plain;charset=utf-8',
+			'x-single': 'single',
+			'x-triple': 'one, two, three',
+			'set-cookie': ['hello1=world1', 'hello2=world2'],
+		});
+	});
+
+	it('Endpoint Astro and Response Single Cookie Header', async () => {
+		await runTest('/endpoints/astro-response-cookie-single', {
+			'content-type': 'text/plain;charset=utf-8',
+			'set-cookie': ['from1=response1', 'from1=astro1'],
+		});
+	});
+
+	it('Endpoint Astro and Response Multi Cookie Header', async () => {
+		await runTest('/endpoints/astro-response-cookie-multi', {
+			'content-type': 'text/plain;charset=utf-8',
+			'set-cookie': ['from1=response1', 'from2=response2', 'from3=astro1', 'from4=astro2'],
+		});
+	});
+
+	it('Endpoint Response Empty Headers Object', async () => {
+		await runTest('/endpoints/response-empty-headers-object', {
+			'content-type': 'text/plain;charset=UTF-8',
+		});
+	});
+
+	it('Endpoint Response undefined Headers Object', async () => {
+		await runTest('/endpoints/response-undefined-headers-object', {
+			'content-type': 'text/plain;charset=UTF-8',
+		});
+	});
+
+	it('Component Astro Single Cookie Header', async () => {
+		await runTest('/astro/component-astro-cookies-single', {
+			'content-type': 'text/html',
+			'set-cookie': 'from1=astro1',
+		});
+	});
+
+	it('Component Astro Multi Cookie Header', async () => {
+		await runTest('/astro/component-astro-cookies-multi', {
+			'content-type': 'text/html',
+			'set-cookie': ['from1=astro1', 'from2=astro2'],
+		});
+	});
+
+	it('Component Response Single Cookie Header', async () => {
+		await runTest('/astro/component-response-cookies-single', {
+			'content-type': 'text/html',
+			'set-cookie': 'from1=value1',
+		});
+	});
+
+	it('Component Response Multi Cookie Header', async () => {
+		await runTest('/astro/component-response-cookies-multi', {
+			'content-type': 'text/html',
+			'set-cookie': ['from1=value1', 'from2=value2'],
+		});
+	});
+
+	it('Component Astro and Response Single Cookie Header', async () => {
+		await runTest('/astro/component-astro-response-cookie-single', {
+			'content-type': 'text/html',
+			'set-cookie': ['from1=response1', 'from1=astro1'],
+		});
+	});
+
+	it('Component Astro and Response Multi Cookie Header', async () => {
+		await runTest('/astro/component-astro-response-cookie-multi', {
+			'content-type': 'text/html',
+			'set-cookie': ['from1=response1', 'from2=response2', 'from3=astro1', 'from4=astro2'],
+		});
+	});
+});
+
+async function runTest(url, expectedHeaders) {
+	const { handler } = await import('./fixtures/headers/dist/server/entry.mjs');
+
+	let { req, res, done } = createRequestAndResponse({
+		method: 'GET',
+		url,
+	});
+
+	handler(req, res);
+
+	req.send();
+
+	await done;
+	const headers = res.getHeaders();
+
+	expect(headers).to.deep.equal(expectedHeaders);
+}
-- 
cgit v1.2.3


From 4f010d71a1416b5ff5b956cd84bf7afe86bf0530 Mon Sep 17 00:00:00 2001
From: Emanuele Stoppa <my.burning@gmail.com>
Date: Thu, 25 Jan 2024 16:17:31 +0000
Subject: chore(@astrojs/node): use Node.js for testing (#9758)

* chore(@astrojs/node): use Node.js for testing

* revert file

* address feedback

* feedback

* Run tests in a single process (#9823)

* Run tests in a single process

* Make test less flaky

* chore: remove module

---------

Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com>
---
 packages/integrations/node/package.json            |  4 +-
 packages/integrations/node/test/api-route.test.js  | 13 ++--
 packages/integrations/node/test/assets.test.js     |  7 +-
 packages/integrations/node/test/bad-urls.test.js   | 11 ++-
 packages/integrations/node/test/encoded.test.js    |  7 +-
 packages/integrations/node/test/errors.test.js     | 29 ++++----
 packages/integrations/node/test/headers.test.js    |  5 +-
 packages/integrations/node/test/image.test.js      |  7 +-
 packages/integrations/node/test/locals.test.js     | 11 +--
 .../integrations/node/test/node-middleware.test.js | 14 ++--
 .../node/test/prerender-404-500.test.js            | 83 +++++++++++----------
 packages/integrations/node/test/prerender.test.js  | 85 ++++++++++++----------
 packages/integrations/node/test/test-utils.js      |  8 ++
 .../integrations/node/test/url-protocol.test.js    | 11 +--
 .../node/test/well-known-locations.test.js         |  9 ++-
 15 files changed, 166 insertions(+), 138 deletions(-)

(limited to 'packages/integrations/node/test/headers.test.js')

diff --git a/packages/integrations/node/package.json b/packages/integrations/node/package.json
index d23d1f339..d1811ce6f 100644
--- a/packages/integrations/node/package.json
+++ b/packages/integrations/node/package.json
@@ -30,7 +30,7 @@
     "build": "astro-scripts build \"src/**/*.ts\" && tsc",
     "build:ci": "astro-scripts build \"src/**/*.ts\"",
     "dev": "astro-scripts dev \"src/**/*.ts\"",
-    "test": "mocha --exit --timeout 20000 test/"
+    "test": "astro-scripts test \"test/**/*.test.js\""
   },
   "dependencies": {
     "send": "^0.18.0",
@@ -45,10 +45,8 @@
     "@types/server-destroy": "^1.0.3",
     "astro": "workspace:*",
     "astro-scripts": "workspace:*",
-    "chai": "^4.3.7",
     "cheerio": "1.0.0-rc.12",
     "express": "^4.18.2",
-    "mocha": "^10.2.0",
     "node-mocks-http": "^1.13.0"
   },
   "publishConfig": {
diff --git a/packages/integrations/node/test/api-route.test.js b/packages/integrations/node/test/api-route.test.js
index 313819188..80e99a2cc 100644
--- a/packages/integrations/node/test/api-route.test.js
+++ b/packages/integrations/node/test/api-route.test.js
@@ -1,7 +1,8 @@
 import nodejs from '../dist/index.js';
 import { loadFixture, createRequestAndResponse } from './test-utils.js';
-import { expect } from 'chai';
 import crypto from 'node:crypto';
+import { describe, it, before } from 'node:test';
+import * as assert from 'node:assert/strict';
 
 describe('API routes', () => {
 	/** @type {import('./test-utils').Fixture} */
@@ -33,9 +34,9 @@ describe('API routes', () => {
 
 		let json = JSON.parse(buffer.toString('utf-8'));
 
-		expect(json.length).to.equal(1);
+		assert.equal(json.length, 1);
 
-		expect(json[0].name).to.equal('Broccoli Soup');
+		assert.equal(json[0].name, 'Broccoli Soup');
 	});
 
 	it('Can get binary data', async () => {
@@ -54,7 +55,7 @@ describe('API routes', () => {
 
 		let [out] = await done;
 		let arr = Array.from(new Uint8Array(out.buffer));
-		expect(arr).to.deep.equal([5, 4, 3, 2, 1]);
+		assert.deepEqual(arr, [5, 4, 3, 2, 1]);
 	});
 
 	it('Can post large binary data', async () => {
@@ -87,7 +88,7 @@ describe('API routes', () => {
 		});
 
 		let [out] = await done;
-		expect(new Uint8Array(out.buffer)).to.deep.equal(expectedDigest);
+		assert.deepEqual(new Uint8Array(out.buffer), new Uint8Array(expectedDigest));
 	});
 
 	it('Can bail on streaming', async () => {
@@ -106,6 +107,6 @@ describe('API routes', () => {
 
 		await done;
 
-		expect(locals).to.deep.include({ cancelledByTheServer: true });
+		assert.deepEqual(locals, { cancelledByTheServer: true });
 	});
 });
diff --git a/packages/integrations/node/test/assets.test.js b/packages/integrations/node/test/assets.test.js
index 9e44ab31d..bcd9bb4bd 100644
--- a/packages/integrations/node/test/assets.test.js
+++ b/packages/integrations/node/test/assets.test.js
@@ -1,4 +1,5 @@
-import { expect } from 'chai';
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
 import * as cheerio from 'cheerio';
@@ -30,7 +31,7 @@ describe('Assets', () => {
 	it('Assets within the _astro folder should be given immutable headers', async () => {
 		let response = await fixture.fetch('/text-file');
 		let cacheControl = response.headers.get('cache-control');
-		expect(cacheControl).to.equal(null);
+		assert.equal(cacheControl, null);
 		const html = await response.text();
 		const $ = cheerio.load(html);
 
@@ -38,6 +39,6 @@ describe('Assets', () => {
 		const fileURL = $('a').attr('href');
 		response = await fixture.fetch(fileURL);
 		cacheControl = response.headers.get('cache-control');
-		expect(cacheControl).to.equal('public, max-age=31536000, immutable');
+		assert.equal(cacheControl, 'public, max-age=31536000, immutable');
 	});
 });
diff --git a/packages/integrations/node/test/bad-urls.test.js b/packages/integrations/node/test/bad-urls.test.js
index bfef81278..6d6c0a2e9 100644
--- a/packages/integrations/node/test/bad-urls.test.js
+++ b/packages/integrations/node/test/bad-urls.test.js
@@ -1,4 +1,5 @@
-import { expect } from 'chai';
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
 
@@ -32,15 +33,17 @@ describe('Bad URLs', () => {
 			'%20foobar%',
 		];
 
+		const statusCodes = [400, 404, 500];
 		for (const weirdUrl of weirdURLs) {
 			const fetchResult = await fixture.fetch(weirdUrl);
-			expect([400, 404, 500]).to.include(
-				fetchResult.status,
+			assert.equal(
+				statusCodes.includes(fetchResult.status),
+				true,
 				`${weirdUrl} returned something else than 400, 404, or 500`
 			);
 		}
 		const stillWork = await fixture.fetch('/');
 		const text = await stillWork.text();
-		expect(text).to.equal('<!DOCTYPE html>Hello!');
+		assert.equal(text, '<!DOCTYPE html>Hello!');
 	});
 });
diff --git a/packages/integrations/node/test/encoded.test.js b/packages/integrations/node/test/encoded.test.js
index bbd264777..2739fcfb7 100644
--- a/packages/integrations/node/test/encoded.test.js
+++ b/packages/integrations/node/test/encoded.test.js
@@ -1,6 +1,7 @@
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture, createRequestAndResponse } from './test-utils.js';
-import { expect } from 'chai';
 
 describe('Encoded Pathname', () => {
 	/** @type {import('./test-utils').Fixture} */
@@ -25,7 +26,7 @@ describe('Encoded Pathname', () => {
 		req.send();
 
 		const html = await text();
-		expect(html).to.include('什么</h1>');
+		assert.equal(html.includes('什么</h1>'), true);
 	});
 
 	it('Can get a Markdown file', async () => {
@@ -39,6 +40,6 @@ describe('Encoded Pathname', () => {
 		req.send();
 
 		const html = await text();
-		expect(html).to.include('什么</h1>');
+		assert.equal(html.includes('什么</h1>'), true);
 	});
 });
diff --git a/packages/integrations/node/test/errors.test.js b/packages/integrations/node/test/errors.test.js
index 6bb93023a..983187475 100644
--- a/packages/integrations/node/test/errors.test.js
+++ b/packages/integrations/node/test/errors.test.js
@@ -1,6 +1,7 @@
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
-import { expect } from 'chai';
 import * as cheerio from 'cheerio';
 
 describe('Errors', () => {
@@ -13,21 +14,19 @@ describe('Errors', () => {
 		});
 		await fixture.build();
 	});
-	describe('Within the stream', async () => {
-		let devPreview;
+	let devPreview;
 
-		before(async () => {
-			devPreview = await fixture.preview();
-		});
-		after(async () => {
-			await devPreview.stop();
-		});
-		it('when mode is standalone', async () => {
-			const res = await fixture.fetch('/in-stream');
-			const html = await res.text();
-			const $ = cheerio.load(html);
+	before(async () => {
+		devPreview = await fixture.preview();
+	});
+	after(async () => {
+		await devPreview.stop();
+	});
+	it('when mode is standalone', async () => {
+		const res = await fixture.fetch('/in-stream');
+		const html = await res.text();
+		const $ = cheerio.load(html);
 
-			expect($('p').text().trim()).to.equal('Internal server error');
-		});
+		assert.equal($('p').text().trim(), 'Internal server error');
 	});
 });
diff --git a/packages/integrations/node/test/headers.test.js b/packages/integrations/node/test/headers.test.js
index 17cfd3701..6a08dca22 100644
--- a/packages/integrations/node/test/headers.test.js
+++ b/packages/integrations/node/test/headers.test.js
@@ -1,6 +1,7 @@
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture, createRequestAndResponse } from './test-utils.js';
-import { expect } from 'chai';
 
 describe('Node Adapter Headers', () => {
 	/** @type {import('./test-utils').Fixture} */
@@ -143,5 +144,5 @@ async function runTest(url, expectedHeaders) {
 	await done;
 	const headers = res.getHeaders();
 
-	expect(headers).to.deep.equal(expectedHeaders);
+	assert.deepEqual(headers, expectedHeaders);
 }
diff --git a/packages/integrations/node/test/image.test.js b/packages/integrations/node/test/image.test.js
index 7bbdadc12..b315c1a30 100644
--- a/packages/integrations/node/test/image.test.js
+++ b/packages/integrations/node/test/image.test.js
@@ -1,4 +1,5 @@
-import { expect } from 'chai';
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
 
@@ -24,12 +25,12 @@ describe.skip('Image endpoint', () => {
 
 	it('it returns images', async () => {
 		const res = await fixture.fetch('/');
-		expect(res.status).to.equal(200);
+		assert.equal(res.status, 200);
 
 		const resImage = await fixture.fetch(
 			'/_image?href=/_astro/some_penguin.97ef5f92.png&w=50&f=webp'
 		);
 
-		expect(resImage.status).to.equal(200);
+		assert.equal(resImage.status, 200);
 	});
 });
diff --git a/packages/integrations/node/test/locals.test.js b/packages/integrations/node/test/locals.test.js
index bba8b2d34..e2a531cce 100644
--- a/packages/integrations/node/test/locals.test.js
+++ b/packages/integrations/node/test/locals.test.js
@@ -1,6 +1,7 @@
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture, createRequestAndResponse } from './test-utils.js';
-import { expect } from 'chai';
 
 describe('API routes', () => {
 	/** @type {import('./test-utils').Fixture} */
@@ -28,7 +29,7 @@ describe('API routes', () => {
 
 		let html = await text();
 
-		expect(html).to.contain('<h1>bar</h1>');
+		assert.equal(html.includes('<h1>bar</h1>'), true);
 	});
 
 	it('Throws an error when provided non-objects as locals', async () => {
@@ -41,7 +42,7 @@ describe('API routes', () => {
 		req.send();
 
 		await done;
-		expect(res).to.deep.include({ statusCode: 500 });
+		assert.equal(res.statusCode, 500);
 	});
 
 	it('Can use locals added by astro middleware', async () => {
@@ -56,7 +57,7 @@ describe('API routes', () => {
 
 		const html = await text();
 
-		expect(html).to.contain('<h1>baz</h1>');
+		assert.equal(html.includes('<h1>baz</h1>'), true);
 	});
 
 	it('Can access locals in API', async () => {
@@ -75,6 +76,6 @@ describe('API routes', () => {
 
 		let json = JSON.parse(buffer.toString('utf-8'));
 
-		expect(json.foo).to.equal('bar');
+		assert.equal(json.foo, 'bar');
 	});
 });
diff --git a/packages/integrations/node/test/node-middleware.test.js b/packages/integrations/node/test/node-middleware.test.js
index 6b6785953..889f72315 100644
--- a/packages/integrations/node/test/node-middleware.test.js
+++ b/packages/integrations/node/test/node-middleware.test.js
@@ -1,6 +1,7 @@
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
-import { loadFixture } from './test-utils.js';
-import { expect } from 'chai';
+import { loadFixture, waitServerListen } from './test-utils.js';
 import * as cheerio from 'cheerio';
 import express from 'express';
 
@@ -31,6 +32,7 @@ describe('behavior from middleware, standalone', () => {
 		const { startServer } = await load();
 		let res = startServer();
 		server = res.server;
+		await waitServerListen(server.server);
 	});
 
 	after(async () => {
@@ -43,13 +45,13 @@ describe('behavior from middleware, standalone', () => {
 		it('when mode is standalone', async () => {
 			const res = await fetch(`http://${server.host}:${server.port}/error-page`);
 
-			expect(res.status).to.equal(404);
+			assert.equal(res.status, 404);
 
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
 			const body = $('body');
-			expect(body.text()).to.equal('Page does not exist');
+			assert.equal(body.text().includes('Page does not exist'), true);
 		});
 	});
 });
@@ -82,12 +84,12 @@ describe('behavior from middleware, middleware', () => {
 	it('when mode is standalone', async () => {
 		const res = await fetch(`http://localhost:8888/ssr`);
 
-		expect(res.status).to.equal(200);
+		assert.equal(res.status, 200);
 
 		const html = await res.text();
 		const $ = cheerio.load(html);
 
 		const body = $('body');
-		expect(body.text()).to.contain("Here's a random number");
+		assert.equal(body.text().includes("Here's a random number"), true);
 	});
 });
diff --git a/packages/integrations/node/test/prerender-404-500.test.js b/packages/integrations/node/test/prerender-404-500.test.js
index 745a1958c..4195db0ec 100644
--- a/packages/integrations/node/test/prerender-404-500.test.js
+++ b/packages/integrations/node/test/prerender-404-500.test.js
@@ -1,6 +1,7 @@
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
-import { loadFixture } from './test-utils.js';
-import { expect } from 'chai';
+import { loadFixture, waitServerListen } from './test-utils.js';
 import * as cheerio from 'cheerio';
 
 /**
@@ -37,6 +38,7 @@ describe('Prerender 404', () => {
 			const { startServer } = await load();
 			let res = startServer();
 			server = res.server;
+			await waitServerListen(server.server);
 		});
 
 		after(async () => {
@@ -50,8 +52,8 @@ describe('Prerender 404', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Hello world!');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Hello world!');
 		});
 
 		it('Can handle prerendered 404', async () => {
@@ -60,20 +62,20 @@ describe('Prerender 404', () => {
 			const res2 = await fetch(url);
 			const res3 = await fetch(url);
 
-			expect(res1.status).to.equal(404);
-			expect(res2.status).to.equal(404);
-			expect(res3.status).to.equal(404);
+			assert.equal(res1.status, 404);
+			assert.equal(res2.status, 404);
+			assert.equal(res3.status, 404);
 
 			const html1 = await res1.text();
 			const html2 = await res2.text();
 			const html3 = await res3.text();
 
-			expect(html1).to.equal(html2);
-			expect(html2).to.equal(html3);
+			assert.equal(html1, html2);
+			assert.equal(html2, html3);
 
 			const $ = cheerio.load(html1);
 
-			expect($('body').text()).to.equal('Page does not exist');
+			assert.equal($('body').text(), 'Page does not exist');
 		});
 
 		it(' Can handle prerendered 500 called indirectly', async () => {
@@ -82,16 +84,16 @@ describe('Prerender 404', () => {
 			const response2 = await fetch(url);
 			const response3 = await fetch(url);
 
-			expect(response1.status).to.equal(500);
+			assert.equal(response1.status, 500);
 
 			const html1 = await response1.text();
 			const html2 = await response2.text();
 			const html3 = await response3.text();
 
-			expect(html1).to.contain('Something went wrong');
+			assert.equal(html1.includes('Something went wrong'), true);
 
-			expect(html1).to.equal(html2);
-			expect(html2).to.equal(html3);
+			assert.equal(html1, html2);
+			assert.equal(html2, html3);
 		});
 
 		it('prerendered 500 page includes expected styles', async () => {
@@ -100,7 +102,7 @@ describe('Prerender 404', () => {
 			const $ = cheerio.load(html);
 
 			// length will be 0 if the stylesheet does not get included
-			expect($('style')).to.have.a.lengthOf(1);
+			assert.equal($('style').length, 1);
 		});
 	});
 
@@ -121,6 +123,7 @@ describe('Prerender 404', () => {
 			const { startServer } = await load();
 			let res = startServer();
 			server = res.server;
+			await waitServerListen(server.server);
 		});
 
 		after(async () => {
@@ -134,8 +137,8 @@ describe('Prerender 404', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Hello world!');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Hello world!');
 		});
 
 		it('Can handle prerendered 404', async () => {
@@ -144,20 +147,20 @@ describe('Prerender 404', () => {
 			const res2 = await fetch(url);
 			const res3 = await fetch(url);
 
-			expect(res1.status).to.equal(404);
-			expect(res2.status).to.equal(404);
-			expect(res3.status).to.equal(404);
+			assert.equal(res1.status, 404);
+			assert.equal(res2.status, 404);
+			assert.equal(res3.status, 404);
 
 			const html1 = await res1.text();
 			const html2 = await res2.text();
 			const html3 = await res3.text();
 
-			expect(html1).to.equal(html2);
-			expect(html2).to.equal(html3);
+			assert.equal(html1, html2);
+			assert.equal(html2, html3);
 
 			const $ = cheerio.load(html1);
 
-			expect($('body').text()).to.equal('Page does not exist');
+			assert.equal($('body').text(), 'Page does not exist');
 		});
 	});
 });
@@ -184,6 +187,7 @@ describe('Hybrid 404', () => {
 			const { startServer } = await load();
 			let res = startServer();
 			server = res.server;
+			await waitServerListen(server.server);
 		});
 
 		after(async () => {
@@ -197,8 +201,8 @@ describe('Hybrid 404', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Hello world!');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Hello world!');
 		});
 
 		it('Can handle prerendered 404', async () => {
@@ -207,20 +211,20 @@ describe('Hybrid 404', () => {
 			const res2 = await fetch(url);
 			const res3 = await fetch(url);
 
-			expect(res1.status).to.equal(404);
-			expect(res2.status).to.equal(404);
-			expect(res3.status).to.equal(404);
+			assert.equal(res1.status, 404);
+			assert.equal(res2.status, 404);
+			assert.equal(res3.status, 404);
 
 			const html1 = await res1.text();
 			const html2 = await res2.text();
 			const html3 = await res3.text();
 
-			expect(html1).to.equal(html2);
-			expect(html2).to.equal(html3);
+			assert.equal(html1, html2);
+			assert.equal(html2, html3);
 
 			const $ = cheerio.load(html1);
 
-			expect($('body').text()).to.equal('Page does not exist');
+			assert.equal($('body').text(), 'Page does not exist');
 		});
 	});
 
@@ -240,6 +244,7 @@ describe('Hybrid 404', () => {
 			const { startServer } = await load();
 			let res = startServer();
 			server = res.server;
+			await waitServerListen(server.server);
 		});
 
 		after(async () => {
@@ -253,8 +258,8 @@ describe('Hybrid 404', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Hello world!');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Hello world!');
 		});
 
 		it('Can handle prerendered 404', async () => {
@@ -263,20 +268,20 @@ describe('Hybrid 404', () => {
 			const res2 = await fetch(url);
 			const res3 = await fetch(url);
 
-			expect(res1.status).to.equal(404);
-			expect(res2.status).to.equal(404);
-			expect(res3.status).to.equal(404);
+			assert.equal(res1.status, 404);
+			assert.equal(res2.status, 404);
+			assert.equal(res3.status, 404);
 
 			const html1 = await res1.text();
 			const html2 = await res2.text();
 			const html3 = await res3.text();
 
-			expect(html1).to.equal(html2);
-			expect(html2).to.equal(html3);
+			assert.equal(html1, html2);
+			assert.equal(html2, html3);
 
 			const $ = cheerio.load(html1);
 
-			expect($('body').text()).to.equal('Page does not exist');
+			assert.equal($('body').text(), 'Page does not exist');
 		});
 	});
 });
diff --git a/packages/integrations/node/test/prerender.test.js b/packages/integrations/node/test/prerender.test.js
index 86a7d3a65..ffa5563fa 100644
--- a/packages/integrations/node/test/prerender.test.js
+++ b/packages/integrations/node/test/prerender.test.js
@@ -1,6 +1,7 @@
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
-import { loadFixture } from './test-utils.js';
-import { expect } from 'chai';
+import { loadFixture, waitServerListen } from './test-utils.js';
 import * as cheerio from 'cheerio';
 
 /**
@@ -30,6 +31,7 @@ describe('Prerendering', () => {
 			const { startServer } = await load();
 			let res = startServer();
 			server = res.server;
+			await waitServerListen(server.server);
 		});
 
 		after(async () => {
@@ -43,8 +45,8 @@ describe('Prerendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('One');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'One');
 		});
 
 		it('Can render prerendered route', async () => {
@@ -52,8 +54,8 @@ describe('Prerendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Two');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Two');
 		});
 
 		it('Can render prerendered route with redirect and query params', async () => {
@@ -61,8 +63,8 @@ describe('Prerendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Two');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Two');
 		});
 
 		it('Can render prerendered route with query params', async () => {
@@ -70,8 +72,8 @@ describe('Prerendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Two');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Two');
 		});
 
 		it('Can render prerendered route without trailing slash', async () => {
@@ -80,8 +82,9 @@ describe('Prerendering', () => {
 			});
 			const html = await res.text();
 			const $ = cheerio.load(html);
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Two');
+			assert.equal(res.status, 301);
+			assert.equal(res.headers.get('location'), '/some-base/two/');
+			assert.equal($('h1').text(), "Two")
 		});
 	});
 
@@ -98,6 +101,7 @@ describe('Prerendering', () => {
 			const { startServer } = await await load();
 			let res = startServer();
 			server = res.server;
+			await waitServerListen(server.server);
 		});
 
 		after(async () => {
@@ -111,8 +115,8 @@ describe('Prerendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('One');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'One');
 		});
 
 		it('Can render prerendered route', async () => {
@@ -120,8 +124,8 @@ describe('Prerendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Two');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Two');
 		});
 
 		it('Can render prerendered route with redirect and query params', async () => {
@@ -129,8 +133,8 @@ describe('Prerendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Two');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Two');
 		});
 
 		it('Can render prerendered route with query params', async () => {
@@ -138,8 +142,8 @@ describe('Prerendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Two');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Two');
 		});
 	});
 
@@ -200,6 +204,7 @@ describe('Hybrid rendering', () => {
 			const { startServer } = await await load();
 			let res = startServer();
 			server = res.server;
+			await waitServerListen(server.server);
 		});
 
 		after(async () => {
@@ -212,8 +217,8 @@ describe('Hybrid rendering', () => {
 			const res = await fetch(`http://${server.host}:${server.port}/some-base/two`);
 			const html = await res.text();
 			const $ = cheerio.load(html);
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Two');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Two');
 		});
 
 		it('Can render prerendered route', async () => {
@@ -221,8 +226,8 @@ describe('Hybrid rendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('One');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'One');
 		});
 
 		it('Can render prerendered route with redirect and query params', async () => {
@@ -230,8 +235,8 @@ describe('Hybrid rendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('One');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'One');
 		});
 
 		it('Can render prerendered route with query params', async () => {
@@ -239,18 +244,17 @@ describe('Hybrid rendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('One');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'One');
 		});
 
 		it('Can render prerendered route without trailing slash', async () => {
 			const res = await fetch(`http://${server.host}:${server.port}/some-base/one`, {
 				redirect: 'manual',
 			});
-			const html = await res.text();
-			const $ = cheerio.load(html);
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('One');
+			assert.equal(res.status, 301);
+			assert.equal(res.headers.get('location'), '/some-base/one/');
+			assert.equal($('h1').text(), 'One');
 		});
 	});
 
@@ -266,6 +270,7 @@ describe('Hybrid rendering', () => {
 			const { startServer } = await await load();
 			let res = startServer();
 			server = res.server;
+			await waitServerListen(server.server);
 		});
 
 		after(async () => {
@@ -279,8 +284,8 @@ describe('Hybrid rendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('Two');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'Two');
 		});
 
 		it('Can render prerendered route', async () => {
@@ -288,8 +293,8 @@ describe('Hybrid rendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('One');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'One');
 		});
 
 		it('Can render prerendered route with redirect and query params', async () => {
@@ -297,8 +302,8 @@ describe('Hybrid rendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('One');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'One');
 		});
 
 		it('Can render prerendered route with query params', async () => {
@@ -306,8 +311,8 @@ describe('Hybrid rendering', () => {
 			const html = await res.text();
 			const $ = cheerio.load(html);
 
-			expect(res.status).to.equal(200);
-			expect($('h1').text()).to.equal('One');
+			assert.equal(res.status, 200);
+			assert.equal($('h1').text(), 'One');
 		});
 	});
 });
diff --git a/packages/integrations/node/test/test-utils.js b/packages/integrations/node/test/test-utils.js
index 8d830fee9..3d582f30b 100644
--- a/packages/integrations/node/test/test-utils.js
+++ b/packages/integrations/node/test/test-utils.js
@@ -64,3 +64,11 @@ export function buffersToString(buffers) {
 	}
 	return str;
 }
+
+export function waitServerListen(server) {
+	return new Promise((resolve) => {
+		server.on('listening', () => {
+			resolve();
+		});
+	});
+}
diff --git a/packages/integrations/node/test/url-protocol.test.js b/packages/integrations/node/test/url-protocol.test.js
index a83cb2a41..444d47ed5 100644
--- a/packages/integrations/node/test/url-protocol.test.js
+++ b/packages/integrations/node/test/url-protocol.test.js
@@ -1,4 +1,5 @@
-import { expect } from 'chai';
+import * as assert from 'node:assert/strict';
+import { describe, it, before } from 'node:test';
 import { TLSSocket } from 'node:tls';
 import nodejs from '../dist/index.js';
 import { createRequestAndResponse, loadFixture } from './test-utils.js';
@@ -26,7 +27,7 @@ describe('URL protocol', () => {
 		req.send();
 
 		const html = await text();
-		expect(html).to.include('http:');
+		assert.equal(html.includes('http:'), true);
 	});
 
 	it('return https when secure', async () => {
@@ -40,7 +41,7 @@ describe('URL protocol', () => {
 		req.send();
 
 		const html = await text();
-		expect(html).to.include('https:');
+		assert.equal(html.includes('https:'), true);
 	});
 
 	it('return http when the X-Forwarded-Proto header is set to http', async () => {
@@ -54,7 +55,7 @@ describe('URL protocol', () => {
 		req.send();
 
 		const html = await text();
-		expect(html).to.include('http:');
+		assert.equal(html.includes('http:'), true);
 	});
 
 	it('return https when the X-Forwarded-Proto header is set to https', async () => {
@@ -68,6 +69,6 @@ describe('URL protocol', () => {
 		req.send();
 
 		const html = await text();
-		expect(html).to.include('https:');
+		assert.equal(html.includes('https:'), true);
 	});
 });
diff --git a/packages/integrations/node/test/well-known-locations.test.js b/packages/integrations/node/test/well-known-locations.test.js
index 31f31bacd..934673cda 100644
--- a/packages/integrations/node/test/well-known-locations.test.js
+++ b/packages/integrations/node/test/well-known-locations.test.js
@@ -1,6 +1,7 @@
+import * as assert from 'node:assert/strict';
+import { describe, it, before, after } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
-import { expect } from 'chai';
 
 describe('test URIs beginning with a dot', () => {
 	/** @type {import('./test-utils').Fixture} */
@@ -29,17 +30,17 @@ describe('test URIs beginning with a dot', () => {
 		it('can load a valid well-known URI', async () => {
 			const res = await fixture.fetch('/.well-known/apple-app-site-association');
 
-			expect(res.status).to.equal(200);
+			assert.equal(res.status, 200);
 
 			const json = await res.json();
 
-			expect(json).to.deep.equal({ applinks: {} });
+			assert.notStrictEqual(json.applinks, {});
 		});
 
 		it('cannot load a dot folder that is not a well-known URI', async () => {
 			const res = await fixture.fetch('/.hidden/file.json');
 
-			expect(res.status).to.equal(404);
+			assert.equal(res.status, 404);
 		});
 	});
 });
-- 
cgit v1.2.3


From 6f233ca630798b2a2d1f9cafc326161cc6de89f1 Mon Sep 17 00:00:00 2001
From: Emanuele Stoppa <my.burning@gmail.com>
Date: Tue, 30 Jan 2024 08:35:08 +0000
Subject: chore: add rule `switch-exhaustiveness-check` (#9867)

* chore: add rule `switch-exhaustiveness-check`

* apply feedback

* routing is always defined
---
 packages/integrations/node/test/encoded.test.js | 2 +-
 packages/integrations/node/test/headers.test.js | 2 +-
 packages/integrations/node/test/locals.test.js  | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

(limited to 'packages/integrations/node/test/headers.test.js')

diff --git a/packages/integrations/node/test/encoded.test.js b/packages/integrations/node/test/encoded.test.js
index 2739fcfb7..15b5654b1 100644
--- a/packages/integrations/node/test/encoded.test.js
+++ b/packages/integrations/node/test/encoded.test.js
@@ -1,5 +1,5 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { describe, it, before } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture, createRequestAndResponse } from './test-utils.js';
 
diff --git a/packages/integrations/node/test/headers.test.js b/packages/integrations/node/test/headers.test.js
index 6a08dca22..b74af9952 100644
--- a/packages/integrations/node/test/headers.test.js
+++ b/packages/integrations/node/test/headers.test.js
@@ -1,5 +1,5 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { describe, it, before } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture, createRequestAndResponse } from './test-utils.js';
 
diff --git a/packages/integrations/node/test/locals.test.js b/packages/integrations/node/test/locals.test.js
index e2a531cce..a310c729d 100644
--- a/packages/integrations/node/test/locals.test.js
+++ b/packages/integrations/node/test/locals.test.js
@@ -1,5 +1,5 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { describe, it, before } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture, createRequestAndResponse } from './test-utils.js';
 
-- 
cgit v1.2.3


From 7106b81b28ebc782ee886d87dd83f4acb72d0850 Mon Sep 17 00:00:00 2001
From: Emanuele Stoppa <my.burning@gmail.com>
Date: Wed, 21 Feb 2024 14:08:19 +0000
Subject: chore: use biome to sort imports - only test files (#10180)

* chore: use biome to sort imports

* do the sorting

* Update package.json

Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com>

---------

Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com>
---
 packages/integrations/node/test/api-route.test.js            | 8 ++++----
 packages/integrations/node/test/assets.test.js               | 4 ++--
 packages/integrations/node/test/bad-urls.test.js             | 2 +-
 packages/integrations/node/test/encoded.test.js              | 4 ++--
 packages/integrations/node/test/errors.test.js               | 4 ++--
 packages/integrations/node/test/headers.test.js              | 4 ++--
 packages/integrations/node/test/image.test.js                | 2 +-
 packages/integrations/node/test/locals.test.js               | 4 ++--
 packages/integrations/node/test/node-middleware.test.js      | 6 +++---
 packages/integrations/node/test/prerender-404-500.test.js    | 4 ++--
 packages/integrations/node/test/prerender.test.js            | 4 ++--
 packages/integrations/node/test/url-protocol.test.js         | 2 +-
 packages/integrations/node/test/well-known-locations.test.js | 2 +-
 13 files changed, 25 insertions(+), 25 deletions(-)

(limited to 'packages/integrations/node/test/headers.test.js')

diff --git a/packages/integrations/node/test/api-route.test.js b/packages/integrations/node/test/api-route.test.js
index 9e0165cb9..804a5ccf4 100644
--- a/packages/integrations/node/test/api-route.test.js
+++ b/packages/integrations/node/test/api-route.test.js
@@ -1,8 +1,8 @@
-import nodejs from '../dist/index.js';
-import { loadFixture, createRequestAndResponse } from './test-utils.js';
-import crypto from 'node:crypto';
-import { describe, it, before, after } from 'node:test';
 import * as assert from 'node:assert/strict';
+import crypto from 'node:crypto';
+import { after, before, describe, it } from 'node:test';
+import nodejs from '../dist/index.js';
+import { createRequestAndResponse, loadFixture } from './test-utils.js';
 
 describe('API routes', () => {
 	/** @type {import('./test-utils').Fixture} */
diff --git a/packages/integrations/node/test/assets.test.js b/packages/integrations/node/test/assets.test.js
index bcd9bb4bd..0b71f94cd 100644
--- a/packages/integrations/node/test/assets.test.js
+++ b/packages/integrations/node/test/assets.test.js
@@ -1,8 +1,8 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { after, before, describe, it } from 'node:test';
+import * as cheerio from 'cheerio';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
-import * as cheerio from 'cheerio';
 
 describe('Assets', () => {
 	/** @type {import('./test-utils').Fixture} */
diff --git a/packages/integrations/node/test/bad-urls.test.js b/packages/integrations/node/test/bad-urls.test.js
index 6d6c0a2e9..cdc0158ff 100644
--- a/packages/integrations/node/test/bad-urls.test.js
+++ b/packages/integrations/node/test/bad-urls.test.js
@@ -1,5 +1,5 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { after, before, describe, it } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
 
diff --git a/packages/integrations/node/test/encoded.test.js b/packages/integrations/node/test/encoded.test.js
index 15b5654b1..edc6ae78b 100644
--- a/packages/integrations/node/test/encoded.test.js
+++ b/packages/integrations/node/test/encoded.test.js
@@ -1,7 +1,7 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before } from 'node:test';
+import { before, describe, it } from 'node:test';
 import nodejs from '../dist/index.js';
-import { loadFixture, createRequestAndResponse } from './test-utils.js';
+import { createRequestAndResponse, loadFixture } from './test-utils.js';
 
 describe('Encoded Pathname', () => {
 	/** @type {import('./test-utils').Fixture} */
diff --git a/packages/integrations/node/test/errors.test.js b/packages/integrations/node/test/errors.test.js
index c4212f058..95bb1be85 100644
--- a/packages/integrations/node/test/errors.test.js
+++ b/packages/integrations/node/test/errors.test.js
@@ -1,8 +1,8 @@
 import assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { after, before, describe, it } from 'node:test';
+import * as cheerio from 'cheerio';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
-import * as cheerio from 'cheerio';
 
 describe('Errors', () => {
 	let fixture;
diff --git a/packages/integrations/node/test/headers.test.js b/packages/integrations/node/test/headers.test.js
index b74af9952..00b1766c7 100644
--- a/packages/integrations/node/test/headers.test.js
+++ b/packages/integrations/node/test/headers.test.js
@@ -1,7 +1,7 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before } from 'node:test';
+import { before, describe, it } from 'node:test';
 import nodejs from '../dist/index.js';
-import { loadFixture, createRequestAndResponse } from './test-utils.js';
+import { createRequestAndResponse, loadFixture } from './test-utils.js';
 
 describe('Node Adapter Headers', () => {
 	/** @type {import('./test-utils').Fixture} */
diff --git a/packages/integrations/node/test/image.test.js b/packages/integrations/node/test/image.test.js
index b315c1a30..c4758f96b 100644
--- a/packages/integrations/node/test/image.test.js
+++ b/packages/integrations/node/test/image.test.js
@@ -1,5 +1,5 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { after, before, describe, it } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
 
diff --git a/packages/integrations/node/test/locals.test.js b/packages/integrations/node/test/locals.test.js
index a310c729d..6d2776079 100644
--- a/packages/integrations/node/test/locals.test.js
+++ b/packages/integrations/node/test/locals.test.js
@@ -1,7 +1,7 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before } from 'node:test';
+import { before, describe, it } from 'node:test';
 import nodejs from '../dist/index.js';
-import { loadFixture, createRequestAndResponse } from './test-utils.js';
+import { createRequestAndResponse, loadFixture } from './test-utils.js';
 
 describe('API routes', () => {
 	/** @type {import('./test-utils').Fixture} */
diff --git a/packages/integrations/node/test/node-middleware.test.js b/packages/integrations/node/test/node-middleware.test.js
index 889f72315..fa496e871 100644
--- a/packages/integrations/node/test/node-middleware.test.js
+++ b/packages/integrations/node/test/node-middleware.test.js
@@ -1,9 +1,9 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
-import nodejs from '../dist/index.js';
-import { loadFixture, waitServerListen } from './test-utils.js';
+import { after, before, describe, it } from 'node:test';
 import * as cheerio from 'cheerio';
 import express from 'express';
+import nodejs from '../dist/index.js';
+import { loadFixture, waitServerListen } from './test-utils.js';
 
 /**
  * @typedef {import('../../../astro/test/test-utils').Fixture} Fixture
diff --git a/packages/integrations/node/test/prerender-404-500.test.js b/packages/integrations/node/test/prerender-404-500.test.js
index 4195db0ec..afa23a151 100644
--- a/packages/integrations/node/test/prerender-404-500.test.js
+++ b/packages/integrations/node/test/prerender-404-500.test.js
@@ -1,8 +1,8 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { after, before, describe, it } from 'node:test';
+import * as cheerio from 'cheerio';
 import nodejs from '../dist/index.js';
 import { loadFixture, waitServerListen } from './test-utils.js';
-import * as cheerio from 'cheerio';
 
 /**
  * @typedef {import('../../../astro/test/test-utils').Fixture} Fixture
diff --git a/packages/integrations/node/test/prerender.test.js b/packages/integrations/node/test/prerender.test.js
index de570511e..bf34fc99d 100644
--- a/packages/integrations/node/test/prerender.test.js
+++ b/packages/integrations/node/test/prerender.test.js
@@ -1,8 +1,8 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { after, before, describe, it } from 'node:test';
+import * as cheerio from 'cheerio';
 import nodejs from '../dist/index.js';
 import { loadFixture, waitServerListen } from './test-utils.js';
-import * as cheerio from 'cheerio';
 
 /**
  * @typedef {import('../../../astro/test/test-utils').Fixture} Fixture
diff --git a/packages/integrations/node/test/url-protocol.test.js b/packages/integrations/node/test/url-protocol.test.js
index 444d47ed5..94d53104b 100644
--- a/packages/integrations/node/test/url-protocol.test.js
+++ b/packages/integrations/node/test/url-protocol.test.js
@@ -1,5 +1,5 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before } from 'node:test';
+import { before, describe, it } from 'node:test';
 import { TLSSocket } from 'node:tls';
 import nodejs from '../dist/index.js';
 import { createRequestAndResponse, loadFixture } from './test-utils.js';
diff --git a/packages/integrations/node/test/well-known-locations.test.js b/packages/integrations/node/test/well-known-locations.test.js
index 39d7ccedb..0951d6c27 100644
--- a/packages/integrations/node/test/well-known-locations.test.js
+++ b/packages/integrations/node/test/well-known-locations.test.js
@@ -1,5 +1,5 @@
 import * as assert from 'node:assert/strict';
-import { describe, it, before, after } from 'node:test';
+import { after, before, describe, it } from 'node:test';
 import nodejs from '../dist/index.js';
 import { loadFixture } from './test-utils.js';
 
-- 
cgit v1.2.3


From 1b6f89c5b4b7a3e3c9cc0ca639e0a5dbfca01952 Mon Sep 17 00:00:00 2001
From: Alexander Niebuhr <alexander@nbhr.io>
Date: Thu, 29 Aug 2024 08:17:42 +0200
Subject: chore: supress linting

---
 packages/integrations/node/CHANGELOG.md             |  2 +-
 packages/integrations/node/package.json             |  9 ++-------
 packages/integrations/node/src/index.ts             |  2 +-
 packages/integrations/node/src/log-listening-on.ts  | 17 +++++++++++------
 packages/integrations/node/src/middleware.ts        |  4 +++-
 packages/integrations/node/src/preview.ts           | 10 ++++++----
 packages/integrations/node/src/serve-app.ts         |  2 +-
 packages/integrations/node/src/serve-static.ts      | 12 +++++++++++-
 packages/integrations/node/src/standalone.ts        |  3 ++-
 packages/integrations/node/test/api-route.test.js   | 20 ++++++++++----------
 packages/integrations/node/test/bad-urls.test.js    |  2 +-
 packages/integrations/node/test/encoded.test.js     |  4 ++--
 packages/integrations/node/test/errors.test.js      |  5 +++--
 packages/integrations/node/test/headers.test.js     |  2 +-
 packages/integrations/node/test/image.test.js       |  2 +-
 packages/integrations/node/test/locals.test.js      | 16 ++++++++--------
 .../integrations/node/test/node-middleware.test.js  |  5 ++++-
 .../node/test/prerender-404-500.test.js             | 12 ++++++++----
 packages/integrations/node/test/prerender.test.js   | 21 +++++++++++++++------
 packages/integrations/node/test/test-utils.js       | 13 +++++++------
 .../integrations/node/test/trailing-slash.test.js   | 18 ++++++++++++------
 packages/integrations/node/test/url.test.js         | 12 ++++++------
 packages/integrations/node/tsconfig.json            |  4 +---
 23 files changed, 117 insertions(+), 80 deletions(-)

(limited to 'packages/integrations/node/test/headers.test.js')

diff --git a/packages/integrations/node/CHANGELOG.md b/packages/integrations/node/CHANGELOG.md
index 738cfc367..ed830f8de 100644
--- a/packages/integrations/node/CHANGELOG.md
+++ b/packages/integrations/node/CHANGELOG.md
@@ -734,7 +734,7 @@
     { darkMode: true },
     {
       expires: '1 month',
-    },
+    }
   );
 
   const prefs = Astro.cookies.get<Prefs>('prefs').json();
diff --git a/packages/integrations/node/package.json b/packages/integrations/node/package.json
index f8042499b..2fc6c35b1 100644
--- a/packages/integrations/node/package.json
+++ b/packages/integrations/node/package.json
@@ -11,10 +11,7 @@
     "url": "https://github.com/withastro/astro.git",
     "directory": "packages/integrations/node"
   },
-  "keywords": [
-    "withastro",
-    "astro-adapter"
-  ],
+  "keywords": ["withastro", "astro-adapter"],
   "bugs": "https://github.com/withastro/astro/issues",
   "homepage": "https://docs.astro.build/en/guides/integrations-guide/node/",
   "exports": {
@@ -23,9 +20,7 @@
     "./preview.js": "./dist/preview.js",
     "./package.json": "./package.json"
   },
-  "files": [
-    "dist"
-  ],
+  "files": ["dist"],
   "scripts": {
     "build": "tsc",
     "test": "astro-scripts test \"test/**/*.test.js\""
diff --git a/packages/integrations/node/src/index.ts b/packages/integrations/node/src/index.ts
index 36d9ee30f..eb3c98a9b 100644
--- a/packages/integrations/node/src/index.ts
+++ b/packages/integrations/node/src/index.ts
@@ -73,7 +73,7 @@ export default function createIntegration(userOptions: UserOptions): AstroIntegr
 
 				if (config.output === 'static') {
 					logger.warn(
-						`\`output: "server"\` or  \`output: "hybrid"\` is required to use this adapter.`,
+						`\`output: "server"\` or  \`output: "hybrid"\` is required to use this adapter.`
 					);
 				}
 			},
diff --git a/packages/integrations/node/src/log-listening-on.ts b/packages/integrations/node/src/log-listening-on.ts
index 7e299740c..2f774c31c 100644
--- a/packages/integrations/node/src/log-listening-on.ts
+++ b/packages/integrations/node/src/log-listening-on.ts
@@ -8,20 +8,20 @@ import type { Options } from './types.js';
 export async function logListeningOn(
 	logger: AstroIntegrationLogger,
 	server: http.Server | https.Server,
-	options: Pick<Options, 'host'>,
+	options: Pick<Options, 'host'>
 ) {
 	await new Promise<void>((resolve) => server.once('listening', resolve));
 	const protocol = server instanceof https.Server ? 'https' : 'http';
 	// Allow to provide host value at runtime
 	const host = getResolvedHostForHttpServer(
-		process.env.HOST !== undefined && process.env.HOST !== '' ? process.env.HOST : options.host,
+		process.env.HOST !== undefined && process.env.HOST !== '' ? process.env.HOST : options.host
 	);
 	const { port } = server.address() as AddressInfo;
 	const address = getNetworkAddress(protocol, host, port);
 
 	if (host === undefined) {
 		logger.info(
-			`Server listening on \n  local: ${address.local[0]} \t\n  network: ${address.network[0]}\n`,
+			`Server listening on \n  local: ${address.local[0]} \t\n  network: ${address.network[0]}\n`
 		);
 	} else {
 		logger.info(`Server listening on ${address.local[0]}`);
@@ -32,9 +32,11 @@ function getResolvedHostForHttpServer(host: string | boolean) {
 	if (host === false) {
 		// Use a secure default
 		return 'localhost';
+		// biome-ignore lint/style/noUselessElse: <explanation>
 	} else if (host === true) {
 		// If passed --host in the CLI without arguments
 		return undefined; // undefined typically means 0.0.0.0 or :: (listen on all IPs)
+		// biome-ignore lint/style/noUselessElse: <explanation>
 	} else {
 		return host;
 	}
@@ -49,29 +51,32 @@ const wildcardHosts = new Set(['0.0.0.0', '::', '0000:0000:0000:0000:0000:0000:0
 
 // this code from vite https://github.com/vitejs/vite/blob/d09bbd093a4b893e78f0bbff5b17c7cf7821f403/packages/vite/src/node/utils.ts#L892-L914
 export function getNetworkAddress(
+	// biome-ignore lint/style/useDefaultParameterLast: <explanation>
 	protocol: 'http' | 'https' = 'http',
 	hostname: string | undefined,
 	port: number,
-	base?: string,
+	base?: string
 ) {
 	const NetworkAddress: NetworkAddressOpt = {
 		local: [],
 		network: [],
 	};
+	// biome-ignore lint/complexity/noForEach: <explanation>
 	Object.values(os.networkInterfaces())
 		.flatMap((nInterface) => nInterface ?? [])
 		.filter(
 			(detail) =>
+				// biome-ignore lint/complexity/useOptionalChain: <explanation>
 				detail &&
 				detail.address &&
 				(detail.family === 'IPv4' ||
 					// @ts-expect-error Node 18.0 - 18.3 returns number
-					detail.family === 4),
+					detail.family === 4)
 		)
 		.forEach((detail) => {
 			let host = detail.address.replace(
 				'127.0.0.1',
-				hostname === undefined || wildcardHosts.has(hostname) ? 'localhost' : hostname,
+				hostname === undefined || wildcardHosts.has(hostname) ? 'localhost' : hostname
 			);
 			// ipv6 host
 			if (host.includes(':')) {
diff --git a/packages/integrations/node/src/middleware.ts b/packages/integrations/node/src/middleware.ts
index 5cc4c4a46..5bb104914 100644
--- a/packages/integrations/node/src/middleware.ts
+++ b/packages/integrations/node/src/middleware.ts
@@ -15,7 +15,7 @@ export default function createMiddleware(app: NodeApp): RequestHandler {
 	const logger = app.getAdapterLogger();
 	// using spread args because express trips up if the function's
 	// stringified body includes req, res, next, locals directly
-	return async function (...args) {
+	return async (...args) => {
 		// assume normal invocation at first
 		const [req, res, next, locals] = args;
 		// short circuit if it is an error invocation
@@ -23,6 +23,7 @@ export default function createMiddleware(app: NodeApp): RequestHandler {
 			const error = req;
 			if (next) {
 				return next(error);
+				// biome-ignore lint/style/noUselessElse: <explanation>
 			} else {
 				throw error;
 			}
@@ -33,6 +34,7 @@ export default function createMiddleware(app: NodeApp): RequestHandler {
 			logger.error(`Could not render ${req.url}`);
 			console.error(err);
 			if (!res.headersSent) {
+				// biome-ignore lint/style/noUnusedTemplateLiteral: <explanation>
 				res.writeHead(500, `Server error`);
 				res.end();
 			}
diff --git a/packages/integrations/node/src/preview.ts b/packages/integrations/node/src/preview.ts
index 518155c4a..7e9415df8 100644
--- a/packages/integrations/node/src/preview.ts
+++ b/packages/integrations/node/src/preview.ts
@@ -8,7 +8,7 @@ import { createServer } from './standalone.js';
 type ServerModule = ReturnType<typeof createExports>;
 type MaybeServerModule = Partial<ServerModule>;
 
-const createPreviewServer: CreatePreviewServer = async function (preview) {
+const createPreviewServer: CreatePreviewServer = async (preview) => {
 	let ssrHandler: ServerModule['handler'];
 	let options: ServerModule['options'];
 	try {
@@ -16,19 +16,21 @@ const createPreviewServer: CreatePreviewServer = async function (preview) {
 		const ssrModule: MaybeServerModule = await import(preview.serverEntrypoint.toString());
 		if (typeof ssrModule.handler === 'function') {
 			ssrHandler = ssrModule.handler;
+			// biome-ignore lint/style/noNonNullAssertion: <explanation>
 			options = ssrModule.options!;
 		} else {
 			throw new AstroError(
-				`The server entrypoint doesn't have a handler. Are you sure this is the right file?`,
+				`The server entrypoint doesn't have a handler. Are you sure this is the right file?`
 			);
 		}
 	} catch (err) {
 		if ((err as any).code === 'ERR_MODULE_NOT_FOUND') {
 			throw new AstroError(
 				`The server entrypoint ${fileURLToPath(
-					preview.serverEntrypoint,
-				)} does not exist. Have you ran a build yet?`,
+					preview.serverEntrypoint
+				)} does not exist. Have you ran a build yet?`
 			);
+			// biome-ignore lint/style/noUselessElse: <explanation>
 		} else {
 			throw err;
 		}
diff --git a/packages/integrations/node/src/serve-app.ts b/packages/integrations/node/src/serve-app.ts
index 72b4e0fd6..2934a01ab 100644
--- a/packages/integrations/node/src/serve-app.ts
+++ b/packages/integrations/node/src/serve-app.ts
@@ -39,7 +39,7 @@ export function createAppHandler(app: NodeApp): RequestHandler {
 					addCookieHeader: true,
 					locals,
 					routeData,
-				}),
+				})
 			);
 			await NodeApp.writeResponse(response, res);
 		} else if (next) {
diff --git a/packages/integrations/node/src/serve-static.ts b/packages/integrations/node/src/serve-static.ts
index 725f7afa6..9221594d7 100644
--- a/packages/integrations/node/src/serve-static.ts
+++ b/packages/integrations/node/src/serve-static.ts
@@ -29,23 +29,28 @@ export function createStaticHandler(app: NodeApp, options: Options) {
 			let isDirectory = false;
 			try {
 				isDirectory = fs.lstatSync(filePath).isDirectory();
-			} catch {}
+			} catch { }
 
 			const { trailingSlash = 'ignore' } = options;
 
 			const hasSlash = urlPath.endsWith('/');
 			switch (trailingSlash) {
 				case 'never':
+					// biome-ignore lint/suspicious/noDoubleEquals: <explanation>
 					if (isDirectory && urlPath != '/' && hasSlash) {
+						// biome-ignore lint/style/useTemplate: <explanation>
+						// biome-ignore lint/suspicious/noFallthroughSwitchClause: <explanation>
 						pathname = urlPath.slice(0, -1) + (urlQuery ? '?' + urlQuery : '');
 						res.statusCode = 301;
 						res.setHeader('Location', pathname);
 						return res.end();
+						// biome-ignore lint/style/noUselessElse: <explanation>
 					} else pathname = urlPath;
 				// intentionally fall through
 				case 'ignore':
 					{
 						if (isDirectory && !hasSlash) {
+							// biome-ignore lint/style/useTemplate: <explanation>
 							pathname = urlPath + '/index.html';
 						} else pathname = urlPath;
 					}
@@ -53,10 +58,12 @@ export function createStaticHandler(app: NodeApp, options: Options) {
 				case 'always':
 					// trailing slash is not added to "subresources"
 					if (!hasSlash && !isSubresourceRegex.test(urlPath)) {
+						// biome-ignore lint/style/useTemplate: <explanation>
 						pathname = urlPath + '/' + (urlQuery ? '?' + urlQuery : '');
 						res.statusCode = 301;
 						res.setHeader('Location', pathname);
 						return res.end();
+						// biome-ignore lint/style/noUselessElse: <explanation>
 					} else pathname = urlPath;
 					break;
 			}
@@ -110,6 +117,7 @@ function resolveClientDir(options: Options) {
 	while (!serverEntryFolderURL.endsWith(serverFolder)) {
 		serverEntryFolderURL = path.dirname(serverEntryFolderURL);
 	}
+	// biome-ignore lint/style/useTemplate: <explanation>
 	const serverEntryURL = serverEntryFolderURL + '/entry.mjs';
 	const clientURL = new URL(appendForwardSlash(rel), serverEntryURL);
 	const client = url.fileURLToPath(clientURL);
@@ -117,9 +125,11 @@ function resolveClientDir(options: Options) {
 }
 
 function prependForwardSlash(pth: string) {
+	// biome-ignore lint/style/useTemplate: <explanation>
 	return pth.startsWith('/') ? pth : '/' + pth;
 }
 
 function appendForwardSlash(pth: string) {
+	// biome-ignore lint/style/useTemplate: <explanation>
 	return pth.endsWith('/') ? pth : pth + '/';
 }
diff --git a/packages/integrations/node/src/standalone.ts b/packages/integrations/node/src/standalone.ts
index 76e672d2f..8ae10a9ba 100644
--- a/packages/integrations/node/src/standalone.ts
+++ b/packages/integrations/node/src/standalone.ts
@@ -39,6 +39,7 @@ export function createStandaloneHandler(app: NodeApp, options: Options) {
 	return (req: http.IncomingMessage, res: http.ServerResponse) => {
 		try {
 			// validate request path
+			// biome-ignore lint/style/noNonNullAssertion: <explanation>
 			decodeURI(req.url!);
 		} catch {
 			res.writeHead(400);
@@ -59,7 +60,7 @@ export function createServer(listener: http.RequestListener, host: string, port:
 				key: fs.readFileSync(process.env.SERVER_KEY_PATH),
 				cert: fs.readFileSync(process.env.SERVER_CERT_PATH),
 			},
-			listener,
+			listener
 		);
 	} else {
 		httpServer = http.createServer(listener);
diff --git a/packages/integrations/node/test/api-route.test.js b/packages/integrations/node/test/api-route.test.js
index 804a5ccf4..5eca5c530 100644
--- a/packages/integrations/node/test/api-route.test.js
+++ b/packages/integrations/node/test/api-route.test.js
@@ -27,7 +27,7 @@ describe('API routes', () => {
 
 	it('Can get the request body', async () => {
 		const { handler } = await import('./fixtures/api-route/dist/server/entry.mjs');
-		let { req, res, done } = createRequestAndResponse({
+		const { req, res, done } = createRequestAndResponse({
 			method: 'POST',
 			url: '/recipes',
 		});
@@ -38,9 +38,9 @@ describe('API routes', () => {
 
 		handler(req, res);
 
-		let [buffer] = await done;
+		const [buffer] = await done;
 
-		let json = JSON.parse(buffer.toString('utf-8'));
+		const json = JSON.parse(buffer.toString('utf-8'));
 
 		assert.equal(json.length, 1);
 
@@ -50,7 +50,7 @@ describe('API routes', () => {
 	it('Can get binary data', async () => {
 		const { handler } = await import('./fixtures/api-route/dist/server/entry.mjs');
 
-		let { req, res, done } = createRequestAndResponse({
+		const { req, res, done } = createRequestAndResponse({
 			method: 'POST',
 			url: '/binary',
 		});
@@ -61,15 +61,15 @@ describe('API routes', () => {
 
 		handler(req, res);
 
-		let [out] = await done;
-		let arr = Array.from(new Uint8Array(out.buffer));
+		const [out] = await done;
+		const arr = Array.from(new Uint8Array(out.buffer));
 		assert.deepEqual(arr, [5, 4, 3, 2, 1]);
 	});
 
 	it('Can post large binary data', async () => {
 		const { handler } = await import('./fixtures/api-route/dist/server/entry.mjs');
 
-		let { req, res, done } = createRequestAndResponse({
+		const { req, res, done } = createRequestAndResponse({
 			method: 'POST',
 			url: '/hash',
 		});
@@ -95,17 +95,17 @@ describe('API routes', () => {
 			expectedDigest = hash.digest();
 		});
 
-		let [out] = await done;
+		const [out] = await done;
 		assert.deepEqual(new Uint8Array(out.buffer), new Uint8Array(expectedDigest));
 	});
 
 	it('Can bail on streaming', async () => {
 		const { handler } = await import('./fixtures/api-route/dist/server/entry.mjs');
-		let { req, res, done } = createRequestAndResponse({
+		const { req, res, done } = createRequestAndResponse({
 			url: '/streaming',
 		});
 
-		let locals = { cancelledByTheServer: false };
+		const locals = { cancelledByTheServer: false };
 
 		handler(req, res, () => {}, locals);
 		req.send();
diff --git a/packages/integrations/node/test/bad-urls.test.js b/packages/integrations/node/test/bad-urls.test.js
index 9323516e8..cdc0158ff 100644
--- a/packages/integrations/node/test/bad-urls.test.js
+++ b/packages/integrations/node/test/bad-urls.test.js
@@ -39,7 +39,7 @@ describe('Bad URLs', () => {
 			assert.equal(
 				statusCodes.includes(fetchResult.status),
 				true,
-				`${weirdUrl} returned something else than 400, 404, or 500`,
+				`${weirdUrl} returned something else than 400, 404, or 500`
 			);
 		}
 		const stillWork = await fixture.fetch('/');
diff --git a/packages/integrations/node/test/encoded.test.js b/packages/integrations/node/test/encoded.test.js
index edc6ae78b..4fc97cf7f 100644
--- a/packages/integrations/node/test/encoded.test.js
+++ b/packages/integrations/node/test/encoded.test.js
@@ -18,7 +18,7 @@ describe('Encoded Pathname', () => {
 
 	it('Can get an Astro file', async () => {
 		const { handler } = await import('./fixtures/encoded/dist/server/entry.mjs');
-		let { req, res, text } = createRequestAndResponse({
+		const { req, res, text } = createRequestAndResponse({
 			url: '/什么',
 		});
 
@@ -32,7 +32,7 @@ describe('Encoded Pathname', () => {
 	it('Can get a Markdown file', async () => {
 		const { handler } = await import('./fixtures/encoded/dist/server/entry.mjs');
 
-		let { req, res, text } = createRequestAndResponse({
+		const { req, res, text } = createRequestAndResponse({
 			url: '/blog/什么',
 		});
 
diff --git a/packages/integrations/node/test/errors.test.js b/packages/integrations/node/test/errors.test.js
index 802fa6e25..9bf4aa29b 100644
--- a/packages/integrations/node/test/errors.test.js
+++ b/packages/integrations/node/test/errors.test.js
@@ -20,6 +20,7 @@ describe('Errors', () => {
 	});
 	let devPreview;
 
+	// biome-ignore lint/suspicious/noDuplicateTestHooks: <explanation>
 	before(async () => {
 		// The two tests that need the server to run are skipped
 		// devPreview = await fixture.preview();
@@ -58,7 +59,7 @@ describe('Errors', () => {
 			const $ = cheerio.load(html);
 
 			assert.equal($('p').text().trim(), 'Internal server error');
-		},
+		}
 	);
 
 	it(
@@ -86,6 +87,6 @@ describe('Errors', () => {
 			} else {
 				throw new Error('The response should take at most 2 chunks.');
 			}
-		},
+		}
 	);
 });
diff --git a/packages/integrations/node/test/headers.test.js b/packages/integrations/node/test/headers.test.js
index 00b1766c7..f2753517e 100644
--- a/packages/integrations/node/test/headers.test.js
+++ b/packages/integrations/node/test/headers.test.js
@@ -132,7 +132,7 @@ describe('Node Adapter Headers', () => {
 async function runTest(url, expectedHeaders) {
 	const { handler } = await import('./fixtures/headers/dist/server/entry.mjs');
 
-	let { req, res, done } = createRequestAndResponse({
+	const { req, res, done } = createRequestAndResponse({
 		method: 'GET',
 		url,
 	});
diff --git a/packages/integrations/node/test/image.test.js b/packages/integrations/node/test/image.test.js
index 5303fe37e..c4758f96b 100644
--- a/packages/integrations/node/test/image.test.js
+++ b/packages/integrations/node/test/image.test.js
@@ -28,7 +28,7 @@ describe.skip('Image endpoint', () => {
 		assert.equal(res.status, 200);
 
 		const resImage = await fixture.fetch(
-			'/_image?href=/_astro/some_penguin.97ef5f92.png&w=50&f=webp',
+			'/_image?href=/_astro/some_penguin.97ef5f92.png&w=50&f=webp'
 		);
 
 		assert.equal(resImage.status, 200);
diff --git a/packages/integrations/node/test/locals.test.js b/packages/integrations/node/test/locals.test.js
index 6d2776079..b8e3ed40f 100644
--- a/packages/integrations/node/test/locals.test.js
+++ b/packages/integrations/node/test/locals.test.js
@@ -18,23 +18,23 @@ describe('API routes', () => {
 
 	it('Can use locals added by node middleware', async () => {
 		const { handler } = await import('./fixtures/locals/dist/server/entry.mjs');
-		let { req, res, text } = createRequestAndResponse({
+		const { req, res, text } = createRequestAndResponse({
 			url: '/from-node-middleware',
 		});
 
-		let locals = { foo: 'bar' };
+		const locals = { foo: 'bar' };
 
 		handler(req, res, () => {}, locals);
 		req.send();
 
-		let html = await text();
+		const html = await text();
 
 		assert.equal(html.includes('<h1>bar</h1>'), true);
 	});
 
 	it('Throws an error when provided non-objects as locals', async () => {
 		const { handler } = await import('./fixtures/locals/dist/server/entry.mjs');
-		let { req, res, done } = createRequestAndResponse({
+		const { req, res, done } = createRequestAndResponse({
 			url: '/from-node-middleware',
 		});
 
@@ -62,19 +62,19 @@ describe('API routes', () => {
 
 	it('Can access locals in API', async () => {
 		const { handler } = await import('./fixtures/locals/dist/server/entry.mjs');
-		let { req, res, done } = createRequestAndResponse({
+		const { req, res, done } = createRequestAndResponse({
 			method: 'POST',
 			url: '/api',
 		});
 
-		let locals = { foo: 'bar' };
+		const locals = { foo: 'bar' };
 
 		handler(req, res, () => {}, locals);
 		req.send();
 
-		let [buffer] = await done;
+		const [buffer] = await done;
 
-		let json = JSON.parse(buffer.toString('utf-8'));
+		const json = JSON.parse(buffer.toString('utf-8'));
 
 		assert.equal(json.foo, 'bar');
 	});
diff --git a/packages/integrations/node/test/node-middleware.test.js b/packages/integrations/node/test/node-middleware.test.js
index d1b016a51..eeb193c73 100644
--- a/packages/integrations/node/test/node-middleware.test.js
+++ b/packages/integrations/node/test/node-middleware.test.js
@@ -23,7 +23,7 @@ describe('behavior from middleware, standalone', () => {
 		});
 		await fixture.build();
 		const { startServer } = await fixture.loadAdapterEntryModule();
-		let res = startServer();
+		const res = startServer();
 		server = res.server;
 		await waitServerListen(server.server);
 	});
@@ -31,6 +31,7 @@ describe('behavior from middleware, standalone', () => {
 	after(async () => {
 		await server.stop();
 		await fixture.clean();
+		// biome-ignore lint/performance/noDelete: <explanation>
 		delete process.env.PRERENDER;
 	});
 
@@ -71,10 +72,12 @@ describe('behavior from middleware, middleware', () => {
 	after(async () => {
 		server.close();
 		await fixture.clean();
+		// biome-ignore lint/performance/noDelete: <explanation>
 		delete process.env.PRERENDER;
 	});
 
 	it('when mode is standalone', async () => {
+		// biome-ignore lint/style/noUnusedTemplateLiteral: <explanation>
 		const res = await fetch(`http://localhost:8888/ssr`);
 
 		assert.equal(res.status, 200);
diff --git a/packages/integrations/node/test/prerender-404-500.test.js b/packages/integrations/node/test/prerender-404-500.test.js
index 2535fcb35..86226c500 100644
--- a/packages/integrations/node/test/prerender-404-500.test.js
+++ b/packages/integrations/node/test/prerender-404-500.test.js
@@ -34,7 +34,7 @@ describe('Prerender 404', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -42,6 +42,7 @@ describe('Prerender 404', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
@@ -124,7 +125,7 @@ describe('Prerender 404', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -132,6 +133,7 @@ describe('Prerender 404', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
@@ -193,7 +195,7 @@ describe('Hybrid 404', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -201,6 +203,7 @@ describe('Hybrid 404', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
@@ -255,7 +258,7 @@ describe('Hybrid 404', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -263,6 +266,7 @@ describe('Hybrid 404', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
diff --git a/packages/integrations/node/test/prerender.test.js b/packages/integrations/node/test/prerender.test.js
index e699a1b3c..0684ff63a 100644
--- a/packages/integrations/node/test/prerender.test.js
+++ b/packages/integrations/node/test/prerender.test.js
@@ -30,7 +30,7 @@ describe('Prerendering', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -38,6 +38,7 @@ describe('Prerendering', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
@@ -105,7 +106,7 @@ describe('Prerendering', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -113,6 +114,7 @@ describe('Prerendering', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
@@ -181,7 +183,7 @@ describe('Prerendering', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -189,6 +191,7 @@ describe('Prerendering', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
@@ -233,10 +236,12 @@ describe('Prerendering', () => {
 
 		after(async () => {
 			await devServer.stop();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
 		it('Can render SSR route', async () => {
+			// biome-ignore lint/style/noUnusedTemplateLiteral: <explanation>
 			const res = await fixture.fetch(`/one`);
 			const html = await res.text();
 			const $ = cheerio.load(html);
@@ -246,6 +251,7 @@ describe('Prerendering', () => {
 		});
 
 		it('Can render prerendered route', async () => {
+			// biome-ignore lint/style/noUnusedTemplateLiteral: <explanation>
 			const res = await fixture.fetch(`/two`);
 			const html = await res.text();
 			const $ = cheerio.load(html);
@@ -277,7 +283,7 @@ describe('Hybrid rendering', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -285,6 +291,7 @@ describe('Hybrid rendering', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
@@ -350,7 +357,7 @@ describe('Hybrid rendering', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -358,6 +365,7 @@ describe('Hybrid rendering', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
@@ -415,7 +423,7 @@ describe('Hybrid rendering', () => {
 			});
 			await fixture.build();
 			const { startServer } = await fixture.loadAdapterEntryModule();
-			let res = startServer();
+			const res = startServer();
 			server = res.server;
 			await waitServerListen(server.server);
 		});
@@ -423,6 +431,7 @@ describe('Hybrid rendering', () => {
 		after(async () => {
 			await server.stop();
 			await fixture.clean();
+			// biome-ignore lint/performance/noDelete: <explanation>
 			delete process.env.PRERENDER;
 		});
 
diff --git a/packages/integrations/node/test/test-utils.js b/packages/integrations/node/test/test-utils.js
index 10faba5cf..4f273756c 100644
--- a/packages/integrations/node/test/test-utils.js
+++ b/packages/integrations/node/test/test-utils.js
@@ -20,18 +20,18 @@ export function loadFixture(inlineConfig) {
 }
 
 export function createRequestAndResponse(reqOptions) {
-	let req = httpMocks.createRequest(reqOptions);
+	const req = httpMocks.createRequest(reqOptions);
 
-	let res = httpMocks.createResponse({
+	const res = httpMocks.createResponse({
 		eventEmitter: EventEmitter,
 		req,
 	});
 
-	let done = toPromise(res);
+	const done = toPromise(res);
 
 	// Get the response as text
 	const text = async () => {
-		let chunks = await done;
+		const chunks = await done;
 		return buffersToString(chunks);
 	};
 
@@ -45,19 +45,20 @@ export function toPromise(res) {
 		const write = res.write;
 		res.write = function (data, encoding) {
 			if (ArrayBuffer.isView(data) && !Buffer.isBuffer(data)) {
+				// biome-ignore lint/style/noParameterAssign: <explanation>
 				data = Buffer.from(data.buffer);
 			}
 			return write.call(this, data, encoding);
 		};
 		res.on('end', () => {
-			let chunks = res._getChunks();
+			const chunks = res._getChunks();
 			resolve(chunks);
 		});
 	});
 }
 
 export function buffersToString(buffers) {
-	let decoder = new TextDecoder();
+	const decoder = new TextDecoder();
 	let str = '';
 	for (const buffer of buffers) {
 		str += decoder.decode(buffer);
diff --git a/packages/integrations/node/test/trailing-slash.test.js b/packages/integrations/node/test/trailing-slash.test.js
index 9ea8fcddd..6f6a2a3ba 100644
--- a/packages/integrations/node/test/trailing-slash.test.js
+++ b/packages/integrations/node/test/trailing-slash.test.js
@@ -32,7 +32,7 @@ describe('Trailing slash', () => {
 				});
 				await fixture.build();
 				const { startServer } = await fixture.loadAdapterEntryModule();
-				let res = startServer();
+				const res = startServer();
 				server = res.server;
 				await waitServerListen(server.server);
 			});
@@ -40,6 +40,7 @@ describe('Trailing slash', () => {
 			after(async () => {
 				await server.stop();
 				await fixture.clean();
+				// biome-ignore lint/performance/noDelete: <explanation>
 				delete process.env.PRERENDER;
 			});
 
@@ -103,7 +104,7 @@ describe('Trailing slash', () => {
 				});
 				await fixture.build();
 				const { startServer } = await fixture.loadAdapterEntryModule();
-				let res = startServer();
+				const res = startServer();
 				server = res.server;
 				await waitServerListen(server.server);
 			});
@@ -111,6 +112,7 @@ describe('Trailing slash', () => {
 			after(async () => {
 				await server.stop();
 				await fixture.clean();
+				// biome-ignore lint/performance/noDelete: <explanation>
 				delete process.env.PRERENDER;
 			});
 
@@ -177,7 +179,7 @@ describe('Trailing slash', () => {
 				});
 				await fixture.build();
 				const { startServer } = await fixture.loadAdapterEntryModule();
-				let res = startServer();
+				const res = startServer();
 				server = res.server;
 				await waitServerListen(server.server);
 			});
@@ -185,6 +187,7 @@ describe('Trailing slash', () => {
 			after(async () => {
 				await server.stop();
 				await fixture.clean();
+				// biome-ignore lint/performance/noDelete: <explanation>
 				delete process.env.PRERENDER;
 			});
 
@@ -241,7 +244,7 @@ describe('Trailing slash', () => {
 				});
 				await fixture.build();
 				const { startServer } = await fixture.loadAdapterEntryModule();
-				let res = startServer();
+				const res = startServer();
 				server = res.server;
 				await waitServerListen(server.server);
 			});
@@ -249,6 +252,7 @@ describe('Trailing slash', () => {
 			after(async () => {
 				await server.stop();
 				await fixture.clean();
+				// biome-ignore lint/performance/noDelete: <explanation>
 				delete process.env.PRERENDER;
 			});
 
@@ -308,7 +312,7 @@ describe('Trailing slash', () => {
 				});
 				await fixture.build();
 				const { startServer } = await fixture.loadAdapterEntryModule();
-				let res = startServer();
+				const res = startServer();
 				server = res.server;
 				await waitServerListen(server.server);
 			});
@@ -316,6 +320,7 @@ describe('Trailing slash', () => {
 			after(async () => {
 				await server.stop();
 				await fixture.clean();
+				// biome-ignore lint/performance/noDelete: <explanation>
 				delete process.env.PRERENDER;
 			});
 
@@ -390,7 +395,7 @@ describe('Trailing slash', () => {
 				});
 				await fixture.build();
 				const { startServer } = await fixture.loadAdapterEntryModule();
-				let res = startServer();
+				const res = startServer();
 				server = res.server;
 				await waitServerListen(server.server);
 			});
@@ -398,6 +403,7 @@ describe('Trailing slash', () => {
 			after(async () => {
 				await server.stop();
 				await fixture.clean();
+				// biome-ignore lint/performance/noDelete: <explanation>
 				delete process.env.PRERENDER;
 			});
 
diff --git a/packages/integrations/node/test/url.test.js b/packages/integrations/node/test/url.test.js
index 77ca45836..81b357b71 100644
--- a/packages/integrations/node/test/url.test.js
+++ b/packages/integrations/node/test/url.test.js
@@ -20,7 +20,7 @@ describe('URL', () => {
 
 	it('return http when non-secure', async () => {
 		const { handler } = await import('./fixtures/url/dist/server/entry.mjs');
-		let { req, res, text } = createRequestAndResponse({
+		const { req, res, text } = createRequestAndResponse({
 			url: '/',
 		});
 
@@ -33,7 +33,7 @@ describe('URL', () => {
 
 	it('return https when secure', async () => {
 		const { handler } = await import('./fixtures/url/dist/server/entry.mjs');
-		let { req, res, text } = createRequestAndResponse({
+		const { req, res, text } = createRequestAndResponse({
 			socket: new TLSSocket(),
 			url: '/',
 		});
@@ -47,7 +47,7 @@ describe('URL', () => {
 
 	it('return http when the X-Forwarded-Proto header is set to http', async () => {
 		const { handler } = await import('./fixtures/url/dist/server/entry.mjs');
-		let { req, res, text } = createRequestAndResponse({
+		const { req, res, text } = createRequestAndResponse({
 			headers: { 'X-Forwarded-Proto': 'http' },
 			url: '/',
 		});
@@ -61,7 +61,7 @@ describe('URL', () => {
 
 	it('return https when the X-Forwarded-Proto header is set to https', async () => {
 		const { handler } = await import('./fixtures/url/dist/server/entry.mjs');
-		let { req, res, text } = createRequestAndResponse({
+		const { req, res, text } = createRequestAndResponse({
 			headers: { 'X-Forwarded-Proto': 'https' },
 			url: '/',
 		});
@@ -75,7 +75,7 @@ describe('URL', () => {
 
 	it('includes forwarded host and port in the url', async () => {
 		const { handler } = await import('./fixtures/url/dist/server/entry.mjs');
-		let { req, res, text } = createRequestAndResponse({
+		const { req, res, text } = createRequestAndResponse({
 			headers: {
 				'X-Forwarded-Proto': 'https',
 				'X-Forwarded-Host': 'abc.xyz',
@@ -95,7 +95,7 @@ describe('URL', () => {
 
 	it('accepts port in forwarded host and forwarded port', async () => {
 		const { handler } = await import('./fixtures/url/dist/server/entry.mjs');
-		let { req, res, text } = createRequestAndResponse({
+		const { req, res, text } = createRequestAndResponse({
 			headers: {
 				'X-Forwarded-Proto': 'https',
 				'X-Forwarded-Host': 'abc.xyz:444',
diff --git a/packages/integrations/node/tsconfig.json b/packages/integrations/node/tsconfig.json
index b9feb9b9b..18443cddf 100644
--- a/packages/integrations/node/tsconfig.json
+++ b/packages/integrations/node/tsconfig.json
@@ -1,8 +1,6 @@
 {
   "extends": "../../tsconfig.base.json",
-  "include": [
-    "src"
-  ],
+  "include": ["src"],
   "compilerOptions": {
     "outDir": "./dist"
   }
-- 
cgit v1.2.3