Change how we print the titles of the test tasks

NomicFoundation · Oct 28, 2024 · f160ce0 · f160ce0
1 parent 9786e45
commit f160ce0
Show file tree

Hide file tree

Showing 4 changed files with 18 additions and 3 deletions.
diff --git a/v-next/hardhat-mocha-test-runner/src/task-action.ts b/v-next/hardhat-mocha-test-runner/src/task-action.ts
@@ -52,6 +52,10 @@ const testWithHardhat: NewTaskActionFunction<TestActionArguments> = async (
 
   const files = await getTestFiles(testFiles, hre.config);
 
+  if (files.length === 0) {
+    return;
+  }
+
   const tsx = fileURLToPath(import.meta.resolve("tsx/esm"));
   process.env.NODE_OPTIONS = `--import ${tsx}`;
 
@@ -81,6 +85,8 @@ const testWithHardhat: NewTaskActionFunction<TestActionArguments> = async (
   }
   testsAlreadyRun = true;
 
+  console.log("Running Mocha tests");
+
   // This instructs Mocha to use the more verbose file loading infrastructure
   // which supports both ESM and CJS
   await mocha.loadFilesAsync();

diff --git a/v-next/hardhat-node-test-runner/src/task-action.ts b/v-next/hardhat-node-test-runner/src/task-action.ts
@@ -61,6 +61,10 @@ const testWithHardhat: NewTaskActionFunction<TestActionArguments> = async (
 
   const files = await getTestFiles(testFiles, hre.config);
 
+  if (files.length === 0) {
+    return 0;
+  }
+
   const tsx = fileURLToPath(import.meta.resolve("tsx/esm"));
   process.env.NODE_OPTIONS = `--import ${tsx}`;
 
@@ -79,6 +83,8 @@ const testWithHardhat: NewTaskActionFunction<TestActionArguments> = async (
       testOnlyMessage,
     });
 
+    console.log("Running node:test tests");
+
     const reporterStream = run(nodeTestOptions)
       .on("test:fail", (event) => {
         if (event.details.type === "suite") {

diff --git a/v-next/hardhat/src/internal/builtin-plugins/solidity-test/task-action.ts b/v-next/hardhat/src/internal/builtin-plugins/solidity-test/task-action.ts
@@ -34,6 +34,12 @@ const runSolidityTests: NewTaskActionFunction<TestActionArguments> = async (
     )
   ).filter((artifact) => artifact !== undefined);
 
+  if (testSuiteIds.length === 0) {
+    return;
+  }
+
+  console.log("Running Solidity tests");
+
   const config = {
     projectRoot: hre.config.paths.root,
   };

diff --git a/v-next/hardhat/src/internal/builtin-plugins/test/task-action.ts b/v-next/hardhat/src/internal/builtin-plugins/test/task-action.ts
@@ -19,9 +19,6 @@ const runAllTests: NewTaskActionFunction<TestActionArguments> = async (
   }
 
   for (const subtask of thisTask.subtasks.values()) {
-    console.log(
-      chalk.bold(`Running the subtask "${formatTaskId(subtask.id)}"`),
-    );
     await subtask.run({ noCompile: true });
     console.log();
   }