Add splitTestsCompilation solidity setting (8): Mocha updates

[alcuadrado]

Just an update to how build is run.

[changeset-bot]

⚠️ No Changeset found

Latest commit: c0f9741

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[Copilot]

Pull request overview

Updates Hardhat Mocha’s test:mocha task to invoke the build task with a noTests flag that reflects the solidity.splitTestsCompilation setting, and adds regression tests to validate the build invocation behavior.

Changes:

• Pass noTests: hre.config.solidity.splitTestsCompilation to the build task when running test:mocha.
• Add tests that override the build task to capture and assert the noTests argument under different configs (splitTestsCompilation true/false, noCompile true).

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Description
packages/hardhat-mocha/src/task-action.ts	Adjusts build invocation args based on solidity.splitTestsCompilation.
packages/hardhat-mocha/test/index.ts	Adds tests to assert build is called with the expected noTests arg and skipped when noCompile is true.

[alcuadrado]

Please ignore the spellcheck failure. The spec file will be deleted before merging.

[Copilot]

Pull request overview

Updates the Hardhat Mocha plugin’s test:mocha task to invoke the build task with noTests derived from solidity.splitTestsCompilation, and adds regression tests to ensure the correct build arguments are used under different configurations.

Changes:

• Pass noTests = hre.config.solidity.splitTestsCompilation when running build from the Mocha test task.
• Add tests that override the build task to capture invocation args and verify noTests behavior.
• Add a test to ensure compilation is skipped when noCompile: true.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File	Description
packages/hardhat-mocha/src/task-action.ts	Adjusts build invocation to set noTests based on solidity.splitTestsCompilation.
packages/hardhat-mocha/test/index.ts	Adds tests that capture and assert build task arguments for different config combinations.

[Copilot]

Pull request overview

Updates the Hardhat Mocha plugin’s build invocation to respect the new solidity.splitTestsCompilation setting, aligning its behavior with other built-in tasks and adding regression tests to confirm the build task arguments.

Changes:

• Change Mocha task action to call build with noTests derived from hre.config.solidity.splitTestsCompilation.
• Add tests that override the build task and assert how noTests is passed depending on splitTestsCompilation.
• Add a noCompile test to ensure compilation is skipped when requested.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File	Description
packages/hardhat-mocha/src/task-action.ts	Adjusts build invocation to set noTests based on solidity.splitTestsCompilation.
packages/hardhat-mocha/test/index.ts	Adds tests to verify build is invoked with the expected noTests flag (and skipped when noCompile is true).

[Copilot]

Pull request overview

Updates the Hardhat Mocha plugin’s test mocha task to invoke the build task with a noTests flag that depends on solidity.splitTestsCompilation, aligning build behavior with the new Solidity setting and adding coverage for this behavior.

Changes:

• Update the Mocha task action to call build with noTests: hre.config.solidity.splitTestsCompilation.
• Extend the Mocha plugin test suite to assert build is called with the expected noTests value for both splitTestsCompilation states and that noCompile skips compilation.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/hardhat-mocha/src/task-action.ts	Adjusts build invocation to pass noTests based on solidity.splitTestsCompilation.
packages/hardhat-mocha/test/index.ts	Adds tests validating build invocation arguments and noCompile behavior.

[Copilot]

Pull request overview

Updates Hardhat Mocha plugin behavior to invoke the build task with noTests aligned to the new solidity.splitTestsCompilation setting, and expands test coverage to validate the new build-invocation behavior.

Changes:

• Adjust test→mocha task action to call build with noTests = hre.config.solidity.splitTestsCompilation.
• Add test cases that assert build is invoked with noTests: false when splitTestsCompilation is disabled, and noTests: true when enabled.
• Add tests ensuring noCompile: true skips build in both configurations.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File	Description
packages/hardhat-mocha/test/index.ts	Adds build-invocation tests that capture build args and validate noTests / noCompile behaviors.
packages/hardhat-mocha/src/task-action.ts	Updates build invocation to set noTests based on solidity.splitTestsCompilation.