chore: replaced test-listen with async-listen (#920)

dequelabs · Nov 7, 2023 · 30af456 · 30af456
1 parent 8731be9
commit 30af456
Show file tree

Hide file tree

Showing 12 changed files with 46 additions and 48 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/packages/playwright/package.json b/packages/playwright/package.json
@@ -56,14 +56,13 @@
     "@types/express": "^4.17.14",
     "@types/mocha": "^10.0.0",
     "@types/node": "^20.8.10",
-    "@types/test-listen": "^1.1.0",
+    "async-listen": "^3.0.1",
     "axe-test-fixtures": "github:dequelabs/axe-test-fixtures#v1",
     "chai": "^4.3.6",
     "express": "^4.18.2",
     "mocha": "^10.0.0",
     "nyc": "^15.1.0",
     "rimraf": "^5.0.5",
-    "test-listen": "^1.1.0",
     "ts-node": "^10.9.1",
     "tsup": "^7.2.0",
     "typescript": "^4.8.4"

diff --git a/packages/playwright/test/axe-playwright.spec.ts b/packages/playwright/test/axe-playwright.spec.ts
@@ -3,7 +3,7 @@ import fs from 'fs';
 import { chromium, ChromiumBrowser, Page } from '@playwright/test';
 import express from 'express';
 import type { AxeResults, Result } from 'axe-core';
-import testListen from 'test-listen';
+import listen from 'async-listen';
 import { assert } from 'chai';
 import path from 'path';
 import { Server, createServer } from 'http';
@@ -38,7 +38,9 @@ describe('@axe-core/playwright', () => {
     const app = express();
     app.use(express.static(fixturesPath));
     server = createServer(app);
-    addr = await testListen(server);
+    // async-listen adds trailing forward slash,
+    // this removes the unnecessary trailing forward slash
+    addr = (await listen(server)).toString().replace(/\/$/, '');
   });
 
   after(async () => {

diff --git a/packages/puppeteer/package.json b/packages/puppeteer/package.json
@@ -37,7 +37,7 @@
     "@types/mocha": "^10.0.0",
     "@types/node": "^20.8.10",
     "@types/sinon": "^10.0.13",
-    "@types/test-listen": "^1.1.0",
+    "async-listen": "^3.0.1",
     "axe-test-fixtures": "github:dequelabs/axe-test-fixtures#v1",
     "chai": "^4.3.6",
     "cross-dirname": "^0.1.0",
@@ -47,7 +47,6 @@
     "puppeteer": "19.7.3",
     "sinon": "^14.0.1",
     "source-map-support": "^0.5.21",
-    "test-listen": "^1.1.0",
     "ts-node": "^10.9.1",
     "tsup": "^7.2.0",
     "typescript": "^4.8.4"

diff --git a/packages/puppeteer/test/axe-puppeteer.spec.ts b/packages/puppeteer/test/axe-puppeteer.spec.ts
@@ -6,7 +6,7 @@ import { assert } from 'chai';
 import Puppeteer, { Browser, Page } from 'puppeteer';
 import { createServer, Server } from 'http';
 import * as sinon from 'sinon';
-import testListen from 'test-listen';
+import listen from 'async-listen';
 import { AxePuppeteer } from '../src/index';
 import {
   startServer,
@@ -355,7 +355,9 @@ describe('AxePuppeteer', function () {
             }, 3000);
           }
         });
-        addr2 = await testListen(server2);
+        // async-listen adds trailing forward slash,
+        // this removes the unnecessary trailing forward slash
+        addr2 = (await listen(server2)).toString().replace(/\/$/, '');
       });
 
       after(() => {

diff --git a/packages/puppeteer/test/utils.ts b/packages/puppeteer/test/utils.ts
@@ -1,6 +1,6 @@
 import express from 'express';
 import { createServer, Server } from 'http';
-import testListen from 'test-listen';
+import listen from 'async-listen';
 import { expect } from 'chai';
 import type { PuppeteerLaunchOptions } from 'puppeteer';
 import { fixturesPath } from 'axe-test-fixtures';
@@ -29,7 +29,9 @@ export async function startServer(): Promise<{ server: Server; addr: string }> {
   const app: express.Application = express();
   app.use(express.static(fixturesPath));
   const server: Server = createServer(app);
-  const addr = await testListen(server);
+  // async-listen adds trailing forward slash,
+  // this removes the unnecessary trailing forward slash
+  const addr = (await listen(server)).toString().replace(/\/$/, '');
 
   return { server, addr };
 }

diff --git a/packages/webdriverio/package.json b/packages/webdriverio/package.json
@@ -55,7 +55,7 @@
     "@types/express": "^4.17.14",
     "@types/mocha": "^10.0.0",
     "@types/node": "^20.8.10",
-    "@types/test-listen": "^1.1.0",
+    "async-listen": "^3.0.1",
     "axe-test-fixtures": "github:dequelabs/axe-test-fixtures#v1",
     "chai": "^4.3.6",
     "chromedriver": "latest",
@@ -66,7 +66,6 @@
     "nyc": "^15.1.0",
     "rimraf": "^5.0.5",
     "source-map-support": "^0.5.21",
-    "test-listen": "^1.1.0",
     "ts-node": "^10.9.1",
     "tsup": "^7.2.0",
     "typescript": "^4.8.4",

diff --git a/packages/webdriverio/test/axe-webdriverio.spec.ts b/packages/webdriverio/test/axe-webdriverio.spec.ts
@@ -1,6 +1,6 @@
 import * as webdriverio from 'webdriverio';
 import express from 'express';
-import testListen from 'test-listen';
+import listen from 'async-listen';
 import { assert } from 'chai';
 import chromedriver from 'chromedriver';
 import path from 'path';
@@ -96,7 +96,9 @@ describe('@axe-core/webdriverio', () => {
         let binaryPath;
         app.use(express.static(fixturesPath));
         server = createServer(app);
-        addr = await testListen(server);
+        // async-listen adds trailing forward slash,
+        // this removes the unnecessary trailing forward slash
+        addr = (await listen(server)).toString().replace(/\/$/, '');
         if (
           fs.existsSync(`C:/Program Files/Google/Chrome/Application/chrome.exe`)
         ) {

diff --git a/packages/webdriverjs/package.json b/packages/webdriverjs/package.json
@@ -72,7 +72,7 @@
     "@types/mocha": "^10.0.0",
     "@types/node": "^20.8.10",
     "@types/selenium-webdriver": "^4.1.5",
-    "@types/test-listen": "^1.1.0",
+    "async-listen": "^3.0.1",
     "axe-test-fixtures": "github:dequelabs/axe-test-fixtures#v1",
     "chai": "^4.3.6",
     "chromedriver": "latest",
@@ -81,7 +81,6 @@
     "nyc": "^15.1.0",
     "rimraf": "^5.0.5",
     "selenium-webdriver": "^4.8.1",
-    "test-listen": "^1.1.0",
     "ts-node": "^10.9.1",
     "tsup": "^7.2.0",
     "typescript": "^4.8.4"

diff --git a/packages/webdriverjs/test/axe-webdriverjs.spec.ts b/packages/webdriverjs/test/axe-webdriverjs.spec.ts
@@ -2,7 +2,7 @@ import 'mocha';
 import type { AxeResults, Result } from 'axe-core';
 import type { WebDriver } from 'selenium-webdriver';
 import express from 'express';
-import testListen from 'test-listen';
+import listen from 'async-listen';
 import { assert } from 'chai';
 import path from 'path';
 import fs from 'fs';
@@ -44,7 +44,9 @@ describe('@axe-core/webdriverjs', () => {
     const app = express();
     app.use(express.static(fixturesPath));
     server = createServer(app);
-    addr = await testListen(server);
+    // async-listen adds trailing forward slash,
+    // this removes the unnecessary trailing forward slash
+    addr = (await listen(server)).toString().replace(/\/$/, '');
   });
 
   beforeEach(async () => {

diff --git a/test/wdio/package.json b/test/wdio/package.json
@@ -14,11 +14,11 @@
     "@wdio/local-runner": "^8.20.0",
     "@wdio/mocha-framework": "^8.20.0",
     "@wdio/spec-reporter": "^8.20.0",
+    "async-listen": "^3.0.1",
     "axe-test-fixtures": "github:dequelabs/axe-test-fixtures#v1",
     "chai": "^4.3.10",
     "express": "^4.18.2",
     "mocha": "^10.2.0",
-    "test-listen": "^1.1.0",
     "ts-node": "^10.9.1",
     "typescript": "^5.2.2"
   }

diff --git a/test/wdio/wdio-globals.spec.ts b/test/wdio/wdio-globals.spec.ts
@@ -4,7 +4,7 @@ import { assert } from 'chai';
 import express from 'express';
 import { fixturesPath } from 'axe-test-fixtures';
 import { Server, createServer } from 'http';
-import testListen from 'test-listen';
+import listen from 'async-listen';
 
 describe('@wdio/globals', () => {
   let server: Server;
@@ -14,7 +14,9 @@ describe('@wdio/globals', () => {
     const app = express();
     app.use(express.static(fixturesPath));
     server = createServer(app);
-    addr = await testListen(server);
+    // async-listen adds trailing forward slash,
+    // this removes the unnecessary trailing forward slash
+    addr = (await listen(server)).toString().replace(/\/$/, '');
   });
 
   beforeEach(() => {