Add splitTestsCompilation solidity setting (10): hardhat-ignition updates

[alcuadrado]

Just an update to how build is run

[changeset-bot]

⚠️ No Changeset found

Latest commit: 24ef213

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-ignition’s ignition deploy and ignition visualize tasks to invoke Hardhat’s build task with noTests aligned to the new solidity.splitTestsCompilation setting, and adds regression tests to validate the build invocation args.

Changes:

• Make ignition visualize pass noTests: hre.config.solidity.splitTestsCompilation to build.
• Make ignition deploy pass noTests: hre.config.solidity.splitTestsCompilation to build (keeping defaultBuildProfile: "production").
• Add tests covering both splitTestsCompilation: false and true for deploy/visualize build invocations.

Reviewed changes

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

File	Description
packages/hardhat-ignition/src/internal/tasks/visualize.ts	Adjusts build invocation args to respect solidity.splitTestsCompilation.
packages/hardhat-ignition/src/internal/tasks/deploy.ts	Adjusts build invocation args to respect solidity.splitTestsCompilation while preserving production build profile usage.
packages/hardhat-ignition/test/plan/build-invocation.ts	Adds tests ensuring ignition visualize calls build with the correct noTests value.
packages/hardhat-ignition/test/deploy/build-invocation.ts	Adds tests ensuring ignition deploy calls build with the correct noTests value and production profile.

[alcuadrado]

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

[Copilot]

Pull request overview

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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

[Copilot]

Pull request overview

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