fix: Correct FOUNDRY_BIN_URL and FOUNDRY_BIN_PATH

[thepluck]

What 💻

• Correcting FOUNDRY_BIN_URL and FOUNDRY_BIN_PATH.

Why ✋

• The update() function in the script install original foundryup instead of update foundryup-zksync.

Evidence 📷

Documentation 📚

Please ensure the following before submitting your PR:

• Check if these changes affect any documented features or workflows.
• Update the book if these changes affect any documented features or workflows.

[Karrq]

Thanks for spotting this! :)

[thepluck]

Thanks for spotting this! :)

No problem :D Hope you guys produce a new version with better performance, testing is extremely slow rn.

[Karrq]

What kind of tests are you running exactly? We are aware of the general performance issue with EraVM, but unless you are running fuzz tests (or invariant tests) those shouldn't take significantly longer, please open an issue.

[thepluck]

What kind of tests are you running exactly? We are aware of the general performance issue with EraVM, but unless you are running fuzz tests (or invariant tests) those shouldn't take significantly longer, please open an issue.

Those tests are fuzz tests, but there are only 26 of them with runs = 16 in foundry.toml and it takes ~ 2 hours to finish testing. Compared to the normal foundry, we run 200+ fuzz tests, it only takes about 1 hour to finish with version < 1.0, and under 30 minutes for 1.0. The project repo is private for now so I can not share more information about it. But anyway, you guys did good work bringing a tool that eliminates the need for Hardhat in foundry projects when building multi-chain Dapps. For anyone curious, I am building Kyber Earn that will live on ZkSync Era soon.

[Karrq]

I see, yeah unfortunately not a lot we can do there other than recommending using a different profile with less fuzz runs for development and keep a higher number of fuzz runs in CI for added security.
Please feel free to open issues in the future if you encounter bugs or unexpected behaviour!