feat: use util.styleText in the test runner instead of picocolors

lib/package_json.test.ts

@@ -4,7 +4,6 @@ import { assertEquals } from "@std/assert";
 import { getPackageJson, type GetPackageJsonOptions } from "./package_json.ts";
 
 const versions = {
-  picocolors: "^1.0.0",
   nodeTypes: "^20.9.0",
   tsLib: "^2.6.2",
 };
@@ -64,7 +63,6 @@ Deno.test("single entrypoint", () => {
     },
     devDependencies: {
       "@types/node": versions.nodeTypes,
-      "picocolors": versions.picocolors,
       "test-dep": "0.1.0",
       "@deno/shim-deno": "~0.1.0",
     },
@@ -519,7 +517,6 @@ Deno.test("peer dependencies", () => {
         },
         devDependencies: {
           "@types/node": versions.nodeTypes,
-          "picocolors": versions.picocolors,
           "test-dep": "0.1.0",
           "@deno/shim-deno": "~0.1.0",
         },

lib/package_json.ts

@@ -71,11 +71,6 @@ export function getPackageJson({
   };
   const testDevDependencies = testEnabled
     ? ({
-      ...(!Object.keys(dependencies).includes("picocolors")
-        ? {
-          "picocolors": "^1.0.0",
-        }
-        : {}),
       // add dependencies from transform
       ...Object.fromEntries(
         // ignore peer dependencies on this

lib/test_runner/get_test_runner_code.test.ts

@@ -16,7 +16,7 @@ Deno.test("gets code when no shim used", () => {
   });
   assertEquals(
     code,
-    `const pc = require("picocolors");
+    `const { styleText } = require("util");
 const process = require("process");
 
 const filePaths = [
@@ -30,7 +30,7 @@ async function main() {
     }
 
     const scriptPath = "./script/" + filePath;
-    console.log("Running tests in " + pc.underline(scriptPath) + "...\\n");
+    console.log("Running tests in " + styleText("underline", scriptPath) + "...\\n");
     process.chdir(__dirname + "/script");
     try {
       require(scriptPath);
@@ -40,7 +40,7 @@ async function main() {
     }
 
     const esmPath = "./esm/" + filePath;
-    console.log("\\nRunning tests in " + pc.underline(esmPath) + "...\\n");
+    console.log("\\nRunning tests in " + styleText("underline", esmPath) + "...\\n");
     process.chdir(__dirname + "/esm");
     await import(esmPath);
   }
@@ -60,7 +60,7 @@ Deno.test("gets code when shim used", () => {
   });
   assertEquals(
     code,
-    `const pc = require("picocolors");
+    `const { styleText } = require("util");
 const process = require("process");
 const { pathToFileURL } = require("url");
 const { testDefinitions } = require("test-shim-package/test-internals");
@@ -73,15 +73,15 @@ const filePaths = [
 async function main() {
   const testContext = {
     process,
-    pc,
+    styleText,
   };
   for (const [i, filePath] of filePaths.entries()) {
     if (i > 0) {
       console.log("");
     }
 
     const scriptPath = "./script/" + filePath;
-    console.log("Running tests in " + pc.underline(scriptPath) + "...\\n");
+    console.log("Running tests in " + styleText("underline", scriptPath) + "...\\n");
     process.chdir(__dirname + "/script");
     const scriptTestContext = {
       origin: pathToFileURL(filePath).toString(),
@@ -96,7 +96,7 @@ async function main() {
     await runTestDefinitions(testDefinitions.splice(0, testDefinitions.length), scriptTestContext);
 
     const esmPath = "./esm/" + filePath;
-    console.log("\\nRunning tests in " + pc.underline(esmPath) + "...\\n");
+    console.log("\\nRunning tests in " + styleText("underline", esmPath) + "...\\n");
     process.chdir(__dirname + "/esm");
     const esmTestContext = {
       origin: pathToFileURL(filePath).toString(),
@@ -163,7 +163,7 @@ Deno.test("gets code when a preload module is used", () => {
   });
   assertEquals(
     code,
-    `const pc = require("picocolors");
+    `const { styleText } = require("util");
 const process = require("process");
 
 const filePaths = [
@@ -187,7 +187,7 @@ async function main() {
     }
 
     const scriptPath = "./script/" + filePath;
-    console.log("Running tests in " + pc.underline(scriptPath) + "...\\n");
+    console.log("Running tests in " + styleText("underline", scriptPath) + "...\\n");
     process.chdir(__dirname + "/script");
     try {
       require(scriptPath);
@@ -197,7 +197,7 @@ async function main() {
     }
 
     const esmPath = "./esm/" + filePath;
-    console.log("\\nRunning tests in " + pc.underline(esmPath) + "...\\n");
+    console.log("\\nRunning tests in " + styleText("underline", esmPath) + "...\\n");
     process.chdir(__dirname + "/esm");
     await import(esmPath);
   }
@@ -218,7 +218,7 @@ Deno.test("gets code when a preload module is used without cjs", () => {
   });
   assertEquals(
     code,
-    `const pc = require("picocolors");
+    `const { styleText } = require("util");
 const process = require("process");
 
 const filePaths = [
@@ -235,7 +235,7 @@ async function main() {
     }
 
     const esmPath = "./esm/" + filePath;
-    console.log("\\nRunning tests in " + pc.underline(esmPath) + "...\\n");
+    console.log("\\nRunning tests in " + styleText("underline", esmPath) + "...\\n");
     process.chdir(__dirname + "/esm");
     await import(esmPath);
   }
@@ -256,7 +256,7 @@ Deno.test("gets code when a preload module is used without esm", () => {
   });
   assertEquals(
     code,
-    `const pc = require("picocolors");
+    `const { styleText } = require("util");
 const process = require("process");
 
 const filePaths = [
@@ -278,7 +278,7 @@ async function main() {
     }
 
     const scriptPath = "./script/" + filePath;
-    console.log("Running tests in " + pc.underline(scriptPath) + "...\\n");
+    console.log("Running tests in " + styleText("underline", scriptPath) + "...\\n");
     process.chdir(__dirname + "/script");
     try {
       require(scriptPath);
@@ -322,7 +322,7 @@ Deno.test("gets code when cjs is not used", () => {
   });
   assertEquals(
     code,
-    `const pc = require("picocolors");
+    `const { styleText } = require("util");
 const process = require("process");
 
 const filePaths = [
@@ -336,7 +336,7 @@ async function main() {
     }
 
     const esmPath = "./esm/" + filePath;
-    console.log("\\nRunning tests in " + pc.underline(esmPath) + "...\\n");
+    console.log("\\nRunning tests in " + styleText("underline", esmPath) + "...\\n");
     process.chdir(__dirname + "/esm");
     await import(esmPath);
   }
@@ -356,7 +356,7 @@ Deno.test("gets code when esm is not used", () => {
   });
   assertEquals(
     code,
-    `const pc = require("picocolors");
+    `const { styleText } = require("util");
 const process = require("process");
 
 const filePaths = [
@@ -370,7 +370,7 @@ async function main() {
     }
 
     const scriptPath = "./script/" + filePath;
-    console.log("Running tests in " + pc.underline(scriptPath) + "...\\n");
+    console.log("Running tests in " + styleText("underline", scriptPath) + "...\\n");
     process.chdir(__dirname + "/script");
     try {
       require(scriptPath);

lib/test_runner/get_test_runner_code.ts

@@ -17,7 +17,7 @@ export function getTestRunnerCode(options: {
     ? undefined
     : toJsFilePath(options.preloadEntryPoint);
   const writer = createWriter();
-  writer.writeLine(`const pc = require("picocolors");`)
+  writer.writeLine(`const { styleText } = require("util");`)
     .writeLine(`const process = require("process");`);
   if (usesDenoTest) {
     writer.writeLine(`const { pathToFileURL } = require("url");`);
@@ -61,7 +61,7 @@ export function getTestRunnerCode(options: {
     if (usesDenoTest) {
       writer.write("const testContext = ").inlineBlock(() => {
         writer.writeLine("process,");
-        writer.writeLine("pc,");
+        writer.writeLine("styleText,");
       }).write(";").newLine();
     }
     if (isolateTestFiles) {
@@ -131,7 +131,7 @@ export function getTestRunnerCode(options: {
     if (options.includeScriptModule) {
       writer.writeLine(`const scriptPath = "./script/" + filePath;`);
       writer.writeLine(
-        `console.log("Running tests in " + pc.underline(scriptPath) + "...\\n");`,
+        `console.log("Running tests in " + styleText("underline", scriptPath) + "...\\n");`,
       );
       writer.writeLine(`process.chdir(__dirname + "/script");`);
       if (usesDenoTest) {
@@ -159,7 +159,7 @@ export function getTestRunnerCode(options: {
       }
       writer.writeLine(`const esmPath = "./esm/" + filePath;`);
       writer.writeLine(
-        `console.log("\\nRunning tests in " + pc.underline(esmPath) + "...\\n");`,
+        `console.log("\\nRunning tests in " + styleText("underline", esmPath) + "...\\n");`,
       );
       writer.writeLine(`process.chdir(__dirname + "/esm");`);
       if (usesDenoTest) {

lib/test_runner/test_runner.test.ts

@@ -131,16 +131,17 @@ function getContext() {
       return output;
     },
     origin: "file:///file.ts",
-    pc: {
-      red(text: string) {
-        return `R${text}R`;
-      },
-      green(text: string) {
-        return `G${text}G`;
-      },
-      gray(text: string) {
-        return `Y${text}Y`;
-      },
+    styleText(format: string, text: string) {
+      switch (format) {
+        case "red":
+          return `R${text}R`;
+        case "green":
+          return `G${text}G`;
+        case "gray":
+          return `Y${text}Y`;
+        default:
+          return text;
+      }
     },
     process: {
       stdout: {

lib/test_runner/test_runner.ts

@@ -1,11 +1,5 @@
 // Copyright 2018-2024 the Deno authors. MIT license.
 
-export interface Picocolors {
-  green(text: string): string;
-  red(text: string): string;
-  gray(text: string): string;
-}
-
 export interface NodeProcess {
   stdout: {
     write(text: string): void;
@@ -14,7 +8,8 @@ export interface NodeProcess {
 }
 
 export interface RunTestDefinitionsOptions {
-  pc: Picocolors;
+  /** `util.styleText` from Node.js. */
+  styleText(format: string, text: string): string;
   process: NodeProcess;
   /** The file the tests are running in. */
   origin: string;
@@ -54,7 +49,9 @@ export async function runTestDefinitions(
   for (const definition of testDefinitions) {
     options.process.stdout.write("test " + definition.name + " ...");
     if (definition.ignore) {
-      options.process.stdout.write(` ${options.pc.gray("ignored")}\n`);
+      options.process.stdout.write(
+        ` ${options.styleText("gray", "ignored")}\n`,
+      );
       continue;
     }
     const context = getTestContext(definition, undefined);
@@ -193,12 +190,12 @@ export async function runTestDefinitions(
   function getStatusText(status: TestContext["status"]) {
     switch (status) {
       case "ok":
-        return options.pc.green(status);
+        return options.styleText("green", status);
       case "fail":
       case "pending":
-        return options.pc.red(status);
+        return options.styleText("red", status);
       case "ignored":
-        return options.pc.gray(status);
+        return options.styleText("gray", status);
       default: {
         const _assertNever: never = status;
         return status;