tests: use relative path in utils contracts

[smol-ninja]

Closes #1168

[andreivladbrg]

didn't we decide to publish 2.0.1? why no version bump in package.json file, and no CHANGELOG update?

also, i was looking at the DeployOptimized utils test contract, this file assumes there is a out-optimized profile under foundry configuration. which might be very unlikely, thus, should we mark it as .t.sol instead?

[smol-ninja]

didn't we decide to publish 2.0.1? why no version bump in package.json file, and no CHANGELOG update?

Will do. I missed it.

DeployOptimized utils test contract, this file assumes there is a out-optimized profile under foundry configuration

Its not looking for the profile, but for the out-optimized directory, which we are already publishing under the artifacts name. Loading from artifacts will be complicated though so happy to rename it to .t.sol.

But, instead of renaming it to DeployOptimized.t.sol, what if I add it to .npmignore? Base imports from DeployOptimized.t.sol and importing from a .t.sol file looks weird.

[andreivladbrg]

Its not looking for the profile, but for the out-optimized directory, which we are already publishing under the artifacts name. Loading from artifacts will be complicated though so happy to rename it to .t.sol.

it is not specifically for the profile, but in order to have an out-optimized directory, we need that profile. otherwise, if we simply run forge build, foundry would create only the out directory (test it with bun run clean && forge b).

i’m not saying to extract it from artifacts, i’m just saying it’s very unlikely for someone to have this profile (which would generate the above directory) and actually use this file (similarly to Precompiles). thus simply rename it to .t.sol (which is already ignored in .npmignore)

importing from a .t.sol file looks weird.

why? :)) all the other files that are not under utils use this format, and they don’t look weird. (e.g. Base.t.sol)

[smol-ninja]

made all the changes. LMK if it looks good now.

[andreivladbrg]

👍