feat: remove test dependency (#239)

* [create-pull-request] automated change

* feat: update node to v20

* chore: wip

* feat: remove test dep

.devcontainer/devcontainer.json

@@ -1,6 +1,6 @@
 {
   "name": "Node.js & TypeScript",
-  "image": "mcr.microsoft.com/devcontainers/typescript-node:16-bullseye",
+  "image": "mcr.microsoft.com/devcontainers/typescript-node:20",
   "features": {},
   "forwardPorts": [],
   "postCreateCommand": "yarn install",

README.md

@@ -369,10 +369,9 @@ this by setting the `private` property in `package.json` to `false`.
 
 _Scope: repo or workspace_
 
-This plugin enables testing with the [test](https://www.npmjs.com/package/test)
-package, which is a port of the native
-[node:test](https://nodejs.org/api/test.html) which is only available in Node
-≥ 18. It uses [tap](https://node-tap.org) for formatting the TAP output.
+This plugin enables testing with the native
+[node --test](https://nodejs.org/api/test.html) functionality. It uses
+[tap](https://node-tap.org) for formatting the TAP output.
 
 When the `typescript` plugin is also installed, it will use
 [ts-node](https://typestrong.org/ts-node/) to load TypeScript test files.

TODO.md

@@ -2,7 +2,7 @@
 
 | Filename                                                                   | line # | TODO                                                             |
 | :------------------------------------------------------------------------- | :----: | :--------------------------------------------------------------- |
-| [packages/cli/test/add.test.ts](packages/cli/test/add.test.ts#L55)         |   55   | remove when https://github.com/nodejs/node/issues/47614 resolves |
+| [packages/cli/test/add.test.ts](packages/cli/test/add.test.ts#L52)         |   52   | remove when https://github.com/nodejs/node/issues/47614 resolves |
 | [packages/core/src/license.ts](packages/core/src/license.ts#L43)           |   43   | Contents is not perfect yet, e.g.:                               |
 | [packages/core/src/yarnrc.ts](packages/core/src/yarnrc.ts#L23)             |   23   | etc, fix later                                                   |
 | [packages/plugins/src/jest/jest.ts](packages/plugins/src/jest/jest.ts#L31) |   31   | install jest without ts-jest                                     |

packages/cli/package.json

@@ -19,7 +19,7 @@
     "clean": "rm -rf dist && rm -rf types",
     "prepublish": "yarn build && cp ../../README.md .",
     "start": "node moker.js",
-    "test": "NODE_OPTIONS='--loader=ts-node/esm --no-warnings' node--test test/*.test.ts | NODE_OPTIONS='--loader=ts-node/esm --no-warnings' tap --no-coverage"
+    "test": "NODE_OPTIONS='--loader=ts-node/esm --no-warnings' node --test test/*.test.ts | NODE_OPTIONS='--loader=ts-node/esm --no-warnings' tap --no-coverage"
   },
   "dependencies": {
     "@mokr/core": "workspace:*",
@@ -30,7 +30,6 @@
   "devDependencies": {
     "@types/node": "20.6.2",
     "tap": "16.3.8",
-    "test": "3.3.0",
     "ts-node": "10.9.1",
     "typescript": "5.2.2"
   },

packages/cli/test/add.test.ts

@@ -1,12 +1,9 @@
 import { createDirectory, isDirectory, writePackage } from "@mokr/core";
 import assert from "node:assert";
+import { beforeEach, describe, it } from "node:test";
 import { temporaryDirectory } from "tempy";
-import test from "test";
 import cli from "../src/cli.js";
 
-/* @ts-expect-error */
-const { describe, beforeEach, it } = test;
-
 describe("add", () => {
   let tempDir: string;
 

packages/core/package.json

@@ -17,7 +17,7 @@
     "build:watch": "tsc --watch",
     "clean": "rm -rf dist && rm -rf types",
     "prepublish": "yarn build",
-    "test": "NODE_OPTIONS='--loader=ts-node/esm --no-warnings' node--test test/*.test.ts | NODE_OPTIONS='--loader=ts-node/esm --no-warnings' tap --no-coverage"
+    "test": "NODE_OPTIONS='--loader=ts-node/esm --no-warnings' node --test test/*.test.ts | NODE_OPTIONS='--loader=ts-node/esm --no-warnings' tap --no-coverage"
   },
   "dependencies": {
     "chalk": "5.3.0",
@@ -34,7 +34,6 @@
     "@types/node": "20.6.2",
     "tap": "16.3.8",
     "tempy": "3.1.0",
-    "test": "3.3.0",
     "ts-node": "10.9.1",
     "typescript": "5.2.2"
   },

packages/core/test/io.test.ts

@@ -1,10 +1,7 @@
 import assert from "node:assert";
-import test from "test";
+import { beforeEach, describe, it } from "node:test";
 import { hasEncounteredErrors, resetState, task } from "../src/io.js";
 
-/* @ts-expect-error */
-const { beforeEach, describe, it } = test;
-
 describe("task", () => {
   beforeEach(() => resetState());
 

packages/core/test/json.test.ts

@@ -1,12 +1,9 @@
 import assert from "node:assert";
 import { writeFileSync } from "node:fs";
+import { describe, it } from "node:test";
 import { temporaryFile } from "tempy";
-import test from "test";
 import { readJson } from "../src/json.js";
 
-/* @ts-expect-error */
-const { describe, it } = test;
-
 describe("readJson", () => {
   it("should read a JSON file", async () => {
     const path = temporaryFile();

packages/plugins/src/test/test.ts

@@ -14,7 +14,7 @@ async function install({ directory }: PluginArgs) {
 
   enqueueInstallDependency({
     directory,
-    identifier: ["test", "tap"],
+    identifier: ["tap"],
     dev: true,
   });
 
@@ -23,7 +23,7 @@ async function install({ directory }: PluginArgs) {
       directory,
       data: {
         scripts: {
-          test: "node--test | tap --no-coverage",
+          test: "node --test | tap --no-coverage",
         },
       },
     });
@@ -38,7 +38,7 @@ async function install({ directory }: PluginArgs) {
       directory,
       data: {
         scripts: {
-          test: "NODE_OPTIONS='--loader=ts-node/esm --no-warnings' node--test test/*.test.ts | NODE_OPTIONS='--loader=ts-node/esm --no-warnings' tap --no-coverage",
+          test: "NODE_OPTIONS='--loader=ts-node/esm --no-warnings' node --test test/*.test.ts | NODE_OPTIONS='--loader=ts-node/esm --no-warnings' tap --no-coverage",
         },
       },
     });
@@ -57,7 +57,7 @@ async function install({ directory }: PluginArgs) {
 }
 
 async function remove({ directory }: PluginArgs) {
-  enqueueRemoveDependency({ directory, identifier: ["test", "ts-node"] });
+  enqueueRemoveDependency({ directory, identifier: ["tap", "ts-node"] });
 
   warning("Please review package.json manually");
 }

packages/templates/src/lib.ts

@@ -8,7 +8,7 @@ import { join } from "path";
 
 async function apply({ directory }: TemplateArgs) {
   await installPlugin({ directory, name: "typescript" });
-  await installPlugin({ directory, name: "jest" });
+  await installPlugin({ directory, name: "test" });
 
   await writeFile({
     path: join(directory, "src/sum.ts"),
@@ -25,12 +25,14 @@ export function sum(a: number, b: number) {
   });
 
   await writeFile({
-    path: join(directory, "tests/sum.test.ts"),
+    path: join(directory, "test/sum.test.ts"),
     contents: `
+import assert from "node:assert";
+import test from "node:test";
 import { sum } from "../src/sum.js";
 
 test("adds 1 + 2 to equal 3", () => {
-  expect(sum(1, 2)).toBe(3);
+  assert.strictEqual(sum(1, 2), 3);
 });
 `,
   });