chore(@astrojs/underscore-redirects): use Node.js for testing (#9923)

* chore: migrate underscore-redirects tests to node

* fix: remove chai/mocha devDependencies

* Update packages/underscore-redirects/package.json

* update lockfile

---------

Co-authored-by: Bjorn Lu <bjornlu.dev@gmail.com>
Co-authored-by: Emanuele Stoppa <my.burning@gmail.com>

packages/underscore-redirects/package.json

@@ -25,15 +25,11 @@
     "build:ci": "astro-scripts build \"src/**/*.ts\"",
     "postbuild": "astro-scripts copy \"src/**/*.js\"",
     "dev": "astro-scripts dev \"src/**/*.ts\"",
-    "test": "mocha --exit --timeout 20000"
+    "test": "astro-scripts test \"test/**/*.test.js\""
   },
   "devDependencies": {
-    "@types/chai": "^4.3.10",
-    "@types/mocha": "^10.0.4",
     "astro": "workspace:*",
-    "astro-scripts": "workspace:*",
+    "astro-scripts": "workspace:*"
-    "chai": "^4.3.7",
-    "mocha": "^10.2.0"
   },
   "keywords": [
     "astro",

packages/underscore-redirects/test/astro.test.js

@@ -1,5 +1,6 @@
 import { createRedirectsFromAstroRoutes } from '../dist/index.js';
-import { expect } from 'chai';
+import * as assert from 'node:assert/strict';
+import { describe, it } from 'node:test';
 
 describe('Astro', () => {
 	const serverConfig = {
@@ -26,6 +27,6 @@ describe('Astro', () => {
 			dir: new URL(import.meta.url),
 		});
 
-		expect(_redirects.definitions).to.have.a.lengthOf(2);
+		assert.equal(_redirects.definitions.length, 2);
 	});
 });

packages/underscore-redirects/test/print.test.js

@@ -1,5 +1,6 @@
 import { Redirects } from '../dist/index.js';
-import { expect } from 'chai';
+import * as assert from 'node:assert/strict';
+import { describe, it } from 'node:test';
 
 describe('Printing', () => {
 	it('Formats long lines in a pretty way', () => {
@@ -22,8 +23,13 @@ describe('Printing', () => {
 
 		let [lineOne, lineTwo] = out.split('\n');
 
-		expect(lineOne.indexOf('/b')).to.equal(lineTwo.indexOf('/b'), 'destinations lined up');
+		const indexLineOne = lineOne.indexOf('/b');
-		expect(lineOne.indexOf('200')).to.equal(lineTwo.indexOf('200'), 'statuses lined up');
+		const indexLineTwo = lineTwo.indexOf('/b');
+		assert.equal(indexLineOne, indexLineTwo, 'destinations lined up');
+
+		const newIndexLineOne = lineOne.indexOf('200');
+		const newIndexLineTwo = lineTwo.indexOf('200');
+		assert.equal(newIndexLineOne, newIndexLineTwo, 'statuses lined up');
 	});
 
 	it('Properly prints dynamic routes', () => {
@@ -37,6 +43,8 @@ describe('Printing', () => {
 		});
 		let out = _redirects.print();
 		let parts = out.split(/\s+/);
-		expect(parts).to.deep.equal(['/pets/:cat', '/pets/:cat/index.html', '200']);
+
+		const expectedParts = ['/pets/:cat', '/pets/:cat/index.html', '200'];
+		assert.deepEqual(parts, expectedParts);
 	});
 });

packages/underscore-redirects/test/weight.test.js

@@ -1,5 +1,6 @@
 import { Redirects } from '../dist/index.js';
-import { expect } from 'chai';
+import * as assert from 'node:assert/strict';
+import { describe, it } from 'node:test';
 
 describe('Weight', () => {
 	it('Puts higher weighted definitions on top', () => {
@@ -26,7 +27,7 @@ describe('Weight', () => {
 			status: 200,
 		});
 		const firstDefn = _redirects.definitions[0];
-		expect(firstDefn.weight).to.equal(1);
+		assert.equal(firstDefn.weight, 1);
-		expect(firstDefn.input).to.equal('/e');
+		assert.equal(firstDefn.input, '/e');
 	});
 });

pnpm-lock.yaml

@@ -5156,24 +5156,12 @@ importers:
 
   packages/underscore-redirects:
     devDependencies:
-      '@types/chai':
-        specifier: ^4.3.10
-        version: 4.3.11
-      '@types/mocha':
-        specifier: ^10.0.4
-        version: 10.0.6
       astro:
         specifier: workspace:*
         version: link:../astro
       astro-scripts:
         specifier: workspace:*
         version: link:../../scripts
-      chai:
-        specifier: ^4.3.7
-        version: 4.3.10
-      mocha:
-        specifier: ^10.2.0
-        version: 10.2.0
 
   packages/upgrade:
     dependencies: