test: import unit specs from node:test

theorderingmachine · theorderingmachine · commit f16a215d642a · 2026-04-08T13:37:02.000-07:00
diff --git a/.github/maintainers_guide.md b/.github/maintainers_guide.md
@@ -21,7 +21,7 @@ Test code should be written in syntax that runs on the oldest supported Node.js
 
 #### Debugging
 
-A useful trick for debugging inside tests is to use the Chrome Debugging Protocol feature of Node.js to set breakpoints and interactively debug. In order to do this you should have already linted the source (`npm run lint`), manually. You can then run a specific test file with Node.js's test runner, for example: `node --inspect-brk --require ts-node/register --require source-map-support/register --require ./test/unit/node-test-globals.cjs --test test/unit/{test-name}.spec.ts` (replace `{test-name}` with an actual test file path).
+A useful trick for debugging inside tests is to use the Chrome Debugging Protocol feature of Node.js to set breakpoints and interactively debug. In order to do this you should have already linted the source (`npm run lint`), manually. You can then run a specific test file with Node.js's test runner, for example: `node --inspect-brk --require ts-node/register --require source-map-support/register --test test/unit/{test-name}.spec.ts` (replace `{test-name}` with an actual test file path).
 
 #### Local Development
 
diff --git a/.vscode/launch.json b/.vscode/launch.json
@@ -16,15 +16,14 @@
         "ts-node/register",
         "--require",
         "source-map-support/register",
-        "--require",
-        "./test/unit/node-test-globals.cjs",
         "--test",
         "test/unit/**/*.spec.ts"
       ],
       "cwd": "${workspaceFolder}",
       "env": {
         "NODE_ENV": "testing",
-        "TS_NODE_PROJECT": "tsconfig.json"
+        "TS_NODE_FILES": "true",
+        "TS_NODE_PROJECT": "tsconfig.test.json"
       },
       "skipFiles": ["<node_internals>/**"]
     }
diff --git a/AGENTS.md b/AGENTS.md
@@ -173,7 +173,7 @@ test/types/         # tsd type tests
 - **Test files** use `*.spec.ts` suffix
 - **Assertions** use the local `test/unit/helpers/assert.ts` compatibility helper backed by Node.js `assert` and Sinon assertions
 - **Mocking** uses sinon (`stub`, `spy`, `fake`) and proxyquire for module-level dependency replacement
-- **Test config** lives in `package.json` scripts plus `test/unit/node-test-globals.cjs`
+- **Test config** lives in `package.json` scripts plus direct `node:test` imports in the spec files
 - **Where to put new tests:** Mirror the source structure. For `src/Foo.ts`, add `test/unit/Foo.spec.ts`. For `src/receivers/Bar.ts`, add `test/unit/receivers/Bar.spec.ts`.
 
 ### CI
diff --git a/package.json b/package.json
@@ -32,8 +32,8 @@
     "lint": "npx @biomejs/biome check docs src test examples",
     "lint:fix": "npx @biomejs/biome check --write docs src test examples",
     "test": "npm run build && npm run lint && npm run test:types && npm run test:coverage",
-    "test:unit": "TS_NODE_PROJECT=tsconfig.test.json TS_NODE_FILES=true node --require ts-node/register --require source-map-support/register --require ./test/unit/node-test-globals.cjs --test test/unit/**/*.spec.ts",
-    "test:coverage": "TS_NODE_PROJECT=tsconfig.test.json TS_NODE_FILES=true node --experimental-test-coverage --require ts-node/register --require source-map-support/register --require ./test/unit/node-test-globals.cjs --test test/unit/**/*.spec.ts",
+    "test:unit": "TS_NODE_PROJECT=tsconfig.test.json TS_NODE_FILES=true node --require ts-node/register --require source-map-support/register --test test/unit/**/*.spec.ts",
+    "test:coverage": "TS_NODE_PROJECT=tsconfig.test.json TS_NODE_FILES=true node --experimental-test-coverage --require ts-node/register --require source-map-support/register --test test/unit/**/*.spec.ts",
     "test:types": "tsd --files test/types",
     "watch": "npx nodemon --watch 'src' --ext 'ts' --exec npm run build"
   },
diff --git a/test/unit/App/basic.spec.ts b/test/unit/App/basic.spec.ts
@@ -17,6 +17,7 @@ import {
   withNoopWebClient,
   withSuccessfulBotUserFetchingWebClient,
 } from '../helpers';
+import { describe, it } from 'node:test';
 
 const fakeAppToken = 'xapp-1234';
 const fakeBotId = 'B_FAKE_BOT_ID';
diff --git a/test/unit/App/middlewares/arguments.spec.ts b/test/unit/App/middlewares/arguments.spec.ts
@@ -29,6 +29,7 @@ import {
   withSetStatus,
   withSuccessfulBotUserFetchingWebClient,
 } from '../../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 describe('App middleware and listener arguments', () => {
   let fakeReceiver: FakeReceiver;
diff --git a/test/unit/App/middlewares/global.spec.ts b/test/unit/App/middlewares/global.spec.ts
@@ -19,6 +19,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 describe('App global middleware Processing', () => {
   let fakeReceiver: FakeReceiver;
diff --git a/test/unit/App/middlewares/ignore-self.spec.ts b/test/unit/App/middlewares/ignore-self.spec.ts
@@ -15,6 +15,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/App/middlewares/listener.spec.ts b/test/unit/App/middlewares/listener.spec.ts
@@ -3,6 +3,7 @@ import sinon, { type SinonSpy } from 'sinon';
 import type App from '../../../../src/App';
 import { ErrorCode, isCodedError } from '../../../../src/errors';
 import { FakeReceiver, createDummyReceiverEvent, importApp } from '../../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 describe('App listener middleware processing', () => {
   let fakeReceiver: FakeReceiver;
diff --git a/test/unit/App/routing-action.spec.ts b/test/unit/App/routing-action.spec.ts
@@ -15,6 +15,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/App/routing-assistant.spec.ts b/test/unit/App/routing-assistant.spec.ts
@@ -16,6 +16,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/App/routing-command.spec.ts b/test/unit/App/routing-command.spec.ts
@@ -13,6 +13,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/App/routing-event.spec.ts b/test/unit/App/routing-event.spec.ts
@@ -14,6 +14,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/App/routing-function.spec.ts b/test/unit/App/routing-function.spec.ts
@@ -14,6 +14,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/App/routing-message.spec.ts b/test/unit/App/routing-message.spec.ts
@@ -13,6 +13,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/App/routing-options.spec.ts b/test/unit/App/routing-options.spec.ts
@@ -13,6 +13,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/App/routing-shortcut.spec.ts b/test/unit/App/routing-shortcut.spec.ts
@@ -13,6 +13,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/App/routing-view.spec.ts b/test/unit/App/routing-view.spec.ts
@@ -14,6 +14,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 function buildOverrides(secondOverrides: Override[]): Override {
   return mergeOverrides(
diff --git a/test/unit/Assistant.spec.ts b/test/unit/Assistant.spec.ts
@@ -24,6 +24,7 @@ import {
   wrapMiddleware,
 } from './helpers';
 import { team } from './helpers/events';
+import { describe, it } from 'node:test';
 
 function importAssistant(overrides: Override = {}): typeof import('../../src/Assistant') {
   const absolutePath = path.resolve(__dirname, '../../src/Assistant');
diff --git a/test/unit/AssistantThreadContextStore.spec.ts b/test/unit/AssistantThreadContextStore.spec.ts
@@ -4,6 +4,7 @@ import sinon from 'sinon';
 import { extractThreadInfo } from '../../src/Assistant';
 import { DefaultThreadContextStore } from '../../src/AssistantThreadContextStore';
 import { createDummyAssistantThreadStartedEventMiddlewareArgs, wrapMiddleware } from './helpers';
+import { describe, it } from 'node:test';
 
 describe('DefaultThreadContextStore class', () => {
   describe('get', () => {
diff --git a/test/unit/CustomFunction.spec.ts b/test/unit/CustomFunction.spec.ts
@@ -8,6 +8,7 @@ import {
 import { CustomFunctionInitializationError } from '../../src/errors';
 import { autoAcknowledge, matchEventType, onlyEvents } from '../../src/middleware/builtin';
 import type { Middleware } from '../../src/types';
+import { describe, it } from 'node:test';
 
 const MOCK_FN = async () => {};
 const MOCK_FN_2 = async () => {};
diff --git a/test/unit/WorkflowStep.spec.ts b/test/unit/WorkflowStep.spec.ts
@@ -15,6 +15,7 @@ import {
 import { WorkflowStepInitializationError } from '../../src/errors';
 import type { AllMiddlewareArgs, AnyMiddlewareArgs, Middleware, WorkflowStepEdit } from '../../src/types';
 import { type Override, noopVoid, proxyquire } from './helpers';
+import { describe, it } from 'node:test';
 
 function importWorkflowStep(overrides: Override = {}): typeof import('../../src/WorkflowStep') {
   const absolutePath = path.resolve(__dirname, '../../src/WorkflowStep');
diff --git a/test/unit/context/create-function-complete.spec.ts b/test/unit/context/create-function-complete.spec.ts
@@ -2,6 +2,7 @@ import { WebClient } from '@slack/web-api';
 import { assert } from '../helpers/assert';
 import sinon from 'sinon';
 import { createFunctionComplete } from '../../../src/context';
+import { describe, it } from 'node:test';
 
 describe('createFunctionComplete', () => {
   it('complete should call functions.completeSuccess', async () => {
diff --git a/test/unit/context/create-function-fail.spec.ts b/test/unit/context/create-function-fail.spec.ts
@@ -2,6 +2,7 @@ import { WebClient } from '@slack/web-api';
 import { assert } from '../helpers/assert';
 import sinon from 'sinon';
 import { createFunctionFail } from '../../../src/context';
+import { describe, it } from 'node:test';
 
 describe('createFunctionFail', () => {
   it('fail should call functions.completeError', async () => {
diff --git a/test/unit/context/create-respond.spec.ts b/test/unit/context/create-respond.spec.ts
@@ -2,6 +2,7 @@ import type { AxiosInstance } from 'axios';
 import { assert } from '../helpers/assert';
 import sinon from 'sinon';
 import { createRespond } from '../../../src/context';
+import { describe, it } from 'node:test';
 
 describe('createRespond', () => {
   it('should post to the response URL with text when given a string', async () => {
diff --git a/test/unit/context/create-say-stream.spec.ts b/test/unit/context/create-say-stream.spec.ts
@@ -3,6 +3,7 @@ import { assert } from '../helpers/assert';
 import sinon from 'sinon';
 import { createSayStream } from '../../../src/context';
 import type { Context } from '../../../src/types';
+import { afterEach, beforeEach, describe, it } from 'node:test';
 
 describe('createSayStream', () => {
   const sandbox = sinon.createSandbox();
diff --git a/test/unit/context/create-say.spec.ts b/test/unit/context/create-say.spec.ts
@@ -2,6 +2,7 @@ import { WebClient } from '@slack/web-api';
 import { assert } from '../helpers/assert';
 import sinon from 'sinon';
 import { createSay } from '../../../src/context';
+import { describe, it } from 'node:test';
 
 describe('createSay', () => {
   it('should call chat.postMessage with text when given a string', async () => {
diff --git a/test/unit/context/create-set-status.spec.ts b/test/unit/context/create-set-status.spec.ts
@@ -2,6 +2,7 @@ import { WebClient } from '@slack/web-api';
 import { assert } from '../helpers/assert';
 import sinon from 'sinon';
 import { createSetStatus } from '../../../src/context';
+import { afterEach, beforeEach, describe, it } from 'node:test';
 
 describe('createSetStatus', () => {
   const sandbox = sinon.createSandbox();
diff --git a/test/unit/conversation-store.spec.ts b/test/unit/conversation-store.spec.ts
@@ -5,6 +5,7 @@ import { assert, AssertionError } from './helpers/assert';
 import sinon, { type SinonSpy } from 'sinon';
 import type { AnyMiddlewareArgs, Context, NextFn } from '../../src/types';
 import { type Override, createFakeLogger, delay, proxyquire } from './helpers';
+import { describe, it } from 'node:test';
 
 /* Testing Harness */
 
diff --git a/test/unit/errors.spec.ts b/test/unit/errors.spec.ts
@@ -10,6 +10,7 @@ import {
   UnknownError,
   asCodedError,
 } from '../../src/errors';
+import { describe, it } from 'node:test';
 
 describe('Errors', () => {
   it('has errors matching codes', () => {
diff --git a/test/unit/helpers.spec.ts b/test/unit/helpers.spec.ts
@@ -11,6 +11,7 @@ import {
   isRecord,
 } from '../../src/helpers';
 import type { AnyMiddlewareArgs, KnownEventFromType, ReceiverEvent, SlackEventMiddlewareArgs } from '../../src/types';
+import { describe, it } from 'node:test';
 
 describe('Helpers', () => {
   describe('getTypeAndConversation()', () => {
diff --git a/test/unit/middleware/builtin.spec.ts b/test/unit/middleware/builtin.spec.ts
@@ -16,6 +16,7 @@ import {
   proxyquire,
   wrapMiddleware,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 interface DummyContext extends Context {
   matches?: RegExpExecArray;
diff --git a/test/unit/node-test-globals.cjs b/test/unit/node-test-globals.cjs
diff --git a/test/unit/node-test-globals.d.ts b/test/unit/node-test-globals.d.ts
diff --git a/test/unit/receivers/AwsLambdaReceiver.spec.ts b/test/unit/receivers/AwsLambdaReceiver.spec.ts
@@ -12,6 +12,7 @@ import {
   withNoopAppMetadata,
   withNoopWebClient,
 } from '../helpers';
+import { describe, it } from 'node:test';
 
 const fakeAuthTestResponse = {
   ok: true,
diff --git a/test/unit/receivers/ExpressReceiver.spec.ts b/test/unit/receivers/ExpressReceiver.spec.ts
@@ -30,6 +30,7 @@ import {
   withHttpCreateServer,
   withHttpsCreateServer,
 } from '../helpers';
+import { afterEach, beforeEach, after, describe, it } from 'node:test';
 
 // Loading the system under test using overrides
 function importExpressReceiver(
diff --git a/test/unit/receivers/HTTPModuleFunctions.spec.ts b/test/unit/receivers/HTTPModuleFunctions.spec.ts
@@ -2,6 +2,7 @@ import { createHmac } from 'node:crypto';
 import { IncomingMessage, ServerResponse } from 'node:http';
 import { assert } from '../helpers/assert';
 import sinon from 'sinon';
+import { describe, it } from 'node:test';
 
 import { AuthorizationError, HTTPReceiverDeferredRequestError, ReceiverMultipleAckError } from '../../../src/errors';
 import type { BufferedIncomingMessage } from '../../../src/receivers/BufferedIncomingMessage';
diff --git a/test/unit/receivers/HTTPReceiver.spec.ts b/test/unit/receivers/HTTPReceiver.spec.ts
@@ -21,6 +21,7 @@ import {
   withHttpCreateServer,
   withHttpsCreateServer,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 // Loading the system under test using overrides
 function importHTTPReceiver(overrides: Override = {}): typeof import('../../../src/receivers/HTTPReceiver').default {
diff --git a/test/unit/receivers/HTTPResponseAck.spec.ts b/test/unit/receivers/HTTPResponseAck.spec.ts
@@ -7,6 +7,7 @@ import * as HTTPModuleFunctions from '../../../src/receivers/HTTPModuleFunctions
 import { HTTPResponseAck } from '../../../src/receivers/HTTPResponseAck';
 import type { ResponseAck } from '../../../src/types';
 import { createFakeLogger } from '../helpers';
+import { afterEach, beforeEach, describe, it } from 'node:test';
 
 describe('HTTPResponseAck', async () => {
   let setTimeoutSpy: sinon.SinonSpy;
diff --git a/test/unit/receivers/SocketModeFunctions.spec.ts b/test/unit/receivers/SocketModeFunctions.spec.ts
@@ -3,6 +3,7 @@ import { AuthorizationError, ReceiverMultipleAckError } from '../../../src/error
 import { defaultProcessEventErrorHandler } from '../../../src/receivers/SocketModeFunctions';
 import type { ReceiverEvent } from '../../../src/types';
 import { createFakeLogger } from '../helpers';
+import { describe, it } from 'node:test';
 
 describe('SocketModeFunctions', async () => {
   describe('Error handlers for event processing', async () => {
diff --git a/test/unit/receivers/SocketModeReceiver.spec.ts b/test/unit/receivers/SocketModeReceiver.spec.ts
@@ -22,6 +22,7 @@ import {
   withHttpCreateServer,
   withHttpsCreateServer,
 } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 // Loading the system under test using overrides
 function importSocketModeReceiver(
diff --git a/test/unit/receivers/SocketModeResponseAck.spec.ts b/test/unit/receivers/SocketModeResponseAck.spec.ts
@@ -4,6 +4,7 @@ import { expectType } from 'tsd';
 import { SocketModeResponseAck } from '../../../src/receivers/SocketModeResponseAck';
 import type { ResponseAck } from '../../../src/types';
 import { createFakeLogger } from '../helpers';
+import { beforeEach, describe, it } from 'node:test';
 
 describe('SocketModeResponseAck', async () => {
   const fakeSocketModeClientAck = sinon.fake();
diff --git a/test/unit/receivers/verify-request.spec.ts b/test/unit/receivers/verify-request.spec.ts
@@ -1,6 +1,7 @@
 import { createHmac } from 'node:crypto';
 import { assert } from '../helpers/assert';
 import { isValidSlackRequest, verifySlackRequest } from '../../../src/receivers/verify-request';
+import { describe, it } from 'node:test';
 
 describe('Request verification', async () => {
   const signingSecret = 'secret';
