Update hardhat-mocha

Author: alcuadrado

packages/hardhat-mocha/src/task-action.ts

@@ -82,9 +82,8 @@ const testWithHardhat: NewTaskActionFunction<TestActionArguments> = async (
   perf.startPhase("Build");
 
   if (!noCompile) {
-    await hre.tasks.getTask("build").run({
-      noTests: true,
-    });
+    const noTests = hre.config.solidity.splitTestsCompilation;
+    await hre.tasks.getTask("build").run({ noTests });
     console.log();
   }
 

packages/hardhat-mocha/test/index.ts

@@ -6,6 +6,9 @@ import {
   assertRejectsWithHardhatError,
   useFixtureProject,
 } from "@nomicfoundation/hardhat-test-utils";
+import { overrideTask } from "hardhat/config";
+
+import HardhatMochaPlugin from "../src/index.js";
 
 describe("Hardhat Mocha plugin", () => {
   describe("Success", () => {
@@ -51,4 +54,79 @@ describe("Hardhat Mocha plugin", () => {
       );
     });
   });
+
+  describe("build invocation", () => {
+    useFixtureProject("test-project");
+
+    function buildArgCaptor() {
+      const buildArgs: any[] = [];
+      const buildOverride = overrideTask("build")
+        .setAction(async () => {
+          return {
+            default: (args: any) => {
+              buildArgs.push(args);
+              return { contractRootPaths: [], testRootPaths: [] };
+            },
+          };
+        })
+        .build();
+      return { buildArgs, buildOverride };
+    }
+
+    it("should call build without noTests when splitTestsCompilation is false", async () => {
+      const { createHardhatRuntimeEnvironment } = await import("hardhat/hre");
+
+      const { buildArgs, buildOverride } = buildArgCaptor();
+      const hre = await createHardhatRuntimeEnvironment({
+        plugins: [HardhatMochaPlugin],
+        tasks: [buildOverride],
+      });
+
+      // The task may throw because of the ESM re-run guard, but we only
+      // care about the build invocation args captured before that point.
+      try {
+        await hre.tasks.getTask(["test", "mocha"]).run({});
+      } catch {}
+
+      assert.equal(buildArgs.length, 1);
+      assert.equal(buildArgs[0].noTests, false);
+    });
+
+    it("should call build with noTests when splitTestsCompilation is true", async () => {
+      const { createHardhatRuntimeEnvironment } = await import("hardhat/hre");
+
+      const { buildArgs, buildOverride } = buildArgCaptor();
+      const hre = await createHardhatRuntimeEnvironment({
+        solidity: {
+          version: "0.8.28",
+          splitTestsCompilation: true,
+        },
+        plugins: [HardhatMochaPlugin],
+        tasks: [buildOverride],
+      });
+
+      try {
+        await hre.tasks.getTask(["test", "mocha"]).run({});
+      } catch {}
+
+      assert.equal(buildArgs.length, 1);
+      assert.equal(buildArgs[0].noTests, true);
+    });
+
+    it("should skip compilation when noCompile is true regardless of splitTestsCompilation", async () => {
+      const { createHardhatRuntimeEnvironment } = await import("hardhat/hre");
+
+      const { buildArgs, buildOverride } = buildArgCaptor();
+      const hre = await createHardhatRuntimeEnvironment({
+        plugins: [HardhatMochaPlugin],
+        tasks: [buildOverride],
+      });
+
+      try {
+        await hre.tasks.getTask(["test", "mocha"]).run({ noCompile: true });
+      } catch {}
+
+      assert.equal(buildArgs.length, 0);
+    });
+  });
 });