@atproto/tap 0.3.1 → 0.3.2

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # @atproto/tap
 
+## 0.3.2
+
+### Patch Changes
+
+- [#5117](https://github.com/bluesky-social/atproto/pull/5117) [`8a4c88b`](https://github.com/bluesky-social/atproto/commit/8a4c88b0f63fb2d82b1391d64c54e0c760fac48b) Thanks [@matthieusieben](https://github.com/matthieusieben)! - Make tests less flaky in CI
+
+- Updated dependencies [[`8a4c88b`](https://github.com/bluesky-social/atproto/commit/8a4c88b0f63fb2d82b1391d64c54e0c760fac48b)]:
+  - @atproto/ws-client@0.1.2
+
 ## 0.3.1
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@atproto/tap",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "license": "MIT",
   "description": "atproto tap client",
   "keywords": [
@@ -24,13 +24,14 @@
     "@atproto/common": "^0.6.3",
     "@atproto/lex": "^0.1.4",
     "@atproto/syntax": "^0.6.2",
-    "@atproto/ws-client": "^0.1.1"
+    "@atproto/ws-client": "^0.1.2"
   },
   "devDependencies": {
     "@types/express": "^4.17.17",
     "@types/ws": "^8.5.4",
-    "express": "^4.18.2",
     "@vitest/coverage-v8": "4.0.16",
+    "express": "^4.18.2",
+    "http-terminator": "^3.2.0",
     "vitest": "^4.0.16"
   },
   "type": "module",

package/tests/client.test.ts

@@ -2,6 +2,8 @@ import { once } from 'node:events'
 import * as http from 'node:http'
 import { AddressInfo } from 'node:net'
 import { default as express } from 'express'
+// eslint-disable-next-line import/default
+import httpTerminator from 'http-terminator'
 import { afterAll, beforeAll, beforeEach, describe, expect, it } from 'vitest'
 import { Tap } from '../src/client.js'
 
@@ -28,10 +30,7 @@ describe('Tap client', () => {
   })
 
   describe('HTTP methods', () => {
-    let server: http.Server<
-      typeof http.IncomingMessage,
-      typeof http.ServerResponse
-    >
+    let terminator: httpTerminator.HttpTerminator
     let tap: Tap
     let requests: {
       path: string
@@ -99,14 +98,15 @@ describe('Tap client', () => {
         })
       })
 
-      server = app.listen()
+      const server = app.listen()
       await once(server, 'listening')
       const { port } = server.address() as AddressInfo
+      terminator = httpTerminator.createHttpTerminator({ server })
       tap = new Tap(`http://localhost:${port}`, { adminPassword: 'secret' })
     })
 
     afterAll(async () => {
-      await new Promise((resolve) => server.close(resolve))
+      await terminator?.terminate()
     })
 
     beforeEach(() => {
@@ -166,10 +166,7 @@ describe('Tap client', () => {
   })
 
   describe('HTTP error handling', () => {
-    let server: http.Server<
-      typeof http.IncomingMessage,
-      typeof http.ServerResponse
-    >
+    let terminator: httpTerminator.HttpTerminator
     let tap: Tap
 
     beforeAll(async () => {
@@ -184,14 +181,15 @@ describe('Tap client', () => {
         res.status(500).send('Internal Server Error')
       })
 
-      server = app.listen()
+      const server = app.listen(0)
       await once(server, 'listening')
       const { port } = server.address() as AddressInfo
+      terminator = httpTerminator.createHttpTerminator({ server })
       tap = new Tap(`http://localhost:${port}`)
     })
 
     afterAll(async () => {
-      await new Promise((resolve) => server.close(resolve))
+      await terminator?.terminate()
     })
 
     it('throws on addRepos failure', async () => {