Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Dec 5, 2025

Migrate test suite from tap to Node.js built-in node:test module.

Changes

  • test/simple-test.ts: Use node:test and node:assert instead of tap
  • package.json:
    • Update test-only script to node --import tsx --test test/*.ts
    • Remove tap, ts-node, snazzy, standardx dev dependencies
    • Add tsx for TypeScript test execution
  • Remove .taprc: No longer needed

Before/After

// Before
import { test } from 'tap';
test('example', async ({ equal }) => {
  equal(queue.size, 0);
});

// After
import { test } from 'node:test';
import assert from 'node:assert';
test('example', async () => {
  assert.strictEqual(queue.size, 0);
});
Original prompt

replace tap tests with node:tests


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

- Update test/simple-test.ts to use node:test and node:assert
- Replace tap with tsx for TypeScript test execution
- Remove tap, ts-node, snazzy, and standardx dependencies
- Add tsx as dev dependency for test execution
- Remove .taprc configuration file
- Update test-only script to use node --import tsx --test

Co-authored-by: metcoder95 <[email protected]>
Copilot AI changed the title [WIP] Replace tap tests with node tests Replace tap tests with node:test Dec 5, 2025
Copilot AI requested a review from metcoder95 December 5, 2025 09:40
@metcoder95 metcoder95 marked this pull request as ready for review December 6, 2025 10:29
@metcoder95 metcoder95 merged commit f1746a8 into current Dec 6, 2025
21 of 23 checks passed
@metcoder95 metcoder95 deleted the copilot/replace-tap-tests-with-node-tests branch December 6, 2025 10:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants